Re: [PATCH RFC 01/15] KVM: Move running VCPU from ARM to common code

From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH RFC 01/15] KVM: Move running VCPU from ARM to common code
Date: Sun, 01 Dec 2019 00:18:57 +0800	[thread overview]
Message-ID: <201911302352.tfAqXqlb%lkp@intel.com> (raw)
In-Reply-To: <20191129213505.18472-2-peterx@redhat.com>

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

Hi Peter,

[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on kvm/linux-next]
[also build test ERROR on next-20191129]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Peter-Xu/KVM-Dirty-ring-interface/20191130-054051
base:   https://git.kernel.org/pub/scm/virt/kvm/kvm.git linux-next
config: arm64-defconfig (attached as .config)
compiler: aarch64-linux-gcc (GCC) 7.4.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=7.4.0 make.cross ARCH=arm64 

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

All error/warnings (new ones prefixed by >>):

   arch/arm64/kvm/../../../virt/kvm/arm/arch_timer.c: In function 'kvm_arch_timer_get_input_level':
>> arch/arm64/kvm/../../../virt/kvm/arm/arch_timer.c:1025:26: error: implicit declaration of function 'kvm_arm_get_running_vcpu'; did you mean 'kvm_get_running_vcpu'? [-Werror=implicit-function-declaration]
     struct kvm_vcpu *vcpu = kvm_arm_get_running_vcpu();
                             ^~~~~~~~~~~~~~~~~~~~~~~~
                             kvm_get_running_vcpu
>> arch/arm64/kvm/../../../virt/kvm/arm/arch_timer.c:1025:26: warning: initialization makes pointer from integer without a cast [-Wint-conversion]
   cc1: some warnings being treated as errors

vim +1025 arch/arm64/kvm/../../../virt/kvm/arm/arch_timer.c

99a1db7a2c9b2e Christoffer Dall 2017-05-02  1022  
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1023  bool kvm_arch_timer_get_input_level(int vintid)
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1024  {
4c60e360d6dfa4 Christoffer Dall 2017-10-27 @1025  	struct kvm_vcpu *vcpu = kvm_arm_get_running_vcpu();
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1026  	struct arch_timer_context *timer;
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1027  
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1028  	if (vintid == vcpu_vtimer(vcpu)->irq.irq)
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1029  		timer = vcpu_vtimer(vcpu);
9e01dc76be6a3b Christoffer Dall 2019-02-19  1030  	else if (vintid == vcpu_ptimer(vcpu)->irq.irq)
9e01dc76be6a3b Christoffer Dall 2019-02-19  1031  		timer = vcpu_ptimer(vcpu);
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1032  	else
9e01dc76be6a3b Christoffer Dall 2019-02-19  1033  		BUG();
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1034  
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1035  	return kvm_timer_should_fire(timer);
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1036  }
4c60e360d6dfa4 Christoffer Dall 2017-10-27  1037  

:::::: The code at line 1025 was first introduced by commit
:::::: 4c60e360d6dfa4d9c3586b687f348eeb3fd675dd KVM: arm/arm64: Provide a get_input_level for the arch timer

:::::: TO: Christoffer Dall <christoffer.dall@linaro.org>
:::::: CC: Christoffer Dall <christoffer.dall@linaro.org>

---
0-DAY kernel test infrastructure                 Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org Intel Corporation

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