On Mon, 27 Jul 2009, David Rientjes wrote: > My patch is already in Pekka's slab-2.6.git tree at > http://git.kernel.org/?p=linux/kernel/git/penberg/slab-2.6.git;a=commit;h=fa5ec8a1f66f3c2a3af723abcf8085509c9ee682 > > You had proposed http://marc.info/?l=linux-kernel&m=124725166205814, which > moves the mask to kmem_cache_open() and calls calculate_sizes() twice. > That eliminates DEBUG_FLAGS_SIZE, but I don't see that define as being > troublesome since we must define DEBUG_FLAGS to specify what options add > metdata anyway.
My prosal was to use the size and objsize parameters. You would only have to call calculate_sizes() twice when the comparison of the order of size and objsize would be different. Doing so would simplify additing future flags. If you do your own calculations (like in the patch) then you have to replicate the size calculation from calculate_sizes() somehow. Is the duplicate calculation really accurate regarding alignment and other special casing? -- To unsubscribe from this list: send the line "unsubscribe kernel-testers" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
