Re: [PATCH 4/4] slub: refactoring unfreeze_partials()

From: Joonsoo Kim
Date: Fri Jun 01 2012 - 08:29:07 EST




On Thu, 17 May 2012, Christoph Lameter wrote:

On Fri, 18 May 2012, Joonsoo Kim wrote:

I think that these are disadvantages of current implementation,
so I do refactoring unfreeze_partials().

The reason the current implementation is so complex is to avoid races. The
state of the list and the state of the partial pages must be consistent at
all times.

Minimizing code in do {} while loop introduce a reduced fail rate
of cmpxchg_double_slab. Below is output of 'slabinfo -r kmalloc-256'
when './perf stat -r 33 hackbench 50 process 4000 > /dev/null' is done.

Looks good. If I can convince myself that this does not open up any
new races then I may ack it.

This is a reminder mail.
Would u give me some comments for this please?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/