From: Sean Christopherson <seanjc@google.com>
To: Sean Christopherson <seanjc@google.com>,
Paolo Bonzini <pbonzini@redhat.com>,
Thomas Gleixner <tglx@kernel.org>,
Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>,
Dave Hansen <dave.hansen@linux.intel.com>,
x86@kernel.org, Kiryl Shutsemau <kas@kernel.org>,
Rick Edgecombe <rick.p.edgecombe@intel.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>,
kvm@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-coco@lists.linux.dev,
Binbin Wu <binbin.wu@linux.intel.com>,
Changyuan Lyu <changyuanl@google.com>,
Sashiko <sashiko-bot@kernel.org>
Subject: [PATCH v2 0/3] x86/cpuid: KVM: TDX CPUID fixes
Date: Thu, 22 Oct 2026 14:01:31 -0700 [thread overview]
Message-ID: <20261022210134.612848-1-seanjc@google.com> (raw)
Boris or Dave, can you grab patch 1 for 7.4, ideally sooner than later? That
would avoid having to carry a stable topic branch for a trivial change.
Annotate CPUID entries that have a significant index when reporting TDX
capabilities to userspace, and enforce that userspace honors significant
indices to avoid creating a misconfiguring between KVM and the TDX Module,
which in the worst case scenario could lead to host state getting clobbered.
Patch 1 is a fix to mark function 0x23 as being indexed. That change is
needed to avoid tripping a WARN sanity check that KVM and the TDX Module
agree on which functions are indexed.
I'd like to take this through the KVM tree as it seems extremely unlikely
the cpuid_function_is_indexed() will conflict or grow other dependencies,
i.e. creating a shared topic branch for this seems rather silly. Actually,
a better alternative would be to rush patch 1 into 7.4 so that I can use
7.4-rc2 as a base (even rc3 or rc4 would be a-ok). /me adds an intro
Note, SIGNIFCANT_INDEX isn't a typo, KVM's uAPI macro is misspelled...
v2:
- Add CPUID.0x23 to the indexed set. [Rick, Binbin]
- Document that zeroing the index when it's supposed to be ignored is KVM
ABI that at least QEMU relies on. [Binbin]
- Zero out all of flags to ensure garbage isn't reported to userspace.
v1: https://lore.kernel.org/all/20260223214336.722463-1-changyuanl@google.com
Changyuan Lyu (1):
KVM: TDX: Set SIGNIFCANT_INDEX flag as appropriate when filling CPUID
entries
Sean Christopherson (2):
x86/cpuid: Mark CPUID function 0x23 as being indexed
KVM: TDX: Reject KVM_TDX_INIT_VM if indexed CPUID functions don't
match
arch/x86/include/asm/cpuid/api.h | 1 +
arch/x86/kvm/vmx/tdx.c | 22 +++++++++++++++++++---
2 files changed, 20 insertions(+), 3 deletions(-)
base-commit: 814607cd40df80d5727957adaa857e0feca43497
--
2.56.0.257.gf7497c7d27-goog
next reply other threads:[~2026-10-22 21:01 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-22 21:01 Sean Christopherson [this message]
2026-10-22 21:01 ` [PATCH v2 1/3] x86/cpuid: Mark CPUID function 0x23 as being indexed Sean Christopherson
2026-10-22 21:01 ` [PATCH v2 2/3] KVM: TDX: Set SIGNIFCANT_INDEX flag as appropriate when filling CPUID entries Sean Christopherson
2026-10-22 21:01 ` [PATCH v2 3/3] KVM: TDX: Reject KVM_TDX_INIT_VM if indexed CPUID functions don't match Sean Christopherson
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=20261022210134.612848-1-seanjc@google.com \
--to=seanjc@google.com \
--cc=binbin.wu@linux.intel.com \
--cc=bp@alien8.de \
--cc=changyuanl@google.com \
--cc=dave.hansen@linux.intel.com \
--cc=hpa@zytor.com \
--cc=kas@kernel.org \
--cc=kvm@vger.kernel.org \
--cc=linux-coco@lists.linux.dev \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=pbonzini@redhat.com \
--cc=rick.p.edgecombe@intel.com \
--cc=sashiko-bot@kernel.org \
--cc=tglx@kernel.org \
--cc=x86@kernel.org \
/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®