From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 1/2] kvm, Add x86_hyper_kvm to complete detect_hypervisor_platform check [v2] Date: Wed, 11 Jul 2012 19:34:24 +0300 Message-ID: <4FFDAB10.8020404@redhat.com> References: <1341493877-16622-1-git-send-email-prarit@redhat.com> <1341493877-16622-2-git-send-email-prarit@redhat.com> <4FF59606.4060504@redhat.com> <4FF5987C.6080905@redhat.com> <20120706112719.GA18059@amt.cnet> <4FF724BB.5040008@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Marcelo Tosatti , linux-kernel@vger.kernel.org, Gleb Natapov , Alex Williamson , Konrad Rzeszutek Wilk , kvm@vger.kernel.org To: Prarit Bhargava Return-path: In-Reply-To: <4FF724BB.5040008@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 07/06/2012 08:47 PM, Prarit Bhargava wrote: > [PATCH 1/2] kvm, Add x86_hyper_kvm to complete detect_hypervisor_platform check [v3] > > While debugging I noticed that unlike all the other hypervisor code in the > kernel, kvm does not have an entry for x86_hyper which is used in > detect_hypervisor_platform() which results in a nice printk in the > syslog. This is only really a stub function but it > does make kvm more consistent with the other hypervisors. Thanks, applied. > > --- a/arch/x86/kernel/cpu/hypervisor.c > +++ b/arch/x86/kernel/cpu/hypervisor.c > @@ -37,6 +37,7 @@ static const __initconst struct hypervisor_x86 * const > hypervisors[] = Corrupted patch, fixed by hand. > #endif -- error compiling committee.c: too many arguments to function