Review of last PR

This commit is contained in:
Gregory Nutt 2016-07-01 13:22:36 -06:00
parent e65249b13d
commit 0e4c247e1d
2 changed files with 4 additions and 7 deletions

View File

@ -405,7 +405,7 @@ config NSH_DISABLE_PS
config NSH_DISABLE_PSSTACKUSAGE
bool "Disable ps stack usage"
depends on !NSH_DISABLE_PS
depends on STACK_COLORATION && !NSH_DISABLE_PS
default n
---help---
Disable to save space and not pull in floating point routines

View File

@ -71,9 +71,6 @@
# undef HAVE_GROUPID
#endif
#if defined(CONFIG_STACK_COLORATION) && !defined(CONFIG_NSH_DISABLE_PSSTACKUSAGE)
# define PS_HAVE_STACKUSAGE
#endif
/****************************************************************************
* Private Types
****************************************************************************/
@ -269,7 +266,7 @@ static int ps_callback(FAR struct nsh_vtbl_s *vtbl, FAR const char *dirpath,
FAR char *nextline;
int ret;
int i;
#ifdef PS_HAVE_STACKUSAGE
#ifndef CONFIG_NSH_DISABLE_PSSTACKUSAGE
int stack_size;
int stack_used;
float stack_filled;
@ -390,7 +387,7 @@ static int ps_callback(FAR struct nsh_vtbl_s *vtbl, FAR const char *dirpath,
nsh_output(vtbl, "%-8s ", status.td_sigmask);
#endif
#ifdef PS_HAVE_STACKUSAGE
#ifndef CONFIG_NSH_DISABLE_PSSTACKUSAGE
/* Get the StackSize and StackUsed */
stack_size = 0;
@ -570,7 +567,7 @@ int cmd_ps(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
#ifndef CONFIG_DISABLE_SIGNALS
nsh_output(vtbl, "%-8s ", "SIGMASK");
#endif
#ifdef PS_HAVE_STACKUSAGE
#ifndef CONFIG_NSH_DISABLE_PSSTACKUSAGE
nsh_output(vtbl, "%6s ", "STACK");
nsh_output(vtbl, "%6s ", "USED");
nsh_output(vtbl, "%7s ", "FILLED");