Re: [PATCH] slab: fix the DEADLOCK issue on l3 alien lock

2012-09-11 Thread Pekka Enberg
On Tue, Sep 11, 2012 at 5:50 AM, Michael Wang wrote: > On 09/08/2012 04:39 PM, Pekka Enberg wrote: >> On Fri, Sep 7, 2012 at 1:29 AM, Paul E. McKenney >> wrote: >>> On Thu, Sep 06, 2012 at 11:05:11AM +0800, Michael Wang wrote: On 09/05/2012 09:55 PM, Christoph Lameter wrote: > On Wed, 5

Re: [PATCH] slab: fix the DEADLOCK issue on l3 alien lock

2012-09-10 Thread Michael Wang
On 09/08/2012 04:39 PM, Pekka Enberg wrote: > On Fri, Sep 7, 2012 at 1:29 AM, Paul E. McKenney > wrote: >> On Thu, Sep 06, 2012 at 11:05:11AM +0800, Michael Wang wrote: >>> On 09/05/2012 09:55 PM, Christoph Lameter wrote: On Wed, 5 Sep 2012, Michael Wang wrote: > Since the cachep and

Re: [PATCH] slab: fix the DEADLOCK issue on l3 alien lock

2012-09-08 Thread Pekka Enberg
On Fri, Sep 7, 2012 at 1:29 AM, Paul E. McKenney wrote: > On Thu, Sep 06, 2012 at 11:05:11AM +0800, Michael Wang wrote: >> On 09/05/2012 09:55 PM, Christoph Lameter wrote: >> > On Wed, 5 Sep 2012, Michael Wang wrote: >> > >> >> Since the cachep and cachep->slabp_cache's l3 alien are in the same lo

Re: [PATCH] slab: fix the DEADLOCK issue on l3 alien lock

2012-09-06 Thread Paul E. McKenney
On Thu, Sep 06, 2012 at 11:05:11AM +0800, Michael Wang wrote: > On 09/05/2012 09:55 PM, Christoph Lameter wrote: > > On Wed, 5 Sep 2012, Michael Wang wrote: > > > >> Since the cachep and cachep->slabp_cache's l3 alien are in the same lock > >> class, > >> fake report generated. > > > > Ahh... Th

Re: [PATCH] slab: fix the DEADLOCK issue on l3 alien lock

2012-09-05 Thread Michael Wang
On 09/05/2012 09:55 PM, Christoph Lameter wrote: > On Wed, 5 Sep 2012, Michael Wang wrote: > >> Since the cachep and cachep->slabp_cache's l3 alien are in the same lock >> class, >> fake report generated. > > Ahh... That is a key insight into why this occurs. > >> This should not happen since w

Re: [PATCH] slab: fix the DEADLOCK issue on l3 alien lock

2012-09-05 Thread Christoph Lameter
On Wed, 5 Sep 2012, Michael Wang wrote: > Since the cachep and cachep->slabp_cache's l3 alien are in the same lock > class, > fake report generated. Ahh... That is a key insight into why this occurs. > This should not happen since we already have init_lock_keys() which will > reassign the lock