On Fri, 3 Nov 2017 04:12:29 +0300 Yury Norov <ynorov@xxxxxxxxxxxxxxxxxx> wrote: > > #if !defined(find_next_bit_le) || !defined(find_next_zero_bit_le) > > -static unsigned long _find_next_bit_le(const unsigned long *addr, > > - unsigned long nbits, unsigned long start, unsigned long invert) > > +static unsigned long _find_next_bit_le(const unsigned long *addr1, > > + const unsigned long *addr2, unsigned long nbits, > > + unsigned long start, unsigned long invert) > > This should also be inline I think. gcc-4.4.4 (at least) does that automatically. > The rest is looking OK. > Andrew, did you see my patch with the test for _find_bit()? > https://lkml.org/lkml/2017/10/28/70 It was unclear from Clement's testing that the change is worthwhile? -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html