From: David Ahern <daahern@cisco.com>
To: Avi Kivity <avi@redhat.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>,
kvm@vger.kernel.org, Ingo Molnar <mingo@elte.hu>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Arnaldo Carvalho de Melo <acme@ghostprotocols.net>
Subject: Re: [PATCH v2 00/11] KVM in-guest performance monitoring
Date: Tue, 14 Jun 2011 12:11:02 -0600 [thread overview]
Message-ID: <4DF7A436.8090308@cisco.com> (raw)
In-Reply-To: <4DF79EFE.1050201@cisco.com>
On 06/14/2011 11:48 AM, David Ahern wrote:
> On 06/14/2011 11:33 AM, David Ahern wrote:
>> Avi: still no luck:
>> [ 0.047996] Performance Events: unsupported p6 CPU model 0 no PMU
>> driver, software events only.
>>
>> qemu-kvm next branch, ce5f0a588b740e8f28f46a6009e12cfa72edc51f with your
>> perfmon cpuid change. Host and guest are both running your kvm next
>> branch with pmu patch series.
>
> The perf init code is going down the !perfmon route:
>
> if (!cpu_has(&boot_cpu_data, X86_FEATURE_ARCH_PERFMON)) {
> switch (boot_cpu_data.x86) {
> case 0x6:
> return p6_pmu_init();
> case 0xf:
> return p4_pmu_init();
> }
> return -ENODEV;
> }
>
> Based on Patch 2 you are expecting the guest to have this feature set.
> I've tried +perfmon and +arch_perfmon in the cpu definition for qemu-kvm
> (e.g., -cpu host,model=0,+perfmon) no luck
>
nevermind. I hand applied your qemu-kvm patch and changed ebx not eax. I
noticed init_intel() looked at eax and discovered the user error.
Application of patch fixed and it works. :-)
David
next prev parent reply other threads:[~2011-06-14 18:11 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-13 13:34 [PATCH v2 00/11] KVM in-guest performance monitoring Avi Kivity
2011-06-13 13:34 ` [PATCH v2 01/11] perf: add context field to perf_event Avi Kivity
2011-06-13 13:34 ` [PATCH v2 02/11] x86, perf: add constraints for architectural PMU v1 Avi Kivity
2011-06-13 13:34 ` [PATCH v2 03/11] perf: export perf_event_refresh() to modules Avi Kivity
2011-06-13 13:34 ` [PATCH v2 04/11] KVM: Expose kvm_lapic_local_deliver() Avi Kivity
2011-06-13 13:35 ` [PATCH v2 05/11] KVM: Expose a version 1 architectural PMU to guests Avi Kivity
2011-06-13 18:10 ` David Ahern
2011-06-14 8:33 ` Avi Kivity
2011-06-27 14:10 ` lidong chen
2011-06-27 14:13 ` Avi Kivity
2011-06-28 2:54 ` lidong chen
2011-06-28 8:15 ` Avi Kivity
2011-06-13 13:35 ` [PATCH v2 06/11] KVM: Add generic RDPMC support Avi Kivity
2011-06-13 13:35 ` [PATCH v2 07/11] KVM: SVM: Intercept RDPMC Avi Kivity
2011-06-13 13:35 ` [PATCH v2 08/11] KVM: VMX: " Avi Kivity
2011-06-13 16:17 ` David Ahern
2011-06-13 16:29 ` Avi Kivity
2011-06-13 16:59 ` David Ahern
2011-06-13 17:38 ` Ingo Molnar
2011-06-13 18:09 ` David Ahern
2011-06-14 8:34 ` Avi Kivity
2011-06-14 9:27 ` Ingo Molnar
2011-06-13 13:35 ` [PATCH v2 09/11] KVM: Expose the architectural performance monitoring CPUID leaf Avi Kivity
2011-06-13 13:35 ` [PATCH v2 10/11] KVM: x86 emulator: fix RDPMC privilege check Avi Kivity
2011-06-13 13:35 ` [PATCH v2 11/11] KVM: x86 emulator: implement RDPMC (0F 33) Avi Kivity
2011-06-13 19:55 ` [PATCH v2 00/11] KVM in-guest performance monitoring David Ahern
2011-06-14 8:36 ` Avi Kivity
2011-06-14 17:15 ` David Ahern
2011-06-14 17:24 ` Jan Kiszka
2011-06-14 17:33 ` David Ahern
2011-06-14 17:48 ` David Ahern
2011-06-14 18:11 ` David Ahern [this message]
2011-06-15 8:57 ` Avi Kivity
2011-06-15 12:40 ` David Ahern
2011-06-15 13:22 ` Avi Kivity
2011-06-15 16:08 ` David Ahern
2011-06-15 16:27 ` Avi Kivity
2011-06-15 16:51 ` David Ahern
2011-06-16 13:53 ` Avi Kivity
2011-06-16 14:08 ` David Ahern
2011-06-16 14:19 ` David Ahern
2011-06-16 14:20 ` Avi Kivity
2011-06-16 14:32 ` David Ahern
2011-06-16 14:36 ` Avi Kivity
2011-06-16 15:04 ` David Ahern
2011-06-16 15:08 ` Peter Zijlstra
2011-06-16 15:18 ` David Ahern
2011-06-16 15:19 ` David Ahern
2011-06-16 15:27 ` Peter Zijlstra
2011-06-16 15:34 ` David Ahern
2011-06-16 15:59 ` Avi Kivity
2011-06-16 16:04 ` David Ahern
2011-06-16 16:31 ` Avi Kivity
2011-06-16 17:16 ` David Ahern
2011-06-29 7:52 ` Avi Kivity
2011-06-29 8:38 ` Peter Zijlstra
2011-06-29 9:26 ` Avi Kivity
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4DF7A436.8090308@cisco.com \
--to=daahern@cisco.com \
--cc=a.p.zijlstra@chello.nl \
--cc=acme@ghostprotocols.net \
--cc=avi@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=mtosatti@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.