[xen stable-4.17] x86/x2apic: remove usage of ACPI_FADT_APIC_CLUSTER

[patchbot@xxxxxxx]

commit d10db37b785ae61e2e2c0326b12823e66afb40fc
Author:     Roger Pau Monné <roger.pau@xxxxxxxxxx>
AuthorDate: Tue Nov 14 14:01:07 2023 +0100
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Tue Nov 14 14:01:07 2023 +0100

    x86/x2apic: remove usage of ACPI_FADT_APIC_CLUSTER
    
    The ACPI FADT APIC_CLUSTER flag mandates that when the interrupt delivery is
    Logical mode APIC must be configured for Cluster destination model.  
However in
    apic_x2apic_probe() such flag is incorrectly used to gate whether Physical 
mode
    can be used.
    
    Since Xen when in x2APIC mode only uses Logical mode together with Cluster
    model completely remove checking for ACPI_FADT_APIC_CLUSTER, as Xen always
    fulfills the requirement signaled by the flag.
    
    Fixes: eb40ae41b658 ('x86/Kconfig: add option for default x2APIC 
destination mode')
    Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
    Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
    master commit: 26a449ce32cef33f2cb50602be19fcc0c4223ba9
    master date: 2023-11-02 10:50:26 +0100
---
 xen/arch/x86/genapic/x2apic.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/xen/arch/x86/genapic/x2apic.c b/xen/arch/x86/genapic/x2apic.c
index d512c50fc5..15a62f874b 100644
--- a/xen/arch/x86/genapic/x2apic.c
+++ b/xen/arch/x86/genapic/x2apic.c
@@ -242,8 +242,7 @@ const struct genapic *__init apic_x2apic_probe(void)
          */
         x2apic_phys = iommu_intremap != iommu_intremap_full ||
                       (acpi_gbl_FADT.flags & ACPI_FADT_APIC_PHYSICAL) ||
-                      (IS_ENABLED(CONFIG_X2APIC_PHYSICAL) &&
-                       !(acpi_gbl_FADT.flags & ACPI_FADT_APIC_CLUSTER));
+                      IS_ENABLED(CONFIG_X2APIC_PHYSICAL);
     }
     else if ( !x2apic_phys )
         switch ( iommu_intremap )
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.17