lkml.org 
[lkml]   [2014]   [Feb]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] slub: fix false-positive lockdep warning in free_partial()
On Tue, 4 Feb 2014, Christoph Lameter wrote:

> > Although this cannot actually result in a race, because on cache
> > destruction there should not be any concurrent frees or allocations from
> > the cache, let's add spin_lock/unlock to free_partial() just to keep
> > lockdep happy.
>
> Please add a comment that says this in the source so we know why this was
> added.
>

Makes sense since there is a comment there already saying we don't need
the lock, then with this patch we end up taking it away. The nice thing
is that there should be no lock contention here :)

I'm not sure we need to disable irqs as in the patch, though.


\
 
 \ /
  Last update: 2014-02-05 02:21    [W:0.186 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site