Re: [PATCH -mm1 1/2] fixing idr_find() locking

2007-09-28 Thread Jarek Poplawski
On Fri, Sep 28, 2007 at 09:08:47AM +0200, Nadia Derbey wrote: > Jarek Poplawski wrote: > >On Thu, Sep 27, 2007 at 04:33:55PM +0200, [EMAIL PROTECTED] wrote: > > > >>[PATCH 01/02] > >> > >> > >>This is a patch that fixes the way idr_find() used to be called in > >>ipc_lock(): > >>in all the paths t

Re: [PATCH -mm1 1/2] fixing idr_find() locking

2007-09-28 Thread Nadia Derbey
Jarek Poplawski wrote: On Thu, Sep 27, 2007 at 04:33:55PM +0200, [EMAIL PROTECTED] wrote: [PATCH 01/02] This is a patch that fixes the way idr_find() used to be called in ipc_lock(): in all the paths that don't imply an update of the ipcs idr, it was called without the idr tree being locked.

Re: [PATCH -mm1 1/2] fixing idr_find() locking

2007-09-27 Thread Andrew Morton
On Fri, 28 Sep 2007 08:31:44 +0200 Jarek Poplawski <[EMAIL PROTECTED]> wrote: > On Thu, Sep 27, 2007 at 04:33:55PM +0200, [EMAIL PROTECTED] wrote: > > [PATCH 01/02] > > > > > > This is a patch that fixes the way idr_find() used to be called in > > ipc_lock(): > > in all the paths that don't imp

Re: [PATCH -mm1 1/2] fixing idr_find() locking

2007-09-27 Thread Jarek Poplawski
On Thu, Sep 27, 2007 at 04:33:55PM +0200, [EMAIL PROTECTED] wrote: > [PATCH 01/02] > > > This is a patch that fixes the way idr_find() used to be called in ipc_lock(): > in all the paths that don't imply an update of the ipcs idr, it was called > without the idr tree being locked. > > The change

[PATCH -mm1 1/2] fixing idr_find() locking

2007-09-27 Thread Nadia . Derbey
[PATCH 01/02] This is a patch that fixes the way idr_find() used to be called in ipc_lock(): in all the paths that don't imply an update of the ipcs idr, it was called without the idr tree being locked. The changes are: . in ipc_ids, the mutex has been changed into a reader/writer semaphore.