Re: [PATCH 2/5 v4] KVM: SVM: Fill in conforming svm_x86_ops via macro

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

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

Hi Krish,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on linus/master]
[also build test WARNING on v5.10-rc3 next-20201109]
[cannot apply to vhost/linux-next kvmarm/next kvm/linux-next kvms390/next]
[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/Krish-Sadhukhan/KVM-x86-Fill-in-conforming-vmx-svm-_x86_ops-and-vmx-svm-_nested_ops-via-macros/20201110-095653
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 407ab579637ced6dc32cfb2295afb7259cca4b22
config: i386-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
        # https://github.com/0day-ci/linux/commit/26275e3a7e11da29dc47a214b3938e7261c61ec7
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Krish-Sadhukhan/KVM-x86-Fill-in-conforming-vmx-svm-_x86_ops-and-vmx-svm-_nested_ops-via-macros/20201110-095653
        git checkout 26275e3a7e11da29dc47a214b3938e7261c61ec7
        # save the attached .config to linux build tree
        make W=1 ARCH=i386 

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 >>):

>> arch/x86/kvm/svm/svm.c:1651:6: warning: no previous prototype for 'svm_get_cs_db_l_bits' [-Wmissing-prototypes]
    1651 | void svm_get_cs_db_l_bits(struct kvm_vcpu *vcpu, int *db, int *l)
         |      ^~~~~~~~~~~~~~~~~~~~
>> arch/x86/kvm/svm/svm.c:3372:6: warning: no previous prototype for 'svm_tlb_flush_all' [-Wmissing-prototypes]
    3372 | void svm_tlb_flush_all(struct kvm_vcpu *vcpu)
         |      ^~~~~~~~~~~~~~~~~
>> arch/x86/kvm/svm/svm.c:3377:6: warning: no previous prototype for 'svm_tlb_flush_current' [-Wmissing-prototypes]
    3377 | void svm_tlb_flush_current(struct kvm_vcpu *vcpu)
         |      ^~~~~~~~~~~~~~~~~~~~~
>> arch/x86/kvm/svm/svm.c:3382:6: warning: no previous prototype for 'svm_tlb_flush_guest' [-Wmissing-prototypes]
    3382 | void svm_tlb_flush_guest(struct kvm_vcpu *vcpu)
         |      ^~~~~~~~~~~~~~~~~~~

vim +/svm_get_cs_db_l_bits +1651 arch/x86/kvm/svm/svm.c

  1650	
> 1651	void svm_get_cs_db_l_bits(struct kvm_vcpu *vcpu, int *db, int *l)
  1652	{
  1653	        struct kvm_segment cs;
  1654	
  1655		kvm_get_segment(vcpu, &cs, VCPU_SREG_CS);
  1656	        *db = cs.db;
  1657	        *l = cs.l;
  1658	}
  1659	

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

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 75868 bytes --]