qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v1] target/i386: Always set leaf 0x1f


From: Manish
Subject: Re: [PATCH v1] target/i386: Always set leaf 0x1f
Date: Mon, 29 Jul 2024 19:42:39 +0700
User-agent: Mozilla Thunderbird


On 29/07/24 7:18 pm, Igor Mammedov wrote:
!-------------------------------------------------------------------|
   CAUTION: External Email

|-------------------------------------------------------------------!

On Wed, 24 Jul 2024 23:00:13 +0800
Zhao Liu <zhao1.liu@intel.com> wrote:

Hi Igor,

On Wed, Jul 24, 2024 at 02:54:32PM +0200, Igor Mammedov wrote:
Date: Wed, 24 Jul 2024 14:54:32 +0200
From: Igor Mammedov <imammedo@redhat.com>
Subject: Re: [PATCH v1] target/i386: Always set leaf 0x1f
X-Mailer: Claws Mail 4.3.0 (GTK 3.24.42; x86_64-redhat-linux-gnu)

On Wed, 24 Jul 2024 12:13:28 +0100
John Levon <john.levon@nutanix.com> wrote:
On Wed, Jul 24, 2024 at 03:59:29PM +0530, Manish wrote:
Leaf 0x1f is superset of 0xb, so it makes sense to set 0x1f equivalent
to 0xb by default and workaround windows issue.>
This change adds a
new property 'cpuid-0x1f-enforce' to set leaf 0x1f equivalent to 0xb in
case extended CPU topology is not configured and behave as before otherwise.
repeating question
why we need to use extra property instead of just adding 0x1f leaf for CPU 
models
that supposed to have it?
As i mentioned in earlier response. "Windows expects it only when we have
set max cpuid level greater than or equal to 0x1f. I mean if it is exposed
it should not be all zeros. SapphireRapids CPU definition raised cpuid level
to 0x20, so we starting seeing it with SapphireRapids."

Windows does not expect 0x1f to be present for any CPU model. But if it is
exposed to the guest, it expects non-zero values.
I think Igor is suggesting:

  - leave x86_cpu_expand_features() alone completely
yep, drop that if possible

  - change the 0x1f handling to always report topology i.e. never report all
    zeroes
Do this but only for CPU models that have this leaf per spec,
to avoid live migration issues create a new version of CPU model,
so it would apply only for new version. This way older versions
and migration won't be affected.
So that in the future every new Intel CPU model will need to always
enable 0x1f. Sounds like an endless game. So my question is: at what
point is it ok to consider defaulting to always enable 0x1f and just
disable it for the old CPU model?
I have suggested to enable 0x1f leaf excluding:
    * existing cpu models (versions)
    * cpu models that do not have this leaf in real world should
      not have it in QEMU either.

If cpu model already exists, you'd need a new version of cpu model to
enable new leaf by default.

For completely new cpu model, it could be enabled from the start.
i.e. workflow for enabling that should be the same as with CPU features
(or as you said 'endless game' of copying base model and making it look like
should be according to spec,
but that's the process we currently use for describing CPU models).

Igor my understanding was that there are two type of features one is real CPU feature, yes those makes sense in CPU models. But on other hand there are features which are emulated ones i.e. kvm-*, these make sense enabling regardless of any CPU model and we usually use machine types to enable these. Does not this features makes sense in 2nd category.


Thanks,
Zhao


Thanks

Manish Mishra




reply via email to

[Prev in Thread] Current Thread [Next in Thread]