Re: [PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-13 Thread Oleg Nesterov
On 10/12, Andrew Morton wrote: > > On Wed, 11 Oct 2017 20:54:32 -0400 Rik van Riel wrote: > > > Documenting the locking on all the existing code, plus the > > new code, seems a little out of scope of an Outreachy > > internship... > > I'm not referring to all the existing code!

Re: [PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-13 Thread Oleg Nesterov
On 10/12, Andrew Morton wrote: > > On Wed, 11 Oct 2017 20:54:32 -0400 Rik van Riel wrote: > > > Documenting the locking on all the existing code, plus the > > new code, seems a little out of scope of an Outreachy > > internship... > > I'm not referring to all the existing code! Just this new >

Re: [PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-12 Thread Andrew Morton
On Wed, 11 Oct 2017 20:54:32 -0400 Rik van Riel wrote: > On Wed, 2017-10-11 at 15:37 -0700, Andrew Morton wrote: > > On Wed, 11 Oct 2017 18:19:38 -0400 Gargi Sharma > > wrote: > > > > > This patch replaces the current bitmap implemetation for > > > Process

Re: [PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-12 Thread Andrew Morton
On Wed, 11 Oct 2017 20:54:32 -0400 Rik van Riel wrote: > On Wed, 2017-10-11 at 15:37 -0700, Andrew Morton wrote: > > On Wed, 11 Oct 2017 18:19:38 -0400 Gargi Sharma > > wrote: > > > > > This patch replaces the current bitmap implemetation for > > > Process ID allocation. Functions that are no

Re: [PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-11 Thread Rik van Riel
On Wed, 2017-10-11 at 15:37 -0700, Andrew Morton wrote: > On Wed, 11 Oct 2017 18:19:38 -0400 Gargi Sharma > wrote: > > > This patch replaces the current bitmap implemetation for > > Process ID allocation. Functions that are no longer required, > > for example, free_pidmap(),

Re: [PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-11 Thread Rik van Riel
On Wed, 2017-10-11 at 15:37 -0700, Andrew Morton wrote: > On Wed, 11 Oct 2017 18:19:38 -0400 Gargi Sharma > wrote: > > > This patch replaces the current bitmap implemetation for > > Process ID allocation. Functions that are no longer required, > > for example, free_pidmap(), alloc_pidmap(), etc.

Re: [PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-11 Thread Andrew Morton
On Wed, 11 Oct 2017 18:19:38 -0400 Gargi Sharma wrote: > This patch replaces the current bitmap implemetation for > Process ID allocation. Functions that are no longer required, > for example, free_pidmap(), alloc_pidmap(), etc. are removed. > The rest of the functions are

Re: [PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-11 Thread Andrew Morton
On Wed, 11 Oct 2017 18:19:38 -0400 Gargi Sharma wrote: > This patch replaces the current bitmap implemetation for > Process ID allocation. Functions that are no longer required, > for example, free_pidmap(), alloc_pidmap(), etc. are removed. > The rest of the functions are modified to use the

Re: [PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-11 Thread Rik van Riel
On Wed, 2017-10-11 at 18:19 -0400, Gargi Sharma wrote: > This patch replaces the current bitmap implemetation for > Process ID allocation. Functions that are no longer required, > for example, free_pidmap(), alloc_pidmap(), etc. are removed. > The rest of the functions are modified to use the IDR

Re: [PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-11 Thread Rik van Riel
On Wed, 2017-10-11 at 18:19 -0400, Gargi Sharma wrote: > This patch replaces the current bitmap implemetation for > Process ID allocation. Functions that are no longer required, > for example, free_pidmap(), alloc_pidmap(), etc. are removed. > The rest of the functions are modified to use the IDR

[PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-11 Thread Gargi Sharma
This patch replaces the current bitmap implemetation for Process ID allocation. Functions that are no longer required, for example, free_pidmap(), alloc_pidmap(), etc. are removed. The rest of the functions are modified to use the IDR API. The change was made to make the PID allocation less

[PATCH v6 1/2] pid: Replace pid bitmap implementation with IDR API

2017-10-11 Thread Gargi Sharma
This patch replaces the current bitmap implemetation for Process ID allocation. Functions that are no longer required, for example, free_pidmap(), alloc_pidmap(), etc. are removed. The rest of the functions are modified to use the IDR API. The change was made to make the PID allocation less