On Thu, 2016-09-29 at 13:56 -0700, Joe Perches wrote: > It doesn't matter match either way to me. > > The case for the unnecessary multiply with <= gcc 4.8 was > removed with: > > commit 91c6a05f72a996bee5133e76374ab3ad7d3b9b72 > Author: Alexey Dobriyan <adobriyan@xxxxxxxxx> > Date: Tue Jul 26 15:22:08 2016 -0700 > > mm: faster kmalloc_array(), kcalloc() > > When both arguments to kmalloc_array() or kcalloc() are known at compile > time then their product is known at compile time but search for kmalloc > cache happens at runtime not at compile time. > > Link: http://lkml.kernel.org/r/20160627213454.GA2440@xxxxxxxxxxxxxxx > Signed-off-by: Alexey Dobriyan <adobriyan@xxxxxxxxx> > Cc: Christoph Lameter <cl@xxxxxxxxx> > Cc: Pekka Enberg <penberg@xxxxxxxxxx> > Cc: David Rientjes <rientjes@xxxxxxxxxx> > Cc: Joonsoo Kim <iamjoonsoo.kim@xxxxxxx> > Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > Signed-off-by: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> You've lost me. Why does this stop you fixing an apparently wrong checkpatch rule, crude as parts of it are (ie, uppercase identifier must be a constant)? Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
