[kbuild-all] Re: [PATCH v8 25/38] KVM: arm64: Trap SME usage in guest

Hi Mark, I love your patch! Perhaps something to improve: [auto build test WARNING on e783362eb54cd99b2cac8b3a9aeac942e6f6ac07] url: https://github.com/0day-ci/linux/commits/Mark-Brown/arm64-sme-Initial-sup... base: e783362eb54cd99b2cac8b3a9aeac942e6f6ac07 config: arm64-randconfig-r033-20220124 (https://download.01.org/0day-ci/archive/20220125/202201252032.d2MrIFVb-lk...) compiler: aarch64-linux-gcc (GCC) 11.2.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://github.com/0day-ci/linux/commit/35adc3582db36b4cefefdbdf3afa245a6... git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Mark-Brown/arm64-sme-Initial-support-for-the-Scalable-Matrix-Extension/20220125-120723 git checkout 35adc3582db36b4cefefdbdf3afa245a69e529ae # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arm64 SHELL=/bin/bash arch/arm64/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; All warnings (new ones prefixed by >>): In file included from arch/arm64/include/asm/cputype.h:148, from arch/arm64/include/asm/cache.h:8, from include/linux/cache.h:6, from include/linux/printk.h:9, from include/asm-generic/bug.h:22, from arch/arm64/include/asm/bug.h:26, from include/linux/bug.h:5, from include/linux/mmdebug.h:5, from include/linux/percpu.h:5, from include/linux/context_tracking_state.h:5, from include/linux/hardirq.h:5, from include/linux/kvm_host.h:7, from arch/arm64/include/asm/kvm_emulate.h:14, from arch/arm64/kvm/hyp/include/hyp/adjust_pc.h:13, from arch/arm64/kvm/hyp/include/hyp/switch.h:10, from arch/arm64/kvm/hyp/nvhe/switch.c:7: arch/arm64/kvm/hyp/nvhe/switch.c: In function '__activate_traps': 1276 | #define HFGxTR_EL2_nTPIDR_EL0_MASK (1 << HFGxTR_EL2_nTPIDR_EL0_SHIFT) | ^~ arch/arm64/kvm/hyp/nvhe/switch.c:59:26: note: in expansion of macro 'HFGxTR_EL2_nTPIDR_EL0_MASK' 59 | val &= ~(HFGxTR_EL2_nTPIDR_EL0_MASK | | ^~~~~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/include/asm/sysreg.h:1278:44: warning: left shift count >= width of type [-Wshift-count-overflow] 1278 | #define HFGxTR_EL2_nSMPRI_EL1_MASK (1 << HFGxTR_EL2_nSMPRI_EL1_SHIFT) | ^~ arch/arm64/kvm/hyp/nvhe/switch.c:60:26: note: in expansion of macro 'HFGxTR_EL2_nSMPRI_EL1_MASK' 60 | HFGxTR_EL2_nSMPRI_EL1_MASK); | ^~~~~~~~~~~~~~~~~~~~~~~~~~ 1276 | #define HFGxTR_EL2_nTPIDR_EL0_MASK (1 << HFGxTR_EL2_nTPIDR_EL0_SHIFT) | ^~ arch/arm64/kvm/hyp/nvhe/switch.c:64:26: note: in expansion of macro 'HFGxTR_EL2_nTPIDR_EL0_MASK' 64 | val &= ~(HFGxTR_EL2_nTPIDR_EL0_MASK | | ^~~~~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/include/asm/sysreg.h:1278:44: warning: left shift count >= width of type [-Wshift-count-overflow] 1278 | #define HFGxTR_EL2_nSMPRI_EL1_MASK (1 << HFGxTR_EL2_nSMPRI_EL1_SHIFT) | ^~ arch/arm64/kvm/hyp/nvhe/switch.c:65:26: note: in expansion of macro 'HFGxTR_EL2_nSMPRI_EL1_MASK' 65 | HFGxTR_EL2_nSMPRI_EL1_MASK); | ^~~~~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c: In function '__deactivate_traps': 1276 | #define HFGxTR_EL2_nTPIDR_EL0_MASK (1 << HFGxTR_EL2_nTPIDR_EL0_SHIFT) | ^~ arch/arm64/kvm/hyp/nvhe/switch.c:117:24: note: in expansion of macro 'HFGxTR_EL2_nTPIDR_EL0_MASK' 117 | val |= HFGxTR_EL2_nTPIDR_EL0_MASK | HFGxTR_EL2_nSMPRI_EL1_MASK; | ^~~~~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/include/asm/sysreg.h:1278:44: warning: left shift count >= width of type [-Wshift-count-overflow] 1278 | #define HFGxTR_EL2_nSMPRI_EL1_MASK (1 << HFGxTR_EL2_nSMPRI_EL1_SHIFT) | ^~ arch/arm64/kvm/hyp/nvhe/switch.c:117:53: note: in expansion of macro 'HFGxTR_EL2_nSMPRI_EL1_MASK' 117 | val |= HFGxTR_EL2_nTPIDR_EL0_MASK | HFGxTR_EL2_nSMPRI_EL1_MASK; | ^~~~~~~~~~~~~~~~~~~~~~~~~~ 1276 | #define HFGxTR_EL2_nTPIDR_EL0_MASK (1 << HFGxTR_EL2_nTPIDR_EL0_SHIFT) | ^~ arch/arm64/kvm/hyp/nvhe/switch.c:121:24: note: in expansion of macro 'HFGxTR_EL2_nTPIDR_EL0_MASK' 121 | val |= HFGxTR_EL2_nTPIDR_EL0_MASK | HFGxTR_EL2_nSMPRI_EL1_MASK; | ^~~~~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/include/asm/sysreg.h:1278:44: warning: left shift count >= width of type [-Wshift-count-overflow] 1278 | #define HFGxTR_EL2_nSMPRI_EL1_MASK (1 << HFGxTR_EL2_nSMPRI_EL1_SHIFT) | ^~ arch/arm64/kvm/hyp/nvhe/switch.c:121:53: note: in expansion of macro 'HFGxTR_EL2_nSMPRI_EL1_MASK' 121 | val |= HFGxTR_EL2_nTPIDR_EL0_MASK | HFGxTR_EL2_nSMPRI_EL1_MASK; | ^~~~~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c: At top level: arch/arm64/kvm/hyp/nvhe/switch.c:227:43: warning: initialized field overwritten [-Woverride-init] 227 | [ESR_ELx_EC_CP15_32] = kvm_hyp_handle_cp15_32, | ^~~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:227:43: note: (near initialization for 'hyp_exit_handlers[3]') arch/arm64/kvm/hyp/nvhe/switch.c:228:43: warning: initialized field overwritten [-Woverride-init] 228 | [ESR_ELx_EC_SYS64] = kvm_hyp_handle_sysreg, | ^~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:228:43: note: (near initialization for 'hyp_exit_handlers[24]') arch/arm64/kvm/hyp/nvhe/switch.c:229:43: warning: initialized field overwritten [-Woverride-init] 229 | [ESR_ELx_EC_SVE] = kvm_hyp_handle_fpsimd, | ^~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:229:43: note: (near initialization for 'hyp_exit_handlers[25]') arch/arm64/kvm/hyp/nvhe/switch.c:230:43: warning: initialized field overwritten [-Woverride-init] 230 | [ESR_ELx_EC_FP_ASIMD] = kvm_hyp_handle_fpsimd, | ^~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:230:43: note: (near initialization for 'hyp_exit_handlers[7]') arch/arm64/kvm/hyp/nvhe/switch.c:231:43: warning: initialized field overwritten [-Woverride-init] 231 | [ESR_ELx_EC_IABT_LOW] = kvm_hyp_handle_iabt_low, | ^~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:231:43: note: (near initialization for 'hyp_exit_handlers[32]') arch/arm64/kvm/hyp/nvhe/switch.c:232:43: warning: initialized field overwritten [-Woverride-init] 232 | [ESR_ELx_EC_DABT_LOW] = kvm_hyp_handle_dabt_low, | ^~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:232:43: note: (near initialization for 'hyp_exit_handlers[36]') arch/arm64/kvm/hyp/nvhe/switch.c:233:43: warning: initialized field overwritten [-Woverride-init] 233 | [ESR_ELx_EC_PAC] = kvm_hyp_handle_ptrauth, | ^~~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:233:43: note: (near initialization for 'hyp_exit_handlers[9]') arch/arm64/kvm/hyp/nvhe/switch.c:238:43: warning: initialized field overwritten [-Woverride-init] 238 | [ESR_ELx_EC_SYS64] = kvm_handle_pvm_sys64, | ^~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:238:43: note: (near initialization for 'pvm_exit_handlers[24]') arch/arm64/kvm/hyp/nvhe/switch.c:239:43: warning: initialized field overwritten [-Woverride-init] 239 | [ESR_ELx_EC_SVE] = kvm_handle_pvm_restricted, | ^~~~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:239:43: note: (near initialization for 'pvm_exit_handlers[25]') arch/arm64/kvm/hyp/nvhe/switch.c:240:43: warning: initialized field overwritten [-Woverride-init] 240 | [ESR_ELx_EC_FP_ASIMD] = kvm_handle_pvm_fpsimd, | ^~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:240:43: note: (near initialization for 'pvm_exit_handlers[7]') arch/arm64/kvm/hyp/nvhe/switch.c:241:43: warning: initialized field overwritten [-Woverride-init] 241 | [ESR_ELx_EC_IABT_LOW] = kvm_hyp_handle_iabt_low, | ^~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:241:43: note: (near initialization for 'pvm_exit_handlers[32]') arch/arm64/kvm/hyp/nvhe/switch.c:242:43: warning: initialized field overwritten [-Woverride-init] 242 | [ESR_ELx_EC_DABT_LOW] = kvm_hyp_handle_dabt_low, | ^~~~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:242:43: note: (near initialization for 'pvm_exit_handlers[36]') arch/arm64/kvm/hyp/nvhe/switch.c:243:43: warning: initialized field overwritten [-Woverride-init] 243 | [ESR_ELx_EC_PAC] = kvm_hyp_handle_ptrauth, | ^~~~~~~~~~~~~~~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:243:43: note: (near initialization for 'pvm_exit_handlers[9]') arch/arm64/kvm/hyp/nvhe/switch.c:380:17: warning: no previous prototype for 'hyp_panic' [-Wmissing-prototypes] 380 | void __noreturn hyp_panic(void) | ^~~~~~~~~ arch/arm64/kvm/hyp/nvhe/switch.c:402:17: warning: no previous prototype for 'kvm_unexpected_el2_exception' [-Wmissing-prototypes] 402 | asmlinkage void kvm_unexpected_el2_exception(void) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ vim +1276 arch/arm64/include/asm/sysreg.h 8a657f71705f9f Hector Martin 2021-03-01 1273 3574e3c8f58afa Mark Brown 2022-01-25 1274 /* HFG[WR]TR_EL2 bit definitions */ 3574e3c8f58afa Mark Brown 2022-01-25 1275 #define HFGxTR_EL2_nTPIDR_EL0_SHIFT 55 3574e3c8f58afa Mark Brown 2022-01-25 @1276 #define HFGxTR_EL2_nTPIDR_EL0_MASK (1 << HFGxTR_EL2_nTPIDR_EL0_SHIFT) 3574e3c8f58afa Mark Brown 2022-01-25 1277 #define HFGxTR_EL2_nSMPRI_EL1_SHIFT 54 3574e3c8f58afa Mark Brown 2022-01-25 1278 #define HFGxTR_EL2_nSMPRI_EL1_MASK (1 << HFGxTR_EL2_nSMPRI_EL1_SHIFT) 3574e3c8f58afa Mark Brown 2022-01-25 1279 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org