diff mbox series

i386/xen: Don't advertise XENFEAT_supervisor_mode_kernel

Message ID 782b63c1f9c41a6bfa771789cde4b45644b3a239.camel@infradead.org (mailing list archive)
State New, archived
Headers show
Series i386/xen: Don't advertise XENFEAT_supervisor_mode_kernel | expand

Commit Message

David Woodhouse Aug. 8, 2023, 5:08 p.m. UTC
From: David Woodhouse <dwmw@amazon.co.uk>

XENFEAT_supervisor_mode_kernel shouldn't be set for HVM guests. It
confuses lscpu into thinking it's running in PVH mode.

No non-cosmetic effects have been observed so far.

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
---
Only really cosmetic. Don't feel strongly about whether it makes 8.1.

 target/i386/kvm/xen-emu.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Paul Durrant Aug. 14, 2023, 12:33 p.m. UTC | #1
On 08/08/2023 18:08, David Woodhouse wrote:
> From: David Woodhouse <dwmw@amazon.co.uk>
> 
> XENFEAT_supervisor_mode_kernel shouldn't be set for HVM guests. It
> confuses lscpu into thinking it's running in PVH mode.
> 
> No non-cosmetic effects have been observed so far.
> 
> Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
> ---
> Only really cosmetic. Don't feel strongly about whether it makes 8.1.
> 
>   target/i386/kvm/xen-emu.c | 1 -
>   1 file changed, 1 deletion(-)
> 

Reviewed-by: Paul Durrant <paul@xen.org>
diff mbox series

Patch

diff --git a/target/i386/kvm/xen-emu.c b/target/i386/kvm/xen-emu.c
index a8146115f0..b307c75713 100644
--- a/target/i386/kvm/xen-emu.c
+++ b/target/i386/kvm/xen-emu.c
@@ -267,7 +267,6 @@  static bool kvm_xen_hcall_xen_version(struct kvm_xen_exit *exit, X86CPU *cpu,
             fi.submap |= 1 << XENFEAT_writable_page_tables |
                          1 << XENFEAT_writable_descriptor_tables |
                          1 << XENFEAT_auto_translated_physmap |
-                         1 << XENFEAT_supervisor_mode_kernel |
                          1 << XENFEAT_hvm_callback_vector |
                          1 << XENFEAT_hvm_safe_pvclock |
                          1 << XENFEAT_hvm_pirqs;