On Fri, 2024-11-01 at 12:14 +0800, kernel test robot wrote: > Hi Amit, > > kernel test robot noticed the following build warnings: > > [auto build test WARNING on kvm/queue] > [also build test WARNING on mst-vhost/linux-next tip/master > tip/x86/core linus/master v6.12-rc5 next-20241031] > [cannot apply to kvm/linux-next tip/auto-latest] > [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/Amit-Shah/x86-cpu-bugs-add-support-for-AMD-ERAPS-feature/20241031-234332 > base: https://git.kernel.org/pub/scm/virt/kvm/kvm.git queue > patch link: > https://lore.kernel.org/r/20241031153925.36216-3-amit%40kernel.org > patch subject: [PATCH 2/2] x86: kvm: svm: add support for ERAPS and > FLUSH_RAP_ON_VMRUN > config: x86_64-kexec > (https://download.01.org/0day-ci/archive/20241101/202411011119.l3yRJp > ht-lkp@xxxxxxxxx/config) > compiler: clang version 19.1.3 > (https://github.com/llvm/llvm-project ab51eccf88f5321e7c60591c5546b25 > 4b6afab99) > reproduce (this is a W=1 build): > (https://download.01.org/0day-ci/archive/20241101/202411011119.l3yRJp > ht-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/202411011119.l3yRJpht-lkp@xxxxxxxxx/ > > All warnings (new ones prefixed by >>): > > In file included from arch/x86/kvm/cpuid.c:13: > In file included from include/linux/kvm_host.h:16: > In file included from include/linux/mm.h:2213: > include/linux/vmstat.h:504:43: warning: arithmetic between > different enumeration types ('enum zone_stat_item' and 'enum > numa_stat_item') [-Wenum-enum-conversion] > 504 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS + > | ~~~~~~~~~~~~~~~~~~~~~ ^ > 505 | item]; > | ~~~~ > include/linux/vmstat.h:511:43: warning: arithmetic between > different enumeration types ('enum zone_stat_item' and 'enum > numa_stat_item') [-Wenum-enum-conversion] > 511 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS + > | ~~~~~~~~~~~~~~~~~~~~~ ^ > 512 | NR_VM_NUMA_EVENT_ITEMS + > | ~~~~~~~~~~~~~~~~~~~~~~ > include/linux/vmstat.h:518:36: warning: arithmetic between > different enumeration types ('enum node_stat_item' and 'enum > lru_list') [-Wenum-enum-conversion] > 518 | return node_stat_name(NR_LRU_BASE + lru) + 3; // > skip "nr_" > | ~~~~~~~~~~~ ^ ~~~ > include/linux/vmstat.h:524:43: warning: arithmetic between > different enumeration types ('enum zone_stat_item' and 'enum > numa_stat_item') [-Wenum-enum-conversion] > 524 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS + > | ~~~~~~~~~~~~~~~~~~~~~ ^ > 525 | NR_VM_NUMA_EVENT_ITEMS + > | ~~~~~~~~~~~~~~~~~~~~~~ > > > arch/x86/kvm/cpuid.c:1362:3: warning: label followed by a > > > declaration is a C23 extension [-Wc23-extensions] > 1362 | unsigned int ebx_mask = 0; > | ^ > 5 warnings generated. [...] > 1361 case 0x80000021: > > 1362 unsigned int ebx_mask = 0; > 1363 > 1364 entry->ecx = entry->edx = 0; > 1365 cpuid_entry_override(entry, > CPUID_8000_0021_EAX); > 1366 > 1367 /* > 1368 * Bits 23:16 in EBX indicate the size of > the RSB. > 1369 * Expose the value in the hardware to the > guest. > 1370 */ > 1371 if (kvm_cpu_cap_has(X86_FEATURE_ERAPS)) > 1372 ebx_mask |= GENMASK(23, 16); > 1373 > 1374 entry->ebx &= ebx_mask; > 1375 break; Right - I'll add braces around this case statement. Amit
