Re: [GIT PULL] x86/apic updates for v3.20

From: Joerg Roedel
Date: Fri Feb 13 2015 - 10:47:24 EST

On Thu, Feb 12, 2015 at 06:08:45PM -0800, Linus Torvalds wrote:
> Any ideas? The Pixel Chromebook does have an odd and often broken
> BIOS/firmware thing, but it *used* to suspend and resume very reliably
> once we got around its insane TPM thing.

Hmm, looking at the patch and the surrounding code my guess is that
x2apic gets enabled now with 5fcee53ce705 on your Chromebook Pixel at
boot, which causes the suspend issue later for other reasons.

So the condition should rather look like this to keep x2apic disabled
like before:

diff --git a/arch/x86/kernel/apic/apic.c b/arch/x86/kernel/apic/apic.c
index b665d24..770a0bb 100644
--- a/arch/x86/kernel/apic/apic.c
+++ b/arch/x86/kernel/apic/apic.c
@@ -1580,8 +1580,8 @@ static __init void try_to_enable_x2apic(int remap_mode)
* under KVM
if (max_physical_apicid > 255 ||
- !hypervisor_x2apic_available())) {
+ hypervisor_x2apic_available())) {
pr_info("x2apic: IRQ remapping doesn't support X2APIC mode\n");

Another solution would be to just revert 5fcee53ce705. This code is not
in any fast-path, so we can live well without that optimization.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at