* [Qemu-devel] [PATCH] x86: Fix Opteron xlevels
@ 2015-04-21 14:04 Alexander Graf
2015-04-21 14:16 ` Eduardo Habkost
0 siblings, 1 reply; 6+ messages in thread
From: Alexander Graf @ 2015-04-21 14:04 UTC (permalink / raw)
To: qemu-devel; +Cc: bwiedemann, afaerber
The AMD Opteron family has different xlevel levels depending on the
generation. I looked up Gen1, Gen2 and Gen3 hardware and adapted the
levels according to real silicon.
The reason this came up is that there is a sanity check in KVM making
sure that SVM is only used when xlevel is high enough. Using real
hardware levels, they now are.
Reported-by: Bernhard M. Wiedemann <bwiedemann@suse.de>
Signed-off-by: Alexander Graf <agraf@suse.de>
---
target-i386/cpu.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 03b33cf..d1b1b8c 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -1234,7 +1234,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
CPUID_EXT2_MTRR | CPUID_EXT2_SYSCALL | CPUID_EXT2_APIC |
CPUID_EXT2_CX8 | CPUID_EXT2_MCE | CPUID_EXT2_PAE | CPUID_EXT2_MSR |
CPUID_EXT2_TSC | CPUID_EXT2_PSE | CPUID_EXT2_DE | CPUID_EXT2_FPU,
- .xlevel = 0x80000008,
+ .xlevel = 0x80000018,
.model_id = "AMD Opteron 240 (Gen 1 Class Opteron)",
},
{
@@ -1262,7 +1262,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
CPUID_EXT2_DE | CPUID_EXT2_FPU,
.features[FEAT_8000_0001_ECX] =
CPUID_EXT3_SVM | CPUID_EXT3_LAHF_LM,
- .xlevel = 0x80000008,
+ .xlevel = 0x80000018,
.model_id = "AMD Opteron 22xx (Gen 2 Class Opteron)",
},
{
@@ -1292,7 +1292,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
.features[FEAT_8000_0001_ECX] =
CPUID_EXT3_MISALIGNSSE | CPUID_EXT3_SSE4A |
CPUID_EXT3_ABM | CPUID_EXT3_SVM | CPUID_EXT3_LAHF_LM,
- .xlevel = 0x80000008,
+ .xlevel = 0x8000001A,
.model_id = "AMD Opteron 23xx (Gen 3 Class Opteron)",
},
{
--
1.7.12.4
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [Qemu-devel] [PATCH] x86: Fix Opteron xlevels
2015-04-21 14:04 [Qemu-devel] [PATCH] x86: Fix Opteron xlevels Alexander Graf
@ 2015-04-21 14:16 ` Eduardo Habkost
2015-04-21 14:15 ` Alexander Graf
0 siblings, 1 reply; 6+ messages in thread
From: Eduardo Habkost @ 2015-04-21 14:16 UTC (permalink / raw)
To: Alexander Graf; +Cc: bwiedemann, qemu-devel, afaerber
On Tue, Apr 21, 2015 at 04:04:21PM +0200, Alexander Graf wrote:
> The AMD Opteron family has different xlevel levels depending on the
> generation. I looked up Gen1, Gen2 and Gen3 hardware and adapted the
> levels according to real silicon.
>
> The reason this came up is that there is a sanity check in KVM making
> sure that SVM is only used when xlevel is high enough. Using real
> hardware levels, they now are.
>
> Reported-by: Bernhard M. Wiedemann <bwiedemann@suse.de>
> Signed-off-by: Alexander Graf <agraf@suse.de>
It needs compatibility properties in HW_COMPAT_2_1. See commit
6b11322e0f724eb0649fdc324a44288b783023ad for reference.
> ---
> target-i386/cpu.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> index 03b33cf..d1b1b8c 100644
> --- a/target-i386/cpu.c
> +++ b/target-i386/cpu.c
> @@ -1234,7 +1234,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
> CPUID_EXT2_MTRR | CPUID_EXT2_SYSCALL | CPUID_EXT2_APIC |
> CPUID_EXT2_CX8 | CPUID_EXT2_MCE | CPUID_EXT2_PAE | CPUID_EXT2_MSR |
> CPUID_EXT2_TSC | CPUID_EXT2_PSE | CPUID_EXT2_DE | CPUID_EXT2_FPU,
> - .xlevel = 0x80000008,
> + .xlevel = 0x80000018,
Why did you choose 0x80000018? The highest 0x80000000 leaf we implement
today is 0x8000000A. SVM info is at 0x8000000A.
> .model_id = "AMD Opteron 240 (Gen 1 Class Opteron)",
> },
> {
> @@ -1262,7 +1262,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
> CPUID_EXT2_DE | CPUID_EXT2_FPU,
> .features[FEAT_8000_0001_ECX] =
> CPUID_EXT3_SVM | CPUID_EXT3_LAHF_LM,
> - .xlevel = 0x80000008,
> + .xlevel = 0x80000018,
> .model_id = "AMD Opteron 22xx (Gen 2 Class Opteron)",
> },
> {
> @@ -1292,7 +1292,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
> .features[FEAT_8000_0001_ECX] =
> CPUID_EXT3_MISALIGNSSE | CPUID_EXT3_SSE4A |
> CPUID_EXT3_ABM | CPUID_EXT3_SVM | CPUID_EXT3_LAHF_LM,
> - .xlevel = 0x80000008,
> + .xlevel = 0x8000001A,
> .model_id = "AMD Opteron 23xx (Gen 3 Class Opteron)",
> },
> {
> --
> 1.7.12.4
>
>
--
Eduardo
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Qemu-devel] [PATCH] x86: Fix Opteron xlevels
2015-04-21 14:16 ` Eduardo Habkost
@ 2015-04-21 14:15 ` Alexander Graf
2015-04-22 14:23 ` Eduardo Habkost
0 siblings, 1 reply; 6+ messages in thread
From: Alexander Graf @ 2015-04-21 14:15 UTC (permalink / raw)
To: Eduardo Habkost; +Cc: bwiedemann, qemu-devel, afaerber
On 04/21/2015 04:16 PM, Eduardo Habkost wrote:
> On Tue, Apr 21, 2015 at 04:04:21PM +0200, Alexander Graf wrote:
>> The AMD Opteron family has different xlevel levels depending on the
>> generation. I looked up Gen1, Gen2 and Gen3 hardware and adapted the
>> levels according to real silicon.
>>
>> The reason this came up is that there is a sanity check in KVM making
>> sure that SVM is only used when xlevel is high enough. Using real
>> hardware levels, they now are.
>>
>> Reported-by: Bernhard M. Wiedemann <bwiedemann@suse.de>
>> Signed-off-by: Alexander Graf <agraf@suse.de>
> It needs compatibility properties in HW_COMPAT_2_1. See commit
> 6b11322e0f724eb0649fdc324a44288b783023ad for reference.
Ah, sure, will do.
>
>> ---
>> target-i386/cpu.c | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
>> index 03b33cf..d1b1b8c 100644
>> --- a/target-i386/cpu.c
>> +++ b/target-i386/cpu.c
>> @@ -1234,7 +1234,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
>> CPUID_EXT2_MTRR | CPUID_EXT2_SYSCALL | CPUID_EXT2_APIC |
>> CPUID_EXT2_CX8 | CPUID_EXT2_MCE | CPUID_EXT2_PAE | CPUID_EXT2_MSR |
>> CPUID_EXT2_TSC | CPUID_EXT2_PSE | CPUID_EXT2_DE | CPUID_EXT2_FPU,
>> - .xlevel = 0x80000008,
>> + .xlevel = 0x80000018,
> Why did you choose 0x80000018? The highest 0x80000000 leaf we implement
> today is 0x8000000A. SVM info is at 0x8000000A.
Because it's what real hardware exposes ;).
Alex
>
>
>> .model_id = "AMD Opteron 240 (Gen 1 Class Opteron)",
>> },
>> {
>> @@ -1262,7 +1262,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
>> CPUID_EXT2_DE | CPUID_EXT2_FPU,
>> .features[FEAT_8000_0001_ECX] =
>> CPUID_EXT3_SVM | CPUID_EXT3_LAHF_LM,
>> - .xlevel = 0x80000008,
>> + .xlevel = 0x80000018,
>> .model_id = "AMD Opteron 22xx (Gen 2 Class Opteron)",
>> },
>> {
>> @@ -1292,7 +1292,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
>> .features[FEAT_8000_0001_ECX] =
>> CPUID_EXT3_MISALIGNSSE | CPUID_EXT3_SSE4A |
>> CPUID_EXT3_ABM | CPUID_EXT3_SVM | CPUID_EXT3_LAHF_LM,
>> - .xlevel = 0x80000008,
>> + .xlevel = 0x8000001A,
>> .model_id = "AMD Opteron 23xx (Gen 3 Class Opteron)",
>> },
>> {
>> --
>> 1.7.12.4
>>
>>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Qemu-devel] [PATCH] x86: Fix Opteron xlevels
2015-04-21 14:15 ` Alexander Graf
@ 2015-04-22 14:23 ` Eduardo Habkost
2015-04-22 14:59 ` Alexander Graf
0 siblings, 1 reply; 6+ messages in thread
From: Eduardo Habkost @ 2015-04-22 14:23 UTC (permalink / raw)
To: Alexander Graf; +Cc: bwiedemann, qemu-devel, afaerber
On Tue, Apr 21, 2015 at 04:15:14PM +0200, Alexander Graf wrote:
> On 04/21/2015 04:16 PM, Eduardo Habkost wrote:
> >On Tue, Apr 21, 2015 at 04:04:21PM +0200, Alexander Graf wrote:
> >>The AMD Opteron family has different xlevel levels depending on the
> >>generation. I looked up Gen1, Gen2 and Gen3 hardware and adapted the
> >>levels according to real silicon.
> >>
> >>The reason this came up is that there is a sanity check in KVM making
> >>sure that SVM is only used when xlevel is high enough. Using real
> >>hardware levels, they now are.
> >>
> >>Reported-by: Bernhard M. Wiedemann <bwiedemann@suse.de>
> >>Signed-off-by: Alexander Graf <agraf@suse.de>
> >It needs compatibility properties in HW_COMPAT_2_1. See commit
> >6b11322e0f724eb0649fdc324a44288b783023ad for reference.
>
> Ah, sure, will do.
>
> >
> >>---
> >> target-i386/cpu.c | 6 +++---
> >> 1 file changed, 3 insertions(+), 3 deletions(-)
> >>
> >>diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> >>index 03b33cf..d1b1b8c 100644
> >>--- a/target-i386/cpu.c
> >>+++ b/target-i386/cpu.c
> >>@@ -1234,7 +1234,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
> >> CPUID_EXT2_MTRR | CPUID_EXT2_SYSCALL | CPUID_EXT2_APIC |
> >> CPUID_EXT2_CX8 | CPUID_EXT2_MCE | CPUID_EXT2_PAE | CPUID_EXT2_MSR |
> >> CPUID_EXT2_TSC | CPUID_EXT2_PSE | CPUID_EXT2_DE | CPUID_EXT2_FPU,
> >>- .xlevel = 0x80000008,
> >>+ .xlevel = 0x80000018,
> >Why did you choose 0x80000018? The highest 0x80000000 leaf we implement
> >today is 0x8000000A. SVM info is at 0x8000000A.
>
> Because it's what real hardware exposes ;).
Real hardware exposes 0x80000018 because it does return useful
information in some of the 0x8000000B-0x80000018 leaves.
We don't return anything useful in CPUID leaves above 0x8000000a[1], so
what exactly are you trying to do by reporting leaves
0x8000000B-0x80000018 as available?
[1] I don't even know what's present in those leaves. I will check the
Intel and AMD docs right now.
--
Eduardo
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Qemu-devel] [PATCH] x86: Fix Opteron xlevels
2015-04-22 14:23 ` Eduardo Habkost
@ 2015-04-22 14:59 ` Alexander Graf
2015-04-22 16:39 ` Eduardo Habkost
0 siblings, 1 reply; 6+ messages in thread
From: Alexander Graf @ 2015-04-22 14:59 UTC (permalink / raw)
To: Eduardo Habkost; +Cc: bwiedemann, qemu-devel, afaerber
On 04/22/2015 04:23 PM, Eduardo Habkost wrote:
> On Tue, Apr 21, 2015 at 04:15:14PM +0200, Alexander Graf wrote:
>> On 04/21/2015 04:16 PM, Eduardo Habkost wrote:
>>> On Tue, Apr 21, 2015 at 04:04:21PM +0200, Alexander Graf wrote:
>>>> The AMD Opteron family has different xlevel levels depending on the
>>>> generation. I looked up Gen1, Gen2 and Gen3 hardware and adapted the
>>>> levels according to real silicon.
>>>>
>>>> The reason this came up is that there is a sanity check in KVM making
>>>> sure that SVM is only used when xlevel is high enough. Using real
>>>> hardware levels, they now are.
>>>>
>>>> Reported-by: Bernhard M. Wiedemann <bwiedemann@suse.de>
>>>> Signed-off-by: Alexander Graf <agraf@suse.de>
>>> It needs compatibility properties in HW_COMPAT_2_1. See commit
>>> 6b11322e0f724eb0649fdc324a44288b783023ad for reference.
>> Ah, sure, will do.
>>
>>>> ---
>>>> target-i386/cpu.c | 6 +++---
>>>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>>>
>>>> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
>>>> index 03b33cf..d1b1b8c 100644
>>>> --- a/target-i386/cpu.c
>>>> +++ b/target-i386/cpu.c
>>>> @@ -1234,7 +1234,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
>>>> CPUID_EXT2_MTRR | CPUID_EXT2_SYSCALL | CPUID_EXT2_APIC |
>>>> CPUID_EXT2_CX8 | CPUID_EXT2_MCE | CPUID_EXT2_PAE | CPUID_EXT2_MSR |
>>>> CPUID_EXT2_TSC | CPUID_EXT2_PSE | CPUID_EXT2_DE | CPUID_EXT2_FPU,
>>>> - .xlevel = 0x80000008,
>>>> + .xlevel = 0x80000018,
>>> Why did you choose 0x80000018? The highest 0x80000000 leaf we implement
>>> today is 0x8000000A. SVM info is at 0x8000000A.
>> Because it's what real hardware exposes ;).
> Real hardware exposes 0x80000018 because it does return useful
> information in some of the 0x8000000B-0x80000018 leaves.
>
> We don't return anything useful in CPUID leaves above 0x8000000a[1], so
> what exactly are you trying to do by reporting leaves
> 0x8000000B-0x80000018 as available?
Considering that the G4 Opteron exposes 0x1a I'd say we have that
problem already. So if we want to fix the unpopulated leafs, we need to
do that regardless.
Alex
>
> [1] I don't even know what's present in those leaves. I will check the
> Intel and AMD docs right now.
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [Qemu-devel] [PATCH] x86: Fix Opteron xlevels
2015-04-22 14:59 ` Alexander Graf
@ 2015-04-22 16:39 ` Eduardo Habkost
0 siblings, 0 replies; 6+ messages in thread
From: Eduardo Habkost @ 2015-04-22 16:39 UTC (permalink / raw)
To: Alexander Graf; +Cc: bwiedemann, qemu-devel, afaerber
On Wed, Apr 22, 2015 at 04:59:49PM +0200, Alexander Graf wrote:
> On 04/22/2015 04:23 PM, Eduardo Habkost wrote:
> >On Tue, Apr 21, 2015 at 04:15:14PM +0200, Alexander Graf wrote:
> >>On 04/21/2015 04:16 PM, Eduardo Habkost wrote:
> >>>On Tue, Apr 21, 2015 at 04:04:21PM +0200, Alexander Graf wrote:
> >>>>The AMD Opteron family has different xlevel levels depending on the
> >>>>generation. I looked up Gen1, Gen2 and Gen3 hardware and adapted the
> >>>>levels according to real silicon.
> >>>>
> >>>>The reason this came up is that there is a sanity check in KVM making
> >>>>sure that SVM is only used when xlevel is high enough. Using real
> >>>>hardware levels, they now are.
> >>>>
> >>>>Reported-by: Bernhard M. Wiedemann <bwiedemann@suse.de>
> >>>>Signed-off-by: Alexander Graf <agraf@suse.de>
> >>>It needs compatibility properties in HW_COMPAT_2_1. See commit
> >>>6b11322e0f724eb0649fdc324a44288b783023ad for reference.
> >>Ah, sure, will do.
> >>
> >>>>---
> >>>> target-i386/cpu.c | 6 +++---
> >>>> 1 file changed, 3 insertions(+), 3 deletions(-)
> >>>>
> >>>>diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> >>>>index 03b33cf..d1b1b8c 100644
> >>>>--- a/target-i386/cpu.c
> >>>>+++ b/target-i386/cpu.c
> >>>>@@ -1234,7 +1234,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
> >>>> CPUID_EXT2_MTRR | CPUID_EXT2_SYSCALL | CPUID_EXT2_APIC |
> >>>> CPUID_EXT2_CX8 | CPUID_EXT2_MCE | CPUID_EXT2_PAE | CPUID_EXT2_MSR |
> >>>> CPUID_EXT2_TSC | CPUID_EXT2_PSE | CPUID_EXT2_DE | CPUID_EXT2_FPU,
> >>>>- .xlevel = 0x80000008,
> >>>>+ .xlevel = 0x80000018,
> >>>Why did you choose 0x80000018? The highest 0x80000000 leaf we implement
> >>>today is 0x8000000A. SVM info is at 0x8000000A.
> >>Because it's what real hardware exposes ;).
> >Real hardware exposes 0x80000018 because it does return useful
> >information in some of the 0x8000000B-0x80000018 leaves.
> >
> >We don't return anything useful in CPUID leaves above 0x8000000a[1], so
> >what exactly are you trying to do by reporting leaves
> >0x8000000B-0x80000018 as available?
>
> Considering that the G4 Opteron exposes 0x1a I'd say we have that problem
> already. So if we want to fix the unpopulated leafs, we need to do that
> regardless.
True, and the issue isn't serious (see my reply to v2, most bits are
reserved). But I would argue that making xlevel=0x1a on Opteron_G4
without an analysis of each of the new CPUID leaves was a mistake. All
we get by increasing xlevel too much is is the false impression that we
are getting closer to real hardware, when we are actually exposing more
information that doesn't match real hardware to the guest.
--
Eduardo
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-04-22 16:39 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-21 14:04 [Qemu-devel] [PATCH] x86: Fix Opteron xlevels Alexander Graf
2015-04-21 14:16 ` Eduardo Habkost
2015-04-21 14:15 ` Alexander Graf
2015-04-22 14:23 ` Eduardo Habkost
2015-04-22 14:59 ` Alexander Graf
2015-04-22 16:39 ` Eduardo Habkost
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).