From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [PATCH 1/1] KVM: SVM: add rdmsr support for AMD event registers Date: Mon, 18 Aug 2014 10:36:28 +0200 Message-ID: <53F1BB0C.9000504@redhat.com> References: <1407945974-27300-1-git-send-email-wehuang@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: gleb@kernel.org To: Wei Huang , kvm@vger.kernel.org Return-path: Received: from mx1.redhat.com ([209.132.183.28]:45180 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752120AbaHRIgd (ORCPT ); Mon, 18 Aug 2014 04:36:33 -0400 In-Reply-To: <1407945974-27300-1-git-send-email-wehuang@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: Il 13/08/2014 18:06, Wei Huang ha scritto: > Current KVM only supports RDMSR for K7_EVNTSEL0 and K7_PERFCTR0 > MSRs. Reading the rest MSRs will trigger KVM to inject #GP into > guest VM. This causes a warning message "Failed to access perfctr > msr (MSR c0010001 is ffffffffffffffff)" on AMD host. This patch > adds RDMSR support for all K7_EVNTSELn and K7_PERFCTRn registers > and thus supresses the warning message. > > Signed-off-by: Wei Huang > --- > arch/x86/kvm/x86.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c > index ef432f8..3f10ca2 100644 > --- a/arch/x86/kvm/x86.c > +++ b/arch/x86/kvm/x86.c > @@ -2399,7 +2399,13 @@ int kvm_get_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 *pdata) > case MSR_K7_HWCR: > case MSR_VM_HSAVE_PA: > case MSR_K7_EVNTSEL0: > + case MSR_K7_EVNTSEL1: > + case MSR_K7_EVNTSEL2: > + case MSR_K7_EVNTSEL3: > case MSR_K7_PERFCTR0: > + case MSR_K7_PERFCTR1: > + case MSR_K7_PERFCTR2: > + case MSR_K7_PERFCTR3: > case MSR_K8_INT_PENDING_MSG: > case MSR_AMD64_NB_CFG: > case MSR_FAM10H_MMIO_CONF_BASE: > Thanks, applying for 3.18. Paolo