lkml.org 
[lkml]   [2016]   [Jul]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 3/3] x86/apic: Improved the setting of interrupt mode for bsp
Date
If we specify the 'notsc' parameter for the dump-capture kernel,
and then trigger a crash(panic) by using "ALT-SysRq-c" or
"echo c > /proc/sysrq-trigger", the dump-capture kernel will
hang in calibrate_delay_converge() and wait for jiffies changes.
serial log as follows:

tsc: Fast TSC calibration using PIT
tsc: Detected 2099.947 MHz processor
Calibrating delay loop...

The reason for jiffies not changes is there's no timer interrupt
passed to dump-capture kernel.

In fact, once kernel panic occurs, the local APIC is disabled
by lapic_shutdown() in reboot path.
generly speaking, local APIC state can be initialized by BIOS
after Power-Up or Reset, which doesn't apply to kdump case.
so the kernel has to be responsible for initialize the interrupt
mode properly according the latest status of APIC in bootup path.

An MP operating system is booted under either PIC mode or
virtual wire mode. Later, the operating system switches to
symmetric I/O mode as it enters multiprocessor mode.
Two kinds of virtual wire mode are defined in Intel MP spec:
virtual wire mode via local APIC or via I/O APIC.

Now we determine the mode of APIC only through a SMP BIOS(MP table).
That's not enough.
It's better to do further check if APIC works with effective mode,
and do some porper setting.

Signed-off-by: Cao jin <caoj.fnst@cn.fujitsu.com>
Signed-off-by: Wei Jiangang <weijg.fnst@cn.fujitsu.com>
---
arch/x86/include/asm/io_apic.h | 5 ++++
arch/x86/kernel/apic/apic.c | 55 +++++++++++++++++++++++++++++++++++++++++-
arch/x86/kernel/apic/io_apic.c | 28 +++++++++++++++++++++
3 files changed, 87 insertions(+), 1 deletion(-)

diff --git a/arch/x86/include/asm/io_apic.h b/arch/x86/include/asm/io_apic.h
index 6cbf2cfb3f8a..6550bd43fa39 100644
--- a/arch/x86/include/asm/io_apic.h
+++ b/arch/x86/include/asm/io_apic.h
@@ -190,6 +190,7 @@ static inline unsigned int io_apic_read(unsigned int apic, unsigned int reg)
}

extern void setup_IO_APIC(void);
+extern bool virtual_wire_via_ioapic(void);
extern void enable_IO_APIC(void);
extern void disable_IO_APIC(void);
extern void setup_ioapic_dest(void);
@@ -231,6 +232,10 @@ static inline void io_apic_init_mappings(void) { }
#define native_disable_io_apic NULL

static inline void setup_IO_APIC(void) { }
+static inline bool virtual_wire_via_ioapic(void)
+{
+ return true;
+}
static inline void enable_IO_APIC(void) { }
static inline void setup_ioapic_dest(void) { }

diff --git a/arch/x86/kernel/apic/apic.c b/arch/x86/kernel/apic/apic.c
index 8e25b9b2d351..04358e0cf1e2 100644
--- a/arch/x86/kernel/apic/apic.c
+++ b/arch/x86/kernel/apic/apic.c
@@ -1124,6 +1124,53 @@ void __init sync_Arb_IDs(void)
}

/*
+ * Return false means the virtual wire mode through-local-apic is inactive
+ */
+static bool virtual_wire_via_lapic(void)
+{
+ unsigned int value;
+
+ /* Check the APIC global enable/disable flag firstly */
+ if (boot_cpu_data.x86 >= 6) {
+ u32 h, l;
+
+ rdmsr(MSR_IA32_APICBASE, l, h);
+ /*
+ * If local APIC is disabled by BIOS
+ * do nothing, but return true
+ */
+ if (!(l & MSR_IA32_APICBASE_ENABLE))
+ return true;
+ }
+
+ /* Check the software enable/disable flag */
+ value = apic_read(APIC_SPIV);
+ if (!(value & APIC_SPIV_APIC_ENABLED))
+ return false;
+
+ /*
+ * Virtual wire mode via local APIC requests
+ * APIC to enable the LINT0 for edge-trggered ExtINT delivery mode
+ * and LINT1 for level-triggered NMI delivery mode
+ */
+ value = apic_read(APIC_LVT0);
+ if (GET_APIC_DELIVERY_MODE(value) != APIC_MODE_EXTINT)
+ return false;
+
+ value = apic_read(APIC_LVT1);
+ if (GET_APIC_DELIVERY_MODE(value) != APIC_MODE_NMI)
+ return false;
+
+ return true;
+}
+
+static bool check_virtual_wire_mode(void)
+{
+ /* Neither of virtual wire mode is active, return false */
+ return virtual_wire_via_lapic() || virtual_wire_via_ioapic();
+}
+
+/*
* An initial setup of the virtual wire mode.
*/
void __init init_bsp_APIC(void)
@@ -1133,8 +1180,14 @@ void __init init_bsp_APIC(void)
/*
* Don't do the setup now if we have a SMP BIOS as the
* through-I/O-APIC virtual wire mode might be active.
+ *
+ * It's better to do further check if either through-I/O-APIC
+ * or through-local-APIC is active.
+ * the worst case is that both of them are inactive,
+ * If so, We need to enable the virtual wire mode via through-local-APIC
*/
- if (smp_found_config || !boot_cpu_has(X86_FEATURE_APIC))
+ if ((smp_found_config && check_virtual_wire_mode()) ||
+ !boot_cpu_has(X86_FEATURE_APIC))
return;

/*
diff --git a/arch/x86/kernel/apic/io_apic.c b/arch/x86/kernel/apic/io_apic.c
index 446702ed99dc..5a32c26938ac 100644
--- a/arch/x86/kernel/apic/io_apic.c
+++ b/arch/x86/kernel/apic/io_apic.c
@@ -1379,6 +1379,34 @@ void __init print_IO_APICs(void)
/* Where if anywhere is the i8259 connect in external int mode */
static struct { int pin, apic; } ioapic_i8259 = { -1, -1 };

+/*
+ * Return false means the virtual wire mode via I/O APIC is inactive
+ */
+bool virtual_wire_via_ioapic(void)
+{
+ int apic, pin;
+
+ for_each_ioapic_pin(apic, pin) {
+ /* See if any of the pins is in ExtINT mode */
+ struct IO_APIC_route_entry entry = ioapic_read_entry(apic, pin);
+
+ /*
+ * If the interrupt line is enabled and in ExtInt mode
+ * I have found the pin where the i8259 is connected.
+ */
+ if ((entry.mask == 0) && (entry.delivery_mode == dest_ExtINT))
+ return true;
+ }
+
+ /*
+ * Virtual wire mode via I/O APIC requests
+ * I/O APIC be connected to i8259 in chapter 3.6.2.2 of the MP v1.4 spec
+ * If no pin in ExtInt mode,
+ * the through-I/O-APIC virtual wire mode can be regarded inactive.
+ */
+ return false;
+}
+
void __init enable_IO_APIC(void)
{
int i8259_apic, i8259_pin;
--
1.9.3


\
 
 \ /
  Last update: 2016-07-22 10:41    [W:0.085 / U:2.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site