Re: [PATCH 1/2] ipc,shm: move BUG_ON check into shm_lock

2015-06-04 Thread Davidlohr Bueso
On Thu, 2015-06-04 at 20:35 +0200, Manfred Spraul wrote: > Hi Davidlohr, > > On 05/30/2015 02:03 AM, Davidlohr Bueso wrote: > > Upon every shm_lock call, we BUG_ON if an error was returned, > > indicating racing either in idr or in RMID. Move this logic > > into the locking. > > > > Signed-off-by:

Re: [PATCH 1/2] ipc,shm: move BUG_ON check into shm_lock

2015-06-04 Thread Davidlohr Bueso
On Thu, 2015-06-04 at 20:35 +0200, Manfred Spraul wrote: > Hi Davidlohr, > > On 05/30/2015 02:03 AM, Davidlohr Bueso wrote: > > Upon every shm_lock call, we BUG_ON if an error was returned, > > indicating racing either in idr or in RMID. Move this logic > > into the locking. > > > > Signed-off-by:

Re: [PATCH 1/2] ipc,shm: move BUG_ON check into shm_lock

2015-06-04 Thread Manfred Spraul
Hi Davidlohr, On 05/30/2015 02:03 AM, Davidlohr Bueso wrote: Upon every shm_lock call, we BUG_ON if an error was returned, indicating racing either in idr or in RMID. Move this logic into the locking. Signed-off-by: Davidlohr Bueso --- ipc/shm.c | 11 +++ 1 file changed, 7 insertion

Re: [PATCH 1/2] ipc,shm: move BUG_ON check into shm_lock

2015-06-03 Thread Davidlohr Bueso
On Mon, 2015-06-01 at 15:52 -0700, Andrew Morton wrote: > On Fri, 29 May 2015 17:03:05 -0700 Davidlohr Bueso wrote: > > > Upon every shm_lock call, we BUG_ON if an error was returned, > > indicating racing either in idr or in RMID. Move this logic > > into the locking. > > > > ... > > > > --- a/

Re: [PATCH 1/2] ipc,shm: move BUG_ON check into shm_lock

2015-06-03 Thread Davidlohr Bueso
On Mon, 2015-06-01 at 15:52 -0700, Andrew Morton wrote: > On Fri, 29 May 2015 17:03:05 -0700 Davidlohr Bueso wrote: > > > Upon every shm_lock call, we BUG_ON if an error was returned, > > indicating racing either in idr or in RMID. Move this logic > > into the locking. > > > > ... > > > > --- a/

Re: [PATCH 1/2] ipc,shm: move BUG_ON check into shm_lock

2015-06-01 Thread Andrew Morton
On Fri, 29 May 2015 17:03:05 -0700 Davidlohr Bueso wrote: > Upon every shm_lock call, we BUG_ON if an error was returned, > indicating racing either in idr or in RMID. Move this logic > into the locking. > > ... > > --- a/ipc/shm.c > +++ b/ipc/shm.c > @@ -155,8 +155,14 @@ static inline struct sh

[PATCH 1/2] ipc,shm: move BUG_ON check into shm_lock

2015-05-29 Thread Davidlohr Bueso
Upon every shm_lock call, we BUG_ON if an error was returned, indicating racing either in idr or in RMID. Move this logic into the locking. Signed-off-by: Davidlohr Bueso --- ipc/shm.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/ipc/shm.c b/ipc/shm.c index 6d7