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 <[email protected]> > 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/[email protected] > Signed-off-by: Alexey Dobriyan <[email protected]> > Cc: Christoph Lameter <[email protected]> > Cc: Pekka Enberg <[email protected]> > Cc: David Rientjes <[email protected]> > Cc: Joonsoo Kim <[email protected]> > Signed-off-by: Andrew Morton <[email protected]> > Signed-off-by: Linus Torvalds <[email protected]>
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
