Skip to contentSkip to conversation

x86/devicetree: Add support for boot time interrupt mode config

PATCH RESEND Superseded Rahul Tanwar 1 participant 2 messages Oct 3, 2022 – Oct 11, 2022 x86

v1 of 5 Superseded Latest is v5 →

lore
Share & tools
lore
#0 lore raw

PATCH RESEND 1/1 x86/devicetree: Add support for boot time interrupt mode config

Presently, init/boot time interrupt delivery mode is enumerated only for ACPI enabled systems by parsing MADT table or for older systems by parsing MP table. But for OF based x86 systems, it is assumed & fixed to legacy PIC mode.

Add support for configuration of init time interrupt delivery mode for x86 OF based systems by introducing a new boolean property 'intel,no-imcr' for interrupt-controller node of local APIC. This property emulates IMCRP Bit 7 of MP feature info byte 2 of MP floating pointer structure.

Defaults to legacy PIC mode if absent. Configures it to virtual wire compatibility mode if present.

---
 arch/x86/kernel/devicetree.c | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)
arch/x86/kernel/devicetree.c +9 -1
index 5cd51f25f446..de14015317f8 100644
@@ -167,7 +167,15 @@ static void __init dtb_lapic_setup(void)
 			return;
 	}
 	smp_found_config = 1;
-	pic_mode = 1;
+	if (of_property_read_bool(dn, "intel,no-imcr")) {
+		pr_info("    Virtual Wire compatibility mode.\n");
+		pic_mode = 0;
+	}
+	else {
+		pr_info("    IMCR and PIC compatibility mode.\n");
+		pic_mode = 1;
+	}
+
 	register_lapic_address(lapic_addr);
 }
 
Signature · 1 line
2.17.1
#1 lore raw
Quoted 5 lines · Rahul Tanwar

Presently, init/boot time interrupt delivery mode is enumerated only for ACPI enabled systems by parsing MADT table or for older systems by parsing MP table. But for OF based x86 systems, it is assumed & fixed to legacy PIC mode.

Quoted 9 lines

Add support for configuration of init time interrupt delivery mode for x86 OF based systems by introducing a new boolean property 'intel,no-imcr' for interrupt-controller node of local APIC. This property emulates IMCRP Bit 7 of MP feature info byte 2 of MP floating pointer structure.

Defaults to legacy PIC mode if absent. Configures it to virtual wire compatibility mode if present.

Any comments or concerns for this patch? IMHO, this configurability of pic_mode was missing for OF systems & it should benefit other x86 based OF platforms as well. Please let me know if i am missing anything about it or if you have other concerns.

Thanks,
Rahul

Quoted 26 lines
---
  arch/x86/kernel/devicetree.c | 10 +++++++++-
  1 file changed, 9 insertions(+), 1 deletion(-)
arch/x86/kernel/devicetree.c +9 -1
index 5cd51f25f446..de14015317f8 100644
@@ -167,7 +167,15 @@ static void __init dtb_lapic_setup(void)
  			return;
  	}
  	smp_found_config = 1;
-	pic_mode = 1;
+	if (of_property_read_bool(dn, "intel,no-imcr")) {
+		pr_info("    Virtual Wire compatibility mode.\n");
+		pic_mode = 0;
+	}
+	else {
+		pr_info("    IMCR and PIC compatibility mode.\n");
+		pic_mode = 1;
+	}
+
  	register_lapic_address(lapic_addr);
  }