In article <20040209112007.08023ba6.davem@xxxxxxxxxx> (at Mon, 9 Feb 2004
11:20:07 -0800), "David S. Miller" <davem@xxxxxxxxxx> says:
> > BTW,
> > 1. do we really need this ALIGN?
> > 2. should 16 be BYTES_PER_WORD (in mm/slab.c)?
> I think this is complete nonsense, and that the alignment is not
> necessary. I can't even come up with a performance reason as SLAB
> is going to align things to hw cache line size anyways.
> Else let's just remove this bogus 16 byte alignment in the
> kmem_cache_create() call.
Let's kill it. It is very likely safe.
===== net/core/neighbour.c 1.24 vs edited =====
--- 1.24/net/core/neighbour.c Tue Jan 20 14:31:23 2004
+++ edited/net/core/neighbour.c Tue Feb 10 13:01:15 2004
@@ -1164,8 +1164,7 @@
tbl->kmem_cachep = kmem_cache_create(tbl->id,
- (tbl->entry_size +
- 15) & ~15,
tbl->lock = RW_LOCK_UNLOCKED;
Hideaki YOSHIFUJI @ USAGI Project <yoshfuji@xxxxxxxxxxxxxx>
GPG FP: 9022 65EB 1ECF 3AD1 0BDF 80D8 4807 F894 E062 0EEA