On Sun, 3 Jul 2022 02:16:09 +0800 kernel test robot <lkp@xxxxxxxxx> wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-unstable > head: 706370e5c2ea7bb4544eee6e1172c4d68117a526 > commit: 5b0de55b24615b5f5f600e2a61c297a0432a5e82 [321/323] sh/mm: enable ARCH_HAS_VM_GET_PAGE_PROT > config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20220703/202207030224.tqi3q1Wh-lkp@xxxxxxxxx/config) > compiler: sh4-linux-gcc (GCC) 11.3.0 > reproduce (this is a W=1 build): > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git/commit/?id=5b0de55b24615b5f5f600e2a61c297a0432a5e82 > git remote add akpm-mm https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git > git fetch --no-tags akpm-mm mm-unstable > git checkout 5b0de55b24615b5f5f600e2a61c297a0432a5e82 > # save the config file > mkdir build_dir && cp config build_dir/.config > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=sh SHELL=/bin/bash > > If you fix the issue, kindly add following tag where applicable > Reported-by: kernel test robot <lkp@xxxxxxxxx> > > All errors (new ones prefixed by >>): > > In file included from arch/sh/include/asm/io.h:21, > from include/linux/io.h:13, > from arch/sh/mm/mmap.c:10: > >> include/linux/pgtable.h:1713:10: error: redefinition of 'vm_get_page_prot' > 1713 | pgprot_t vm_get_page_prot(unsigned long vm_flags) \ > | ^~~~~~~~~~~~~~~~ > arch/sh/mm/mmap.c:184:1: note: in expansion of macro 'DECLARE_VM_GET_PAGE_PROT' > 184 | DECLARE_VM_GET_PAGE_PROT > | ^~~~~~~~~~~~~~~~~~~~~~~~ > In file included from arch/sh/mm/mmap.c:11: > include/linux/mm.h:2864:24: note: previous definition of 'vm_get_page_prot' with type 'pgprot_t(long unsigned int)' > 2864 | static inline pgprot_t vm_get_page_prot(unsigned long vm_flags) > | ^~~~~~~~~~~~~~~~ > um, OK, CONFIG_MMU=n. I'll let Anshuman ponder this one. btw, that macro should be called DEFINE_VM_GET_PAGE_PROT - it provides a definition, not a declaration.