linux-kernel - [RFC -tip] x86,apic: set cpu_has_apic manually for discrete apic

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20090816195736.GB4788@lenovo>
Date:	Sun, 16 Aug 2009 23:57:36 +0400
From:	Cyrill Gorcunov <gorcunov@...il.com>
To:	Ingo Molnar <mingo@...e.hu>
Cc:	"H. Peter Anvin" <hpa@...or.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	"Maciej W. Rozycki" <macro@...ux-mips.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: [RFC -tip] x86,apic: set cpu_has_apic manually for discrete apic

In case of discrete apic (due to hardware or firmware reason)
cpuid may not retrieve cpu_has_apic bit. But we rely on this
bit in SMP compiled kernels. So instead of long condition checking
we set cpu_has_apic bit manually and check it only in further
code flow.

Note that the case is applied to x86-32 pretty old machines only.
Also verify_local_APIC is the only place where such a trick
could be done. We can't do it earlier since there could be
a BIOS bug and other checks which we are to pass to ensure
that APIC is functional.

The patch also make lapic_shutdown, disable_IO_APIC callable
which should take place even on discrete apics.

As a side effect it makes lapic sysfs device available
for such machines so lets check for apic being integrated
since we don't support suspend/resume for disrete apics.

Signed-off-by: Cyrill Gorcunov <gorcunov@...nvz.org>
---

Please review carefully. I've tested it with manually disabled
cpu_has_apic and apic was set to "discrete" state as well
under qemu.

Actually this "two-line" patch could be more dangerous
then it looks like and need more wide range _long_ testing.

Or, perhaps, we could just fix lapic_shutdown and disable_IO_APIC
instead since old machines already that rare to find out.

Anyway if someone has 486DX machine please give the patch a try.

 arch/x86/kernel/apic/apic.c |   21 ++++++++++++++++++++-
 1 file changed, 20 insertions(+), 1 deletion(-)

Index: linux-2.6.git/arch/x86/kernel/apic/apic.c
=====================================================================
--- linux-2.6.git.orig/arch/x86/kernel/apic/apic.c
+++ linux-2.6.git/arch/x86/kernel/apic/apic.c
@@ -1004,6 +1004,25 @@ int __init verify_local_APIC(void)
 	unsigned int reg0, reg1;

 	/*
+	 * For discrete APIC cpu_has_apic bit may not
+	 * be retrieved via cpuid instruction. Same time
+	 * we rely on this bit (mostly for SMP compiled
+	 * kernel) and instead of checking the long condition
+	 * (as shown below) all the time we may check for
+	 * plain cpu_has_apic further in code.
+	 *
+	 * At this moment all the checks are passed and we've
+	 * already complained if there was a BIOS bug.
+	 * This place is safe for such a trick. --cvg
+	 */
+#ifdef CONFIG_X86_32
+	if (smp_found_config && !cpu_has_apic && !disable_apic) {
+		set_cpu_cap(&boot_cpu_data, X86_FEATURE_APIC);
+		pr_info("APIC: cpu_has_apic being set manually\n");
+	}
+#endif
+
+	/*
 	 * The version register is read-only in a real APIC.
 	 */
 	reg0 = apic_read(APIC_LVR);
@@ -2145,7 +2164,7 @@ static int __init init_lapic_sysfs(void)
 {
 	int error;

-	if (!cpu_has_apic)
+	if (!cpu_has_apic || !lapic_is_integrated())
 		return 0;
 	/* XXX: remove suspend/resume procs if !apic_pm_state.active? */

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/