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

Wang, Xiaoming
2015-04-17T03:15:49Z

Dear tejun

> -----Original Message-----
> From: Tejun Heo [mailto:htejun@gmail.com] On Behalf Of Tejun Heo
> Sent: Friday, April 17, 2015 10:56 AM
> To: Wang, Xiaoming
> Cc: akpm@linux-foundation.org; oleg@redhat.com;
> andriy.shevchenko@linux.intel.com; linux@rasmusvillemoes.dk;
> ebiederm@xmission.com; eparis@redhat.com; chenhanxiao@cn.fujitsu.com;
> tglx@linutronix.de; linux-kernel@vger.kernel.org; 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