Re: [patch 05/41] cpu alloc: Percpu_counter conversion

From: Christoph Lameter
Date: Fri May 30 2008 - 13:54:18 EST


On Fri, 30 May 2008, Rusty Russell wrote:

> On Friday 30 May 2008 13:56:25 Christoph Lameter wrote:
> > Use cpu_alloc instead of allocpercpu.
>
> These patches seem like useless churn.
>
> Plus, the new code is uglier than the old code :(

It drastically reduces the memory size f.e. 4 byte allocations require
SLAB f.e. to allocate a 32 byte chunk. This reduces memory requirements by
32/4 = 8 times.

Plus the per cpu counters allocated in order are likely placed in the same
cacheline (whereas the slab allocators avoid placing multiple objects in
the same cacheline). Reduces cache footprint.

--
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/