From: Florian Weimer <fweimer@redhat.com>
To: Zhao Liu <zhao1.liu@intel.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
qemu-devel@nongnu.org, kvm@vger.kernel.org,
"Chang S . Bae" <chang.seok.bae@intel.com>,
Zide Chen <zide.chen@intel.com>,
Xudong Hao <xudong.hao@intel.com>,
Peter Fang <peter.fang@intel.com>
Subject: Re: [PATCH 4/5] i386/cpu: Support APX CPUIDs
Date: Wed, 19 Nov 2025 19:04:36 +0100 [thread overview]
Message-ID: <lhuh5upzyob.fsf@oldenburg.str.redhat.com> (raw)
In-Reply-To: <aR1zIb4GHh9FrK31@intel.com> (Zhao Liu's message of "Wed, 19 Nov 2025 15:34:57 +0800")
* Zhao Liu:
>> Please just make the new leaf have constant values based on just
>> APX_F. We'll add the optional NCI/NDD/NF support if needed, i.e.
>> never. :)
>
> Maybe not never?
>
>> > Note, APX_NCI_NDD_NF is documented as always enabled for Intel
>> > processors since APX spec (revision v7.0). Now any Intel processor
>> > that enumerates support for APX_F (CPUID.(EAX=0x7, ECX=1).EDX[21])
>> > will also enumerate support for APX_NCI_NDD_NF.
>
> This sentence (from APX spec rev.7) emphasizes the “Intel” vendor,
> and its primary goal was to address and explain compatibility concern
> for pre-enabling work based on APX spec v6. Prior to v7, APX included
> NCI_NDD_NF by default, but this feature has now been separated from
> basic APX and requires explicit checking CPUID bit.
>
> x86 ecosystem advisory group has aligned on APX so it may be possible
> for other x86 vendors to implement APX without NCI_NDD_NF and this still
> match with the APX spec.
Well yes, but I doubt that the ecosystem will produce binaries
specialized for APX *without* NDD. It's fine to enumerate it
separately, but that doesn't have any immediate consequences. GCC makes
it rather hard to build for APX without NDD, for example. At least more
difficult than building for AVX-512F without AVX-512VL.
I just don't think software vendors are enthusiastic about having to
create and support not one, but two builds for APX. If NDD is optional
in practice, it will not be possible to use it except for run-time
generated code and perhaps very targeted optimizations because that
single extra APX will just not use NDD.
I feel like there has been a misunderstanding somewhere.
(sorry for off-topic)
Thanks,
Florian
next prev parent reply other threads:[~2025-11-19 18:04 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-18 6:58 [PATCH 0/5] i386/cpu: Support APX for KVM Zhao Liu
2025-11-18 6:58 ` [PATCH 1/5] i386/cpu: Add APX EGPRs into xsave area Zhao Liu
2025-11-18 6:58 ` [PATCH 2/5] i386/cpu: Cache EGPRs in CPUX86State Zhao Liu
2025-11-18 8:43 ` Paolo Bonzini
2025-11-19 7:47 ` Zhao Liu
2025-11-18 6:58 ` [PATCH 3/5] i386/cpu: Add APX migration support Zhao Liu
2025-11-18 6:58 ` [PATCH 4/5] i386/cpu: Support APX CPUIDs Zhao Liu
2025-11-18 8:44 ` Paolo Bonzini
2025-11-19 7:34 ` Zhao Liu
2025-11-19 18:04 ` Florian Weimer [this message]
2025-11-19 18:08 ` Paolo Bonzini
2025-11-18 6:58 ` [PATCH 5/5] i386/cpu: Mark apx xstate as migratable Zhao Liu
2025-11-18 8:45 ` [PATCH 0/5] i386/cpu: Support APX for KVM Paolo Bonzini
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=lhuh5upzyob.fsf@oldenburg.str.redhat.com \
--to=fweimer@redhat.com \
--cc=chang.seok.bae@intel.com \
--cc=kvm@vger.kernel.org \
--cc=pbonzini@redhat.com \
--cc=peter.fang@intel.com \
--cc=qemu-devel@nongnu.org \
--cc=xudong.hao@intel.com \
--cc=zhao1.liu@intel.com \
--cc=zide.chen@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