[lkml]   [2012]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] slub: change cmpxchg_double_slab in get_freelist() to __cmpxchg_double_slab
On Mon, 14 May 2012, David Rientjes wrote:
> > On Fri, 11 May 2012, Joonsoo Kim wrote:
> >
> > > get_freelist() is only called by __slab_alloc with interrupt disabled,
> > > so __cmpxchg_double_slab is suitable.
> > >
> > > Signed-off-by: Joonsoo Kim <>
> >
> > Acked-by: David Rientjes <>

On Thu, 31 May 2012, David Rientjes wrote:
> Pekka, did you want to pick this up so it can get into linux-next?

We now made get_freelist() *require* interrupts to be disabled which
deserves a comment, no?

Also, what do we gain from patches like this? It's somewhat
counterintuitive that we have a function with "cmpxchg" in it which is not
always atomic (i.e. you need to have interrupts disabled).

IIRC, there was even a long rant about this by Linus but I'm unable to
find it in my email archives.


 \ /
  Last update: 2012-06-01 09:01    [W:0.155 / U:0.988 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site