Hi Neil, On Wed, Feb 9, 2022 at 10:52 AM NeilBrown <neilb@xxxxxxx> wrote: > Many functions declared in include/linux/swap.h are only used within mm/ > > Create a new "mm/swap.h" and move some of these declarations there. > Remove the redundant 'extern' from the function declarations. > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> > Signed-off-by: NeilBrown <neilb@xxxxxxx> Thanks for your patch! > --- /dev/null > +++ b/mm/swap.h > @@ -0,0 +1,129 @@ > + scripts/checkpatch.pl: WARNING: Missing or malformed SPDX-License-Identifier tag in line 1 Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds