[linux] 01/01: [x86] boot/smp: Don't try to poke disabled/non-existent APIC

debian-kernel at lists.debian.org debian-kernel at lists.debian.org
Thu Oct 27 04:36:23 UTC 2016


This is an automated email from the git hooks/post-receive script.

carnil pushed a commit to branch master
in repository linux.

commit f86bc880754e89a71cdda2f6d2b58f2f268efdcf
Author: Salvatore Bonaccorso <carnil at debian.org>
Date:   Wed Oct 26 08:14:56 2016 +0200

    [x86] boot/smp: Don't try to poke disabled/non-existent APIC
    
    Closes: #841850
---
 debian/changelog                                   |  5 ++
 ...p-Don-t-try-to-poke-disabled-non-existent.patch | 74 ++++++++++++++++++++++
 debian/patches/series                              |  1 +
 3 files changed, 80 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 7242a7f..c5aef53 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,10 +1,15 @@
 linux (4.8.4-1~exp2) UNRELEASED; urgency=medium
 
+  [ Ben Hutchings ]
   * cpufreq: Re-enable CPU_FREQ_STAT, which can no longer be modular
   * [armhf] dts: imx53: add support for USB armory board (Closes: #840137)
   * kconfig: Renumber SYMBOL_NEW, fixing regression of allnoconfig
     (Closes: #841357)
 
+  [ Salvatore Bonaccorso ]
+  * [x86] boot/smp: Don't try to poke disabled/non-existent APIC
+    (Closes: #841850)
+
  -- Ben Hutchings <ben at decadent.org.uk>  Mon, 24 Oct 2016 02:25:26 +0100
 
 linux (4.8.4-1~exp1) experimental; urgency=medium
diff --git a/debian/patches/bugfix/x86/x86-boot-smp-Don-t-try-to-poke-disabled-non-existent.patch b/debian/patches/bugfix/x86/x86-boot-smp-Don-t-try-to-poke-disabled-non-existent.patch
new file mode 100644
index 0000000..141eeca
--- /dev/null
+++ b/debian/patches/bugfix/x86/x86-boot-smp-Don-t-try-to-poke-disabled-non-existent.patch
@@ -0,0 +1,74 @@
+From: =?UTF-8?q?Ville=20Syrj=C3=A4l=C3=A4?= <ville.syrjala at linux.intel.com>
+Date: Sat, 22 Oct 2016 05:18:04 +0300
+Subject: x86/boot/smp: Don't try to poke disabled/non-existent APIC
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+Origin: https://git.kernel.org/linus/ff8560512b8d4b7ca3ef4fd69166634ac30b2525
+Bug-Debian: https://bugs.debian.org/841850
+
+Apparently trying to poke a disabled or non-existent APIC
+leads to a box that doesn't even boot. Let's not do that.
+
+No real clue if this is the right fix, but at least my
+P3 machine boots again.
+
+Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
+Cc: Andy Lutomirski <luto at kernel.org>
+Cc: Borislav Petkov <bp at alien8.de>
+Cc: Borislav Petkov <bp at suse.de>
+Cc: Brian Gerst <brgerst at gmail.com>
+Cc: Denys Vlasenko <dvlasenk at redhat.com>
+Cc: Eric Biederman <ebiederm at xmission.com>
+Cc: H. Peter Anvin <hpa at zytor.com>
+Cc: Jiri Olsa <jolsa at redhat.com>
+Cc: Josh Poimboeuf <jpoimboe at redhat.com>
+Cc: Juergen Gross <jgross at suse.com>
+Cc: Len Brown <len.brown at intel.com>
+Cc: Linus Torvalds <torvalds at linux-foundation.org>
+Cc: Peter Zijlstra <peterz at infradead.org>
+Cc: Prarit Bhargava <prarit at redhat.com>
+Cc: Thomas Gleixner <tglx at linutronix.de>
+Cc: Yinghai Lu <yinghai at kernel.org>
+Cc: dyoung at redhat.com
+Cc: kexec at lists.infradead.org
+Cc: stable at vger.kernel.org
+Fixes: 2a51fe083eba ("arch/x86: Handle non enumerated CPU after physical hotplug")
+Link: http://lkml.kernel.org/r/1477102684-5092-1-git-send-email-ville.syrjala@linux.intel.com
+Signed-off-by: Ingo Molnar <mingo at kernel.org>
+---
+ arch/x86/kernel/smpboot.c | 16 +++++++++-------
+ 1 file changed, 9 insertions(+), 7 deletions(-)
+
+diff --git a/arch/x86/kernel/smpboot.c b/arch/x86/kernel/smpboot.c
+index 951f093..42f5eb7 100644
+--- a/arch/x86/kernel/smpboot.c
++++ b/arch/x86/kernel/smpboot.c
+@@ -1409,15 +1409,17 @@ __init void prefill_possible_map(void)
+ 
+ 	/* No boot processor was found in mptable or ACPI MADT */
+ 	if (!num_processors) {
+-		int apicid = boot_cpu_physical_apicid;
+-		int cpu = hard_smp_processor_id();
++		if (boot_cpu_has(X86_FEATURE_APIC)) {
++			int apicid = boot_cpu_physical_apicid;
++			int cpu = hard_smp_processor_id();
+ 
+-		pr_warn("Boot CPU (id %d) not listed by BIOS\n", cpu);
++			pr_warn("Boot CPU (id %d) not listed by BIOS\n", cpu);
+ 
+-		/* Make sure boot cpu is enumerated */
+-		if (apic->cpu_present_to_apicid(0) == BAD_APICID &&
+-		    apic->apic_id_valid(apicid))
+-			generic_processor_info(apicid, boot_cpu_apic_version);
++			/* Make sure boot cpu is enumerated */
++			if (apic->cpu_present_to_apicid(0) == BAD_APICID &&
++			    apic->apic_id_valid(apicid))
++				generic_processor_info(apicid, boot_cpu_apic_version);
++		}
+ 
+ 		if (!num_processors)
+ 			num_processors = 1;
+-- 
+2.10.1
+
diff --git a/debian/patches/series b/debian/patches/series
index 791f1c9..a52dfd7 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -48,6 +48,7 @@ debian/fanotify-taint-on-use-of-fanotify_access_permissions.patch
 # Arch bug fixes
 bugfix/powerpc/powerpc-xmon-don-t-use-ld-on-32-bit.patch
 bugfix/arm64/arm64-mm-limit-task_size_64-for-compatibility.patch
+bugfix/x86/x86-boot-smp-Don-t-try-to-poke-disabled-non-existent.patch
 
 # Arch features
 features/mips/MIPS-increase-MAX-PHYSMEM-BITS-on-Loongson-3-only.patch

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/kernel/linux.git



More information about the Kernel-svn-changes mailing list