Re: [RFC PATCH 10/16] mm: Map page tables with privileged pkey

[kernel test robot <lkp@intel.com>]

Hi Kevin,

[This is a private test report for your RFC patch.]
kernel test robot noticed the following build errors:

[auto build test ERROR on linus/master]
[also build test ERROR on v6.13-rc1]
[cannot apply to arm64/for-next/core akpm-mm/mm-everything kees/for-next/hardening next-20241205]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Kevin-Brodsky/mm-Introduce-kpkeys/20241206-181928
base:   linus/master
patch link:    https://lore.kernel.org/r/20241206101110.1646108-11-kevin.brodsky%40arm.com
patch subject: [RFC PATCH 10/16] mm: Map page tables with privileged pkey
config: i386-buildonly-randconfig-002 (https://download.01.org/0day-ci/archive/20241206/202412062148.fQlAVl6y-lkp@intel.com/config)
compiler: clang version 19.1.3 (https://github.com/llvm/llvm-project ab51eccf88f5321e7c60591c5546b254b6afab99)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241206/202412062148.fQlAVl6y-lkp@intel.com/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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202412062148.fQlAVl6y-lkp@intel.com/

All errors (new ones prefixed by >>):

   In file included from arch/x86/kernel/idt.c:8:
   In file included from arch/x86/include/asm/set_memory.h:5:
   In file included from include/linux/mm.h:34:
   In file included from include/linux/kpkeys.h:8:
>> include/linux/set_memory.h:20:12: error: call to undeclared function 'set_memory_ro'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
      20 |         int ret = set_memory_ro(addr, numpages);
         |                   ^
   include/linux/set_memory.h:20:12: note: did you mean 'set_memory_rox'?
   include/linux/set_memory.h:18:19: note: 'set_memory_rox' declared here
      18 | static inline int set_memory_rox(unsigned long addr, int numpages)
         |                   ^
      19 | {
      20 |         int ret = set_memory_ro(addr, numpages);
         |                   ~~~~~~~~~~~~~
         |                   set_memory_rox
>> include/linux/set_memory.h:23:9: error: call to undeclared function 'set_memory_x'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
      23 |         return set_memory_x(addr, numpages);
         |                ^
   In file included from arch/x86/kernel/idt.c:8:
   In file included from arch/x86/include/asm/set_memory.h:5:
   In file included from include/linux/mm.h:1133:
   In file included from include/linux/huge_mm.h:7:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:98:11: warning: array index 3 is past the end of the array (that has type 'unsigned long[2]') [-Warray-bounds]
      98 |                 return (set->sig[3] | set->sig[2] |
         |                         ^        ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
      24 |         unsigned long sig[_NSIG_WORDS];
         |         ^
   In file included from arch/x86/kernel/idt.c:8:
   In file included from arch/x86/include/asm/set_memory.h:5:
   In file included from include/linux/mm.h:1133:
   In file included from include/linux/huge_mm.h:7:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:98:25: warning: array index 2 is past the end of the array (that has type 'unsigned long[2]') [-Warray-bounds]
      98 |                 return (set->sig[3] | set->sig[2] |
         |                                       ^        ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
      24 |         unsigned long sig[_NSIG_WORDS];
         |         ^
   In file included from arch/x86/kernel/idt.c:8:
   In file included from arch/x86/include/asm/set_memory.h:5:
   In file included from include/linux/mm.h:1133:
   In file included from include/linux/huge_mm.h:7:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:114:11: warning: array index 3 is past the end of the array (that has type 'const unsigned long[2]') [-Warray-bounds]
     114 |                 return  (set1->sig[3] == set2->sig[3]) &&
         |                          ^         ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
      24 |         unsigned long sig[_NSIG_WORDS];
         |         ^
   In file included from arch/x86/kernel/idt.c:8:
   In file included from arch/x86/include/asm/set_memory.h:5:
   In file included from include/linux/mm.h:1133:
   In file included from include/linux/huge_mm.h:7:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:114:27: warning: array index 3 is past the end of the array (that has type 'const unsigned long[2]') [-Warray-bounds]
     114 |                 return  (set1->sig[3] == set2->sig[3]) &&
         |                                          ^         ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
      24 |         unsigned long sig[_NSIG_WORDS];
         |         ^
   In file included from arch/x86/kernel/idt.c:8:
   In file included from arch/x86/include/asm/set_memory.h:5:
   In file included from include/linux/mm.h:1133:
   In file included from include/linux/huge_mm.h:7:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:115:5: warning: array index 2 is past the end of the array (that has type 'const unsigned long[2]') [-Warray-bounds]
     115 |                         (set1->sig[2] == set2->sig[2]) &&
         |                          ^         ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
      24 |         unsigned long sig[_NSIG_WORDS];
         |         ^
   In file included from arch/x86/kernel/idt.c:8:
   In file included from arch/x86/include/asm/set_memory.h:5:
   In file included from include/linux/mm.h:1133:
   In file included from include/linux/huge_mm.h:7:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:115:21: warning: array index 2 is past the end of the array (that has type 'const unsigned long[2]') [-Warray-bounds]
     115 |                         (set1->sig[2] == set2->sig[2]) &&
         |                                          ^         ~
   arch/x86/include/asm/signal.h:24:2: note: array 'sig' declared here
      24 |         unsigned long sig[_NSIG_WORDS];
         |         ^
   In file included from arch/x86/kernel/idt.c:8:
   In file included from arch/x86/include/asm/set_memory.h:5:
   In file included from include/linux/mm.h:1133:
   In file included from include/linux/huge_mm.h:7:
   In file included from include/linux/fs.h:33:
   In file included from include/linux/percpu-rwsem.h:7:
   In file included from include/linux/rcuwait.h:6:
   In file included from include/linux/sched/signal.h:6:
   include/linux/signal.h:157:1: warning: array index 3 is past the end of the array (that has type 'const unsigned long[2]') [-Warray-bounds]
     157 | _SIG_SET_BINOP(sigorsets, _sig_or)
         | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/signal.h:138:8: note: expanded from macro '_SIG_SET_BINOP'
     138 |                 a3 = a->sig[3]; a2 = a->sig[2];                         \
         |                      ^      ~


vim +/set_memory_ro +20 include/linux/set_memory.h

938f846492d6682 Michael Ellerman 2017-07-06  16  
60463628c9e0a80 Peter Zijlstra   2022-10-29  17  #ifndef set_memory_rox
d48567c9a0d1e60 Peter Zijlstra   2022-10-26  18  static inline int set_memory_rox(unsigned long addr, int numpages)
d48567c9a0d1e60 Peter Zijlstra   2022-10-26  19  {
d48567c9a0d1e60 Peter Zijlstra   2022-10-26 @20  	int ret = set_memory_ro(addr, numpages);
d48567c9a0d1e60 Peter Zijlstra   2022-10-26  21  	if (ret)
d48567c9a0d1e60 Peter Zijlstra   2022-10-26  22  		return ret;
d48567c9a0d1e60 Peter Zijlstra   2022-10-26 @23  	return set_memory_x(addr, numpages);
d48567c9a0d1e60 Peter Zijlstra   2022-10-26  24  }
60463628c9e0a80 Peter Zijlstra   2022-10-29  25  #endif
d48567c9a0d1e60 Peter Zijlstra   2022-10-26  26  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki