From: "Chang S. Bae" <chang.seok.bae@intel.com>
To: Sean Christopherson <seanjc@google.com>
Cc: <linux-kernel@vger.kernel.org>, <kvm@vger.kernel.org>,
<pbonzini@redhat.com>, <stable@vger.kernel.org>,
Zhao Liu <zhao1.liu@intel.com>,
Binbin Wu <binbin.wu@linux.intel.com>
Subject: Re: [PATCH v2] KVM: x86: Remove AMX-TF32 enumeration
Date: Thu, 23 Jul 2026 12:44:37 -0700 [thread overview]
Message-ID: <52cd5021-0290-401e-85d8-bdff0d07a9f7@intel.com> (raw)
In-Reply-To: <amJXsVGv3xe7GKu1@google.com>
On 7/23/2026 11:04 AM, Sean Christopherson wrote:
>
> I'm not convinced that yanking out support is the right move. By publishing the
> feature, for several years, Intel pretty much burned that CPUID bit. E.g. it's
> not impossible that someone other than Intel has implemented TF32 instructions.
>
> The opcodes are fair game in the future, because their use can be qualified and
> guarded, but the same doesn't hold true for enumeration. E.g. similar to how
> MPX got ripped out and its XSTATE assets got reused for APX, but the CPUID feature
> enumeration is forever reserved for MPX (I hope).
>
> If the argument is that "Intel Architecture Instruction Set Extensions and Future
> Features" isn't binding documentation, i.e. that Intel can do take backs until a
> feature is documented in the SDM, then I honestly don't see any option but to
> refuse to accept new features until they land in the SDM. Which I don't think
> any of us want.
One distinction I could think of is whether a feature ever ships in
implementations. If a feature is actually implemented and is later
deprecated or removed, then I agree the bit should remain reserved. That
appears quite clear.
My understanding is that the revert was considered because TF32 never
reached any implementation. If that's incorrect, I'd certainly welcome a
correction. But if it holds true, then I also want to clarify the case
is different from retiring a feature that was already in products.
That said though, I think you made a point. If new features are
published there, we add support based on that, and are later asked to
remove the support, it becomes harder to convince what level of
stability the ISE doc can provide.
I don't think these decisions are made lightly. We humans may sometimes
reconsider features before they ever ship. But cases like this
inevitably damage the credibility. If this happens repeatedly, it
becomes increasingly reasonable to take a more conservative approach --
perhaps even until there is hardware implementation at hand.
I'd relay this feedback to the relevant folks, as I do think this is an
important point to consider. I'll cc you.
Thanks,
Chang
next prev parent reply other threads:[~2026-07-23 19:45 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-20 20:52 [PATCH v2] KVM: x86: Remove AMX-TF32 enumeration Chang S. Bae
2026-07-21 0:37 ` Xiaoyao Li
2026-07-23 18:04 ` Sean Christopherson
2026-07-23 19:44 ` Chang S. Bae [this message]
2026-07-27 15:27 ` Sean Christopherson
-- strict thread matches above, loose matches on Subject: below --
2026-07-24 15:37 Christian Ludloff
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=52cd5021-0290-401e-85d8-bdff0d07a9f7@intel.com \
--to=chang.seok.bae@intel.com \
--cc=binbin.wu@linux.intel.com \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pbonzini@redhat.com \
--cc=seanjc@google.com \
--cc=stable@vger.kernel.org \
--cc=zhao1.liu@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