From: "Edgecombe, Rick P" <rick.p.edgecombe@intel.com>
To: "Li, Xiaoyao" <xiaoyao.li@intel.com>,
"binbin.wu@linux.intel.com" <binbin.wu@linux.intel.com>
Cc: "Gao, Chao" <chao.gao@intel.com>,
"seanjc@google.com" <seanjc@google.com>,
"dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
"kas@kernel.org" <kas@kernel.org>,
"Maloor, Kishen" <kishen.maloor@intel.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"tony.lindgren@linux.intel.com" <tony.lindgren@linux.intel.com>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
"pbonzini@redhat.com" <pbonzini@redhat.com>,
"nik.borisov@suse.com" <nik.borisov@suse.com>,
"dedekind1@gmail.com" <dedekind1@gmail.com>,
"andrew.cooper3@citrix.com" <andrew.cooper3@citrix.com>
Subject: Re: [PATCH v4 1/4] KVM: TDX: Track configurable CPUID bits allowed by KVM
Date: Wed, 23 Sep 2026 14:25:49 +0000 [thread overview]
Message-ID: <a085134ba1e80e04ec5737dffaadd5abeb8c6d68.camel@intel.com> (raw)
In-Reply-To: <2cc4bbc3-11d3-48bd-9a9b-a9659fc101a1@intel.com>
On Wed, 2026-09-23 at 19:21 +0800, Xiaoyao Li wrote:
> > But userspace doesn't know these bits are not supported by TDX module and
> > KVM. If KVM report them as supported, and it causes problems, we cannot say
> > it's userspace's fault.
>
> It sounds like a bug fix to existing KVM behavior.
>
> Current KVM doesn't allow HLE/RTM/WAITPKG because it might cause bad effect on
> the host. But for those #VE related, though KVM doesn't support virtualizing
> the features, it doesn't cause any harm to the host when KVM allows them to be
> configured to TDs. It only causes problems to TDs.
>
> For normal VMs, KVM_GET_SUPPORTED_CPUID and other KVM CAPs serve as the
> interface to report to userspace if a feature is support or not. If userspace
> exposes a feature to guest when KVM reports the feature is not supported, and
> it causes problem to guests, we can say it's userspace's fault.
Yes.
> But for TDX,
> there is not interface to report KVM's support capabilities. After this
> series, KVM_TDX_CAPABILITIES can serve as the interface. So it looks like a
> bug fix to me.
KVM needs to support all the bits that are 1 in the TD. Either directly
configurable or fixed-1 or whatever. Userspace doesn't find out which bits these
are until it does the KVM_TDX_GET_CPUID, at which point it finds out the
"support" in a heavy handed way. So I think userspace does learn everything it
needs already. And as long we don't have new fixed-1 bits, we will avoid KVM
being forced to support things too early.
Except for exposing TD KVM PV bit support, which we already talked about doing
as follow on.
next prev parent reply other threads:[~2026-09-23 14:25 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-17 7:25 [PATCH v4 0/4] KVM: TDX: Validate directly configurable CPUID bits Binbin Wu
2026-09-17 7:25 ` [PATCH v4 1/4] KVM: TDX: Track configurable CPUID bits allowed by KVM Binbin Wu
2026-09-23 0:01 ` Edgecombe, Rick P
2026-09-23 0:14 ` Binbin Wu
2026-09-23 0:45 ` Edgecombe, Rick P
2026-09-23 0:57 ` Binbin Wu
2026-09-23 1:09 ` Edgecombe, Rick P
2026-09-23 1:17 ` Binbin Wu
2026-09-23 11:21 ` Xiaoyao Li
2026-09-23 14:25 ` Edgecombe, Rick P [this message]
2026-09-17 7:25 ` [PATCH v4 2/4] KVM: TDX: Report CORE_CAPABILITIES as configurable Binbin Wu
2026-09-22 21:11 ` Edgecombe, Rick P
2026-09-23 0:03 ` Binbin Wu
2026-09-17 7:25 ` [PATCH v4 3/4] KVM: TDX: Filter configurable CPUID bits Binbin Wu
2026-09-23 0:16 ` Edgecombe, Rick P
2026-09-23 0:28 ` Binbin Wu
2026-09-23 0:34 ` Edgecombe, Rick P
2026-09-17 7:25 ` [PATCH v4 4/4] KVM: TDX: Validate userspace CPUID input for KVM_TDX_INIT_VM Binbin Wu
2026-09-23 0:16 ` Edgecombe, Rick P
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=a085134ba1e80e04ec5737dffaadd5abeb8c6d68.camel@intel.com \
--to=rick.p.edgecombe@intel.com \
--cc=andrew.cooper3@citrix.com \
--cc=binbin.wu@linux.intel.com \
--cc=chao.gao@intel.com \
--cc=dave.hansen@linux.intel.com \
--cc=dedekind1@gmail.com \
--cc=kas@kernel.org \
--cc=kishen.maloor@intel.com \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=nik.borisov@suse.com \
--cc=pbonzini@redhat.com \
--cc=seanjc@google.com \
--cc=tony.lindgren@linux.intel.com \
--cc=xiaoyao.li@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®