On Wed, 17 Apr 2013 17:03:21 -0700 (PDT) David Rientjes <rient...@google.com> wrote:
> On Wed, 17 Apr 2013, Steven Rostedt wrote: > > > The slab.c code has a size check macro that checks the size of the > > following structs: > > > > struct arraycache_init > > struct kmem_list3 > > > > The index_of() function that takes the sizeof() of the above two structs > > and does an unnecessary __builtin_constant_p() on that. As sizeof() will > > always end up being a constant making this always be true. The code is > > not incorrect, but it just adds added complexity, and confuses users and > > wastes the time of reviewers of the code, who spends time trying to > > figure out why the builtin_constant_p() was used. > > > > This patch is just a clean up that makes the index_of() code a little > > bit less complex. > > > > Signed-off-by: Steven Rostedt <rost...@goodmis.org> > > Acked-by: David Rientjes <rient...@google.com> > > Adding Pekka to the cc. I ducked this patch because it seemed rather pointless - but a little birdie told me that there is a secret motivation which seems pretty reasonable to me. So I shall await chirp-the-second, which hopefully will have a fuller and franker changelog ;) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
