On Mon, Jun 29, 2009 at 05:06:37PM -0400, Christoph Lameter wrote: > On Fri, 26 Jun 2009, Pekka Enberg wrote: > > > Yes, but AFAICT, SLUB_DEBUG doesn't respect the given alignment without > > my patch. > > The size parameter to calculate_alignment has no relevance unless the size > is smaller than half of the alignment. Then for some reason we do not > properly align the stuff. I never liked that.... It is properly aligned, if it was not, then it would be a bug. It should always align to *at least* the requested alignment. SLAB_HWCACHE_ALIGN is defined to align the object such that it occupies the minimal number of cache lines. This is not subjective thing to like or dislike, but just fact because that is how it was implemented from the start and that is what callers expect. If you want to minimise false sharing, then normal way would be to request internode cacheline alignment, right? This is very widely used. -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
