RE: [PATCH] proc: move the adding option Ngid to the end of proc/PID/status

From: Wang, Xiaoming
Date: Thu Apr 16 2015 - 23:16:28 EST


Dear tejun

> -----Original Message-----
> From: Tejun Heo [mailto:htejun@xxxxxxxxx] On Behalf Of Tejun Heo
> Sent: Friday, April 17, 2015 10:56 AM
> To: Wang, Xiaoming
> Cc: akpm@xxxxxxxxxxxxxxxxxxxx; oleg@xxxxxxxxxx;
> andriy.shevchenko@xxxxxxxxxxxxxxx; linux@xxxxxxxxxxxxxxxxxx;
> ebiederm@xxxxxxxxxxxx; eparis@xxxxxxxxxx; chenhanxiao@xxxxxxxxxxxxxx;
> tglx@xxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Schallberger, Timothy M;
> Zhang, Dongxing
> Subject: Re: [PATCH] proc: move the adding option Ngid to the end of
> proc/PID/status
>
> On Fri, Apr 17, 2015 at 10:13:15AM +0800, Wang Xiaoming wrote:
> > Move debugging has been done and the following Kernel issue was found
> > with a number of applications.
> > Take a look at: (even though the comments are for Weibo.browser they
> > also pertain to other apps that use Libsecuritysdk-x.x.x.so
> >
> > In kernel(3.14) is a little different than before it will generate
> > /proc/PID/status in this way:
> > Name: a.weibo.browser
> > State: T (stopped)
> > Tgid: 8487
> > Ngid: 0 ---- add in kernel after (3.11 maybe)
>
> Well, that's kinda hilarious and I don't know. 3.11 is way back and what if there
> are others depending on the current ordering? Both situations kinda suck so
> what's the point of changing?
>
I am not sure exactly which kernel introduced this Ngid.
I check the git and found it added in
commit e29cf08b05dc0b8151d65704d96d525a9e179a6b
And this change has destroyed the order already.
Moving the adding option Ngid to the end of proc/PID/status
is to keep order .
> --
> tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/