Re: [RFC PATCH] KVM: x86: add kvm per-vCPU exits disable capability

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

[-- Attachment #1: Type: text/plain, Size: 3585 bytes --]

Hi Kechen,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on kvm/queue]
[also build test WARNING on mst-vhost/linux-next tip/master linux/master linus/master v5.16-rc5]
[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]

url:    https://github.com/0day-ci/linux/commits/Kechen-Lu/KVM-x86-add-kvm-per-vCPU-exits-disable-capability/20211214-113446
base:   https://git.kernel.org/pub/scm/virt/kvm/kvm.git queue
config: i386-randconfig-a014-20211213 (https://download.01.org/0day-ci/archive/20211214/202112141530.ZQIO4PKE-lkp(a)intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
        # https://github.com/0day-ci/linux/commit/08f20df47a09e327aee3e73a8d7c77b1ffc01bc0
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Kechen-Lu/KVM-x86-add-kvm-per-vCPU-exits-disable-capability/20211214-113446
        git checkout 08f20df47a09e327aee3e73a8d7c77b1ffc01bc0
        # save the config file to linux build tree
        mkdir build_dir
        make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash arch/x86/kvm/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   In file included from include/linux/kvm_host.h:38,
                    from arch/x86/kvm/x86.c:19:
   arch/x86/kvm/x86.c: In function 'kvm_vm_ioctl_enable_cap':
>> include/uapi/linux/kvm.h:801:51: warning: left shift count >= width of type [-Wshift-count-overflow]
     801 | #define KVM_X86_DISABLE_EXITS_PER_VCPU       (1UL << 63)
         |                                                   ^~
   include/uapi/linux/kvm.h:806:12: note: in expansion of macro 'KVM_X86_DISABLE_EXITS_PER_VCPU'
     806 |            KVM_X86_DISABLE_EXITS_PER_VCPU)
         |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   arch/x86/kvm/x86.c:5775:23: note: in expansion of macro 'KVM_X86_DISABLE_VALID_EXITS'
    5775 |   if (cap->args[0] & ~KVM_X86_DISABLE_VALID_EXITS)
         |                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>> include/uapi/linux/kvm.h:801:51: warning: left shift count >= width of type [-Wshift-count-overflow]
     801 | #define KVM_X86_DISABLE_EXITS_PER_VCPU       (1UL << 63)
         |                                                   ^~
   arch/x86/kvm/x86.c:5787:23: note: in expansion of macro 'KVM_X86_DISABLE_EXITS_PER_VCPU'
    5787 |   if ((cap->args[0] & KVM_X86_DISABLE_EXITS_PER_VCPU) &&
         |                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~


vim +801 include/uapi/linux/kvm.h

   796	
   797	#define KVM_X86_DISABLE_EXITS_MWAIT          (1 << 0)
   798	#define KVM_X86_DISABLE_EXITS_HLT            (1 << 1)
   799	#define KVM_X86_DISABLE_EXITS_PAUSE          (1 << 2)
   800	#define KVM_X86_DISABLE_EXITS_CSTATE         (1 << 3)
 > 801	#define KVM_X86_DISABLE_EXITS_PER_VCPU       (1UL << 63)
   802	#define KVM_X86_DISABLE_VALID_EXITS          (KVM_X86_DISABLE_EXITS_MWAIT | \
   803	                                              KVM_X86_DISABLE_EXITS_HLT | \
   804	                                              KVM_X86_DISABLE_EXITS_PAUSE | \
   805						      KVM_X86_DISABLE_EXITS_CSTATE| \
   806						      KVM_X86_DISABLE_EXITS_PER_VCPU)
   807	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org