On Thu, 2024-04-11 at 16:36 +1200, Barry Song wrote: > On Thu, Apr 11, 2024 at 4:27 PM kernel test robot <lkp@xxxxxxxxx> wrote: > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-unstable > > head: de14d5f2a9ff2df737a01575ee95bc5a35f5853d > > commit: ca23b4d4cae5d32a9d22fb3c39550dcf43df83f9 [360/372] mm: swap: introduce swap_free_nr() for batched swap_free() > > config: x86_64-allnoconfig (https://download.01.org/0day-ci/archive/20240411/202404111223.DDjEQDHj-lkp@xxxxxxxxx/config) > > compiler: clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18) > > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240411/202404111223.DDjEQDHj-lkp@xxxxxxxxx/reproduce) > > > > If you fix the issue in a separate patch/commit (i.e. not just a new version of > > the same patch/commit), kindly add following tags > > > Reported-by: kernel test robot <lkp@xxxxxxxxx> > > > Closes: https://lore.kernel.org/oe-kbuild-all/202404111223.DDjEQDHj-lkp@xxxxxxxxx/ > > > > All warnings (new ones prefixed by >>): > > > > In file included from arch/x86/kernel/asm-offsets.c:14: > > In file included from include/linux/suspend.h:5: > > > > include/linux/swap.h:565:6: warning: no previous prototype for function 'swap_free_nr' [-Wmissing-prototypes] > > 565 | void swap_free_nr(swp_entry_t entry, int nr_pages) > > | ^ > > include/linux/swap.h:565:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > > 565 | void swap_free_nr(swp_entry_t entry, int nr_pages) > > | ^ > > | static > > 1 warning generated. > > -- > > In file included from arch/x86/kernel/reboot.c:33: > > In file included from arch/x86/include/asm/efi.h:7: > > In file included from arch/x86/include/asm/tlb.h:8: > > In file included from include/asm-generic/tlb.h:15: > > > > include/linux/swap.h:565:6: warning: no previous prototype for function 'swap_free_nr' [-Wmissing-prototypes] > > 565 | void swap_free_nr(swp_entry_t entry, int nr_pages) > > | ^ > > include/linux/swap.h:565:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > > 565 | void swap_free_nr(swp_entry_t entry, int nr_pages) > > | ^ > > | static > > arch/x86/kernel/reboot.c:972:20: warning: unused function 'nmi_shootdown_cpus_on_restart' [-Wunused-function] > > 972 | static inline void nmi_shootdown_cpus_on_restart(void) { } > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > 2 warnings generated. > > -- > > In file included from arch/x86/kernel/asm-offsets.c:14: > > In file included from include/linux/suspend.h:5: > > > > include/linux/swap.h:565:6: warning: no previous prototype for function 'swap_free_nr' [-Wmissing-prototypes] > > 565 | void swap_free_nr(swp_entry_t entry, int nr_pages) > > | ^ > > include/linux/swap.h:565:1: note: declare 'static' if the function is not intended to be used outside of this translation unit > > 565 | void swap_free_nr(swp_entry_t entry, int nr_pages) > > | ^ > > | static > > 1 warning generated. > > > > > > vim +/swap_free_nr +565 include/linux/swap.h > > > > 564 > > > 565 void swap_free_nr(swp_entry_t entry, int nr_pages) > > 566 { > > 567 } > > 568 > > Hi, > thank you! > > SeongJae reported this and fixed it with a patch here: > https://lore.kernel.org/linux-mm/20240410233759.274907-1-sj@xxxxxxxxxx/ > > Can you please test it? We tested SJ's patch and it fixes the build issues in this report. Tested-by: Yujie Liu <yujie.liu@xxxxxxxxx> Thanks > > > > -- > > 0-DAY CI Kernel Test Service > > https://github.com/intel/lkp-tests/wiki > > >
