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>
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':
> arch/arm64/include/asm/sysreg.h:1276:44: warning: left shift
count >= width of type [-Wshift-count-overflow]
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);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
> arch/arm64/include/asm/sysreg.h:1276:44: warning: left shift
count >= width of type [-Wshift-count-overflow]
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':
> arch/arm64/include/asm/sysreg.h:1276:44: warning: left shift
count >= width of type [-Wshift-count-overflow]
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;
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
> arch/arm64/include/asm/sysreg.h:1276:44: warning: left shift
count >= width of type [-Wshift-count-overflow]
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