From f48b3c73b09a87d99fb79a7bd0803c18febebd7c Mon Sep 17 00:00:00 2001
From: David Alessio <david.alessio@yahoo.com>
Date: Fri, 15 Jul 2016 07:01:02 -0600
Subject: [PATCH] =?UTF-8?q?When=20Priority=20Inheritance=20is=20enabled,?=
 =?UTF-8?q?=20the=20format=20of=20/proc/<PID>/status=20changes=20to=20show?=
 =?UTF-8?q?=20both=20the=20current=20priority=20and=20the=20thread?=
 =?UTF-8?q?=E2=80=99s=20base=20priority.=20=20This=20messes=20up=20the=20f?=
 =?UTF-8?q?ormat=20of=20cmd=5Fps.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The attached patch fixes cmd_ps to ignore the base priority.
---
 nshlib/nsh_proccmds.c | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/nshlib/nsh_proccmds.c b/nshlib/nsh_proccmds.c
index 0ebbbe559..6851991ac 100644
--- a/nshlib/nsh_proccmds.c
+++ b/nshlib/nsh_proccmds.c
@@ -235,7 +235,17 @@ static void nsh_parse_statusline(FAR char *line,
     }
   else if (strncmp(line, g_priority, strlen(g_priority)) == 0)
     {
-      status->td_priority = nsh_trimspaces(&line[12]);
+      FAR char *ptr =
+          status->td_priority = nsh_trimspaces(&line[12]);
+
+      /* If priority inheritance is enabled, use current pri, ignore base */
+
+      while (isdigit(*ptr))
+        {
+          ++ptr;
+        }
+
+      *ptr = '\0';
     }
   else if (strncmp(line, g_scheduler, strlen(g_scheduler)) == 0)
     {
@@ -474,11 +484,11 @@ static int ps_callback(FAR struct nsh_vtbl_s *vtbl, FAR const char *dirpath,
       stack_filled = 10 * 100 * stack_used / stack_size;
     }
 
-  /* Additionally print a "!" if the stack is filled more than 80% */
+  /* Additionally print a '!' if the stack is filled more than 80% */
 
-  nsh_output(vtbl, "%3d.%1d%%%s ",
+  nsh_output(vtbl, "%3d.%1d%%%c ",
              stack_filled / 10, stack_filled % 10,
-             (stack_filled >= 10 * 80 ? "!" : " "));
+             (stack_filled >= 10 * 80 ? '!' : ' '));
 #endif
 #endif