RE: [PATCH v4 2/2] pid: Remove pidhash

2017-10-11 Thread Luck, Tony
>DEFINE(IA64_UPID_SHIFT, 5); > > Grepping for IA64_UPID_SHIFT leads us to some assembly > code implementing fsys_getpid (why is that in assembly?!): The fast system call path has a whole host of serious restrictions on what it can touch. See Documentation/ia64/fsys.txt. Why is getpid() a

Re: [PATCH v4 2/2] pid: Remove pidhash

2017-10-11 Thread Rik van Riel
On Wed, 2017-10-11 at 17:47 +0800, kbuild test robot wrote: > > > '__compiletime_assert_33' declared with attribute error: > > > BUILD_BUG_ON failed: sizeof(struct upid) != 32 > >  _compiletime_assert(condition, msg, __compiletime_assert_, > __LINE__) >    

Re: [PATCH v4 2/2] pid: Remove pidhash

2017-10-11 Thread kbuild test robot
Hi Gargi, [auto build test ERROR on linus/master] [also build test ERROR on v4.14-rc4] [cannot apply to next-20171009] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Gargi-Sharma/pid-Replace-pid

[PATCH v4 2/2] pid: Remove pidhash

2017-10-09 Thread Gargi Sharma
pidhash is no longer required as all the information can be looked up from idr tree. nr_hashed represented the number of pids that had been hashed. Since, nr_hashed and PIDNS_HASH_ADDING are no longer relevant, it has been renamed to pid_allocated and PIDNS_ADDING respectively. Signed-off-by: Garg