From mboxrd@z Thu Jan 1 00:00:00 1970 From: Radim =?utf-8?B?S3LEjW3DocWZ?= Subject: Re: [PATCH] KVM: x86: don't hold kvm->lock in KVM_SET_GSI_ROUTING Date: Wed, 26 Apr 2017 16:24:26 +0200 Message-ID: <20170426142425.GA26029@potion> References: <20170425190306.20392-1-david@redhat.com> <201704262026.gFdqSV4P%fengguang.wu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: David Hildenbrand , kbuild-all@01.org, srutherford@google.com, kvm@vger.kernel.org, pbonzini@redhat.com To: kbuild test robot Return-path: Received: from mx1.redhat.com ([209.132.183.28]:44374 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1437682AbdDZOYh (ORCPT ); Wed, 26 Apr 2017 10:24:37 -0400 Content-Disposition: inline In-Reply-To: <201704262026.gFdqSV4P%fengguang.wu@intel.com> Sender: kvm-owner@vger.kernel.org List-ID: 2017-04-26 20:33+0800, kbuild test robot: > Hi David, > > [auto build test ERROR on kvm/linux-next] > [also build test ERROR on next-20170424] > [cannot apply to v4.11-rc8] > [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] > > url: https://github.com/0day-ci/linux/commits/David-Hildenbrand/KVM-x86-don-t-hold-kvm-lock-in-KVM_SET_GSI_ROUTING/20170426-171503 > base: https://git.kernel.org/pub/scm/virt/kvm/kvm.git linux-next > config: powerpc-currituck_defconfig (attached as .config) > compiler: powerpc-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 > reproduce: > wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # save the attached .config to linux build tree > make.cross ARCH=powerpc > > All errors (new ones prefixed by >>): > > In file included from arch/powerpc/include/asm/kvm_ppc.h:30:0, > from arch/powerpc/kernel/smp.c:41: > >> include/linux/kvm_host.h:515:13: error: 'kvm_arch_can_set_irq_routing' defined but not used [-Werror=unused-function] > static bool kvm_arch_can_set_irq_routing(struct kvm *kvm) > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ > cc1: all warnings being treated as errors > > vim +/kvm_arch_can_set_irq_routing +515 include/linux/kvm_host.h > > 509 static inline void kvm_arch_post_irq_ack_notifier_list_update(struct kvm *kvm) > 510 { > 511 } > 512 static inline void kvm_arch_post_irq_routing_update(struct kvm *kvm) > 513 { > 514 } > > 515 static bool kvm_arch_can_set_irq_routing(struct kvm *kvm) Oh, we need both static and inline to supress the warning ... > 516 { > 517 return true; > 518 } > > --- > 0-DAY kernel test infrastructure Open Source Technology Center > https://lists.01.org/pipermail/kbuild-all Intel Corporation