[Toybox] ps on Android

Rob Landley rob at landley.net
Thu Jun 9 12:57:58 PDT 2016


The reason I hadn't already applied it is my local copy has extensive
changes (working on the "right fix") and this patch conflicts with it.

If I can't get it finished today I'll delete what I've done, apply this,
and consider the matter closed until my head's back above water.

Rob

On 06/09/2016 11:24 AM, enh wrote:
> i'll submit this (same patch as yesterday, minus the width workaround
> since that was fixed properly last night) to reduce the urgency, and
> revert when we have a real fix:
> 
>>From be7bbced789dbb80daa8fe2707ebe9736a2c0732 Mon Sep 17 00:00:00 2001
> From: Elliott Hughes <enh at google.com>
> Date: Wed, 8 Jun 2016 17:03:26 -0700
> Subject: [PATCH] HACK: restore Android ps behavior.
> 
> "NAME" is no longer doing what we want; "TNAME" is what "NAME" used to be,
> except that "TNAME" implies -T. This patch switches us over to "TNAME",
> disables the implicit -T.
> 
> Change-Id: I5553703d3939b24eaf39976162d2f75a591e1ce8
> ---
>  toys/posix/ps.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/toys/posix/ps.c b/toys/posix/ps.c
> index 4c66bcd..9402b77 100644
> --- a/toys/posix/ps.c
> +++ b/toys/posix/ps.c
> @@ -1194,7 +1194,7 @@ void ps_main(void)
>    else if (toys.optflags&FLAG_l)
>      not_o = "F,S,UID,%sPPID,C,PRI,NI,ADDR,SZ,WCHAN,TTY,TIME,CMD";
>    else if (CFG_TOYBOX_ON_ANDROID)
> -    not_o = "USER,%sPPID,VSIZE,RSS,WCHAN:10,ADDR:10=PC,S,NAME";
> +    not_o = "USER,%sPPID,VSIZE,RSS,WCHAN:10,ADDR:10=PC,S,TNAME";
>    sprintf(toybuf, not_o, (toys.optflags & FLAG_T) ? "PID,TID," : "PID,");
> 
>    // Init TT.fields. This only uses toybuf if TT.ps.o is NULL
> @@ -1227,7 +1227,7 @@ void ps_main(void)
>    if (!(toys.optflags&(FLAG_k|FLAG_M))) TT.show_process = (void *)show_ps;
>    TT.match_process = ps_match_process;
>    dt = dirtree_read("/proc",
> -    ((toys.optflags&FLAG_T) || (TT.bits&(_PS_TID|_PS_TCNT|_PS_TNAME)))
> +    ((toys.optflags&FLAG_T) || (TT.bits&(_PS_TID|_PS_TCNT/*|_PS_TNAME*/)))
>        ? get_threads : get_ps);
> 
>    if (toys.optflags&(FLAG_k|FLAG_M)) {
> 



More information about the Toybox mailing list