Re: [PATCH 3/3] slub: release a lock if freeing object with a lock is failed in __slab_free()

2012-07-08 Thread JoonSoo Kim
2012/7/7 Christoph Lameter : > On Fri, 6 Jul 2012, JoonSoo Kim wrote: > >> >> At CPU2, we don't need lock anymore, because this slab already in partial >> >> list. >> > >> > For that scenario we could also simply do a trylock there and redo >> > the loop if we fail. But still what guarantees that

Re: [PATCH 3/3] slub: release a lock if freeing object with a lock is failed in __slab_free()

2012-07-08 Thread JoonSoo Kim
2012/7/7 Christoph Lameter c...@linux.com: On Fri, 6 Jul 2012, JoonSoo Kim wrote: At CPU2, we don't need lock anymore, because this slab already in partial list. For that scenario we could also simply do a trylock there and redo the loop if we fail. But still what guarantees that

Re: [PATCH 3/3] slub: release a lock if freeing object with a lock is failed in __slab_free()

2012-07-06 Thread Christoph Lameter
On Fri, 6 Jul 2012, JoonSoo Kim wrote: > >> At CPU2, we don't need lock anymore, because this slab already in partial > >> list. > > > > For that scenario we could also simply do a trylock there and redo > > the loop if we fail. But still what guarantees that another process will > > not modify

Re: [PATCH 3/3] slub: release a lock if freeing object with a lock is failed in __slab_free()

2012-07-06 Thread Christoph Lameter
On Fri, 6 Jul 2012, JoonSoo Kim wrote: At CPU2, we don't need lock anymore, because this slab already in partial list. For that scenario we could also simply do a trylock there and redo the loop if we fail. But still what guarantees that another process will not modify the page