From: "Chen, Zide" <zide.chen@intel.com>
To: Jim Mattson <jmattson@google.com>,
"Mi, Dapeng" <dapeng1.mi@linux.intel.com>
Cc: Sean Christopherson <seanjc@google.com>,
Paolo Bonzini <pbonzini@redhat.com>,
kvm@vger.kernel.org, linux-kernel@vger.kernel.org,
Mingwei Zhang <mizhang@google.com>,
Das Sandipan <Sandipan.Das@amd.com>,
Shukla Manali <Manali.Shukla@amd.com>,
Falcon Thomas <thomas.falcon@intel.com>,
Xudong Hao <xudong.hao@intel.com>
Subject: Re: [PATCH v6 8/8] KVM: selftests: Add PERF_METRICS and fixed counter 3 tests
Date: Tue, 21 Jul 2026 14:17:45 -0500 [thread overview]
Message-ID: <7fc29642-fba2-4752-9095-10c89cacb8d2@intel.com> (raw)
In-Reply-To: <CALMp9eR0yXVPqOUZzoBSmGTpONojSALS7X-Rrq8QkaivnNgt8w@mail.gmail.com>
On 7/14/2026 12:06 PM, Jim Mattson wrote:
> On Mon, Jul 13, 2026 at 10:28 PM Mi, Dapeng <dapeng1.mi@linux.intel.com> wrote:
>>
>>
>> On 7/14/2026 1:11 PM, Jim Mattson wrote:
>>> On Mon, Jul 13, 2026 at 12:12 AM Mi, Dapeng <dapeng1.mi@linux.intel.com> wrote:
>>>
>>>> rdpmc is the preferable way to read PMU counters since the better
>>>> performance than rdmsr, and most of current code regardless of kernel or
>>>> user-space already use rdpmc to read PMC counters. If we set the rdpmc to
>>>> interception by default, then it may cause extra performance hit.
>>>>
>>>> Currently I didn't hear there is any plan to extend the rdpmc to support
>>>> more types for Intel platforms ...
>>> Is Intel willing to commit to that?
>>
>> I'm afraid not. :(
>
> Then, KVM must always intercept RDPMC.
>
>> rdpmc is still possible to be extended to support more types in the future,
>> but recently we don't see such requirements.
>
> That's no comfort.
>
> An alternative would be for Intel to provide a way for KVM to query
> which types of PMC are supported in ECX[31:16], before or concurrently
> with any microarchitecture that supports more types than CPUs support
> today.
While I can't speak for future Intel architectures, we're not aware of
any plans to extend RDPMC with additional resource types in the
foreseeable future.
Regarding the proposed enumeration mechanism, I'm not sure it changes
the fundamental situation. Even if a future architecture provides an
enumeration mechanism for new RDPMC resource types, RDPMC passthrough on
an older KVM could still exist until KVM gains support for the new
resource types.
Given that, I'm not sure the possibility of a future RDPMC extension by
itself is sufficient justification for intercepting RDPMC
unconditionally today. RDPMC is the standard low-overhead interface for
reading PMU data, and forcing every RDPMC through a VM-Exit would impose
a significant performance penalty.
next prev parent reply other threads:[~2026-07-21 19:17 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-29 23:19 [PATCH V6 0/8] KVM: x86/pmu: Add hardware Topdown metrics support Zide Chen
2026-06-29 23:19 ` [PATCH v6 1/8] KVM: x86/pmu: Do not map fixed counters >= 3 to generic perf events Zide Chen
2026-06-30 2:13 ` Mi, Dapeng
2026-07-22 21:51 ` Jim Mattson
2026-06-29 23:19 ` [PATCH v6 2/8] KVM: x86/pmu: Support Intel fixed counter 3 on mediated vPMU Zide Chen
2026-06-30 2:16 ` Mi, Dapeng
2026-07-22 22:17 ` Jim Mattson
2026-07-23 15:36 ` Chen, Zide
2026-07-23 19:19 ` Jim Mattson
2026-06-29 23:19 ` [PATCH v6 3/8] KVM: x86/pmu: Rename and move vcpu_get_perf_capabilities() to pmu.h Zide Chen
2026-06-30 2:18 ` Mi, Dapeng
2026-07-22 22:47 ` Jim Mattson
2026-06-29 23:19 ` [PATCH v6 4/8] KVM: x86/pmu: Snapshot host IA32_PERF_CAPABILITIES in kvm_host Zide Chen
2026-06-30 2:19 ` Mi, Dapeng
2026-07-22 22:58 ` Jim Mattson
2026-06-29 23:19 ` [PATCH v6 5/8] KVM: x86/pmu: Support PERF_METRICS MSR in mediated vPMU Zide Chen
2026-06-30 2:20 ` Mi, Dapeng
2026-07-22 23:05 ` Jim Mattson
2026-07-23 16:30 ` Chen, Zide
2026-07-23 17:59 ` Jim Mattson
2026-07-23 23:44 ` Chen, Zide
2026-07-24 3:15 ` Jim Mattson
2026-07-24 20:47 ` Chen, Zide
2026-07-24 21:37 ` Jim Mattson
2026-07-24 22:59 ` Chen, Zide
2026-07-26 15:49 ` Jim Mattson
2026-07-27 15:34 ` Chen, Zide
2026-06-29 23:19 ` [PATCH v6 6/8] KVM: x86/pmu: Move RDPMC emulation into per-vendor callbacks Zide Chen
2026-06-30 2:23 ` Mi, Dapeng
2026-07-22 23:19 ` Jim Mattson
2026-06-29 23:19 ` [PATCH v6 7/8] KVM: x86/pmu: Emulate RDPMC on performance metrics Zide Chen
2026-06-30 2:23 ` Mi, Dapeng
2026-07-23 0:02 ` Jim Mattson
2026-07-23 1:44 ` Mi, Dapeng
2026-07-23 2:25 ` Jim Mattson
2026-07-23 2:52 ` Mi, Dapeng
2026-07-23 3:10 ` Jim Mattson
2026-07-23 5:47 ` Mi, Dapeng
2026-07-23 16:33 ` Chen, Zide
2026-07-23 19:08 ` Chen, Zide
2026-06-29 23:19 ` [PATCH v6 8/8] KVM: selftests: Add PERF_METRICS and fixed counter 3 tests Zide Chen
2026-06-30 2:36 ` Mi, Dapeng
2026-07-09 12:35 ` Jim Mattson
2026-07-10 8:08 ` Mi, Dapeng
2026-07-10 14:52 ` Chen, Zide
2026-07-13 6:57 ` Mi, Dapeng
2026-07-10 15:22 ` Jim Mattson
2026-07-13 7:11 ` Mi, Dapeng
2026-07-14 5:11 ` Jim Mattson
2026-07-14 5:28 ` Mi, Dapeng
2026-07-14 17:06 ` Jim Mattson
2026-07-21 19:17 ` Chen, Zide [this message]
2026-07-21 19:48 ` Jim Mattson
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=7fc29642-fba2-4752-9095-10c89cacb8d2@intel.com \
--to=zide.chen@intel.com \
--cc=Manali.Shukla@amd.com \
--cc=Sandipan.Das@amd.com \
--cc=dapeng1.mi@linux.intel.com \
--cc=jmattson@google.com \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mizhang@google.com \
--cc=pbonzini@redhat.com \
--cc=seanjc@google.com \
--cc=thomas.falcon@intel.com \
--cc=xudong.hao@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®