Commit c8980fcb authored by Roger Pau Monne's avatar Roger Pau Monne Committed by Juergen Gross

xen/x2apic: enable x2apic mode when supported for HVM

There's no point in disabling x2APIC mode when running as a Xen HVM
guest, just enable it when available.

Remove some unneeded wrapping around the detection functions, and
simply provide a xen_x2apic_available helper that's a wrapper around
x2apic_supported.
Signed-off-by: default avatarRoger Pau Monné <roger.pau@citrix.com>
Reviewed-by: default avatarBoris Ostrovsky <boris.ostrovsky@oracle.com>
Link: https://lore.kernel.org/r/20220121090146.13697-1-roger.pau@citrix.comSigned-off-by: default avatarJuergen Gross <jgross@suse.com>
parent e783362e
...@@ -43,20 +43,6 @@ static inline uint32_t xen_cpuid_base(void) ...@@ -43,20 +43,6 @@ static inline uint32_t xen_cpuid_base(void)
return hypervisor_cpuid_base("XenVMMXenVMM", 2); return hypervisor_cpuid_base("XenVMMXenVMM", 2);
} }
#ifdef CONFIG_XEN
extern bool __init xen_hvm_need_lapic(void);
static inline bool __init xen_x2apic_para_available(void)
{
return xen_hvm_need_lapic();
}
#else
static inline bool __init xen_x2apic_para_available(void)
{
return (xen_cpuid_base() != 0);
}
#endif
struct pci_dev; struct pci_dev;
#ifdef CONFIG_XEN_PV_DOM0 #ifdef CONFIG_XEN_PV_DOM0
......
...@@ -9,6 +9,7 @@ ...@@ -9,6 +9,7 @@
#include <xen/events.h> #include <xen/events.h>
#include <xen/interface/memory.h> #include <xen/interface/memory.h>
#include <asm/apic.h>
#include <asm/cpu.h> #include <asm/cpu.h>
#include <asm/smp.h> #include <asm/smp.h>
#include <asm/io_apic.h> #include <asm/io_apic.h>
...@@ -242,15 +243,9 @@ static __init int xen_parse_no_vector_callback(char *arg) ...@@ -242,15 +243,9 @@ static __init int xen_parse_no_vector_callback(char *arg)
} }
early_param("xen_no_vector_callback", xen_parse_no_vector_callback); early_param("xen_no_vector_callback", xen_parse_no_vector_callback);
bool __init xen_hvm_need_lapic(void) static __init bool xen_x2apic_available(void)
{ {
if (xen_pv_domain()) return x2apic_supported();
return false;
if (!xen_hvm_domain())
return false;
if (xen_feature(XENFEAT_hvm_pirqs) && xen_have_vector_callback)
return false;
return true;
} }
static __init void xen_hvm_guest_late_init(void) static __init void xen_hvm_guest_late_init(void)
...@@ -312,7 +307,7 @@ struct hypervisor_x86 x86_hyper_xen_hvm __initdata = { ...@@ -312,7 +307,7 @@ struct hypervisor_x86 x86_hyper_xen_hvm __initdata = {
.detect = xen_platform_hvm, .detect = xen_platform_hvm,
.type = X86_HYPER_XEN_HVM, .type = X86_HYPER_XEN_HVM,
.init.init_platform = xen_hvm_guest_init, .init.init_platform = xen_hvm_guest_init,
.init.x2apic_available = xen_x2apic_para_available, .init.x2apic_available = xen_x2apic_available,
.init.init_mem_mapping = xen_hvm_init_mem_mapping, .init.init_mem_mapping = xen_hvm_init_mem_mapping,
.init.guest_late_init = xen_hvm_guest_late_init, .init.guest_late_init = xen_hvm_guest_late_init,
.runtime.pin_vcpu = xen_pin_vcpu, .runtime.pin_vcpu = xen_pin_vcpu,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment