On Fri, Dec 17, 2021 at 10:48:22AM +1100, NeilBrown 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. Good idea! Looks good modulo the extra includes that the buildbot asks for: Reviewed-by: Christoph Hellwig <hch@xxxxxx>