From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] Remove MSR_P6_{EVNTSEL0,PERFCTR0} from printk warning list. Date: Thu, 01 Jul 2010 17:38:02 +0300 Message-ID: <4C2CA84A.4090506@redhat.com> References: <1277908040-15087-1-git-send-email-Jes.Sorensen@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org To: Jes.Sorensen@redhat.com Return-path: Received: from mx1.redhat.com ([209.132.183.28]:18039 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754103Ab0GAOiE (ORCPT ); Thu, 1 Jul 2010 10:38:04 -0400 Received: from int-mx05.intmail.prod.int.phx2.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.18]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id o61Ec4jp013563 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Thu, 1 Jul 2010 10:38:04 -0400 Received: from cleopatra.tlv.redhat.com (cleopatra.tlv.redhat.com [10.35.255.11]) by int-mx05.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id o61Ec2UN003450 for ; Thu, 1 Jul 2010 10:38:03 -0400 In-Reply-To: <1277908040-15087-1-git-send-email-Jes.Sorensen@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 06/30/2010 05:27 PM, Jes.Sorensen@redhat.com wrote: > From: Jes Sorensen > > MSR_P6_EVNTSEL0 and MSR_P6_PERFCTR0 are used to probe for the P6 PMU > for older family 6 CPUs, which is also the default in QEMU. Ie. per > default we get the noise of these warnings in dmesg, confusing users > for no reason. > > For the reason, see the comment above. The guest may be trying to do something with the counters and expect them to work. Without the warning we'd have no idea we are knowningly misemulating something. > Signed-off-by: Jes Sorensen > --- > arch/x86/kvm/x86.c | 2 -- > 1 files changed, 0 insertions(+), 2 deletions(-) > > diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c > index d0b9252..ac55b92 100644 > --- a/arch/x86/kvm/x86.c > +++ b/arch/x86/kvm/x86.c > @@ -1306,7 +1306,6 @@ int kvm_set_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 data) > * which we perfectly emulate ;-). Any other value should be at least > * reported, some guests depend on them. > */ > - case MSR_P6_EVNTSEL0: > case MSR_P6_EVNTSEL1: > case MSR_K7_EVNTSEL0: > case MSR_K7_EVNTSEL1: > @@ -1319,7 +1318,6 @@ int kvm_set_msr_common(struct kvm_vcpu *vcpu, u32 msr, u64 data) > /* at least RHEL 4 unconditionally writes to the perfctr registers, > * so we ignore writes to make it happy. > */ > - case MSR_P6_PERFCTR0: > case MSR_P6_PERFCTR1: > case MSR_K7_PERFCTR0: > case MSR_K7_PERFCTR1: > -- error compiling committee.c: too many arguments to function