Bug 17772
Summary: | Unable to locate IOAPIC for GSI * | ||
---|---|---|---|
Product: | Platform Specific/Hardware | Reporter: | Vasyl Demin (vasyl.demin) |
Component: | i386 | Assignee: | platform_i386 |
Status: | RESOLVED PATCH_ALREADY_AVAILABLE | ||
Severity: | normal | CC: | florian, kurup_avinash, maciej.rutecki, mingo, rjw, sedat.dilek, vasyl.demin |
Priority: | P1 | ||
Hardware: | All | ||
OS: | Linux | ||
Kernel Version: | 2.6.35.4 | Subsystem: | |
Regression: | Yes | Bisected commit-id: | |
Bug Depends on: | |||
Bug Blocks: | 16055 | ||
Attachments: | Bootlog with "debug apic=debug" |
Description
Vasyl Demin
2010-09-04 21:28:37 UTC
Created attachment 29822 [details]
Bootlog with "debug apic=debug"
(In reply to comment #0) > Notebook: HP Compaq nx9020 > OS: Arch Linux i686 > > After kernel upgrade up to 2.6.35 I see errors during boot: > ERROR: Unable to locate IOAPIC for GSI 8 > ERROR: Unable to locate IOAPIC for GSI 13 > ERROR: Unable to locate IOAPIC for GSI 1 > ERROR: Unable to locate IOAPIC for GSI 12 Hi zersaa, I had faced this problem a while ago. This patch provided by Eric biederman solved the problem for me...check it and see if it works... --- diff --git a/arch/x86/kernel/apic/io_apic.c b/arch/x86/kernel/apic/io_apic.c index 33f3563..226a6d1 100644 --- a/arch/x86/kernel/apic/io_apic.c +++ b/arch/x86/kernel/apic/io_apic.c @@ -4066,6 +4066,9 @@ int acpi_get_override_irq(u32 gsi, int *trigger, int *polarity) { int ioapic, pin, idx; + if (acpi_irq_model != ACPI_IRQ_MODEL_IOAPIC) + return -1; + if (skip_ioapic_setup) return -1; The original patch from Eric W. Biedermann is from [1]: "Skip looking for ioapic overrides when ioapics are not present" [1] https://patchwork.kernel.org/patch/104501/ Patch: https://patchwork.kernel.org/patch/104501/ Seems this patch has fallen through the cracks. Switched to Email On Thu, 30 Sep 2010 06:54:15 GMT bugzilla-daemon@bugzilla.kernel.org wrote: > https://bugzilla.kernel.org/show_bug.cgi?id=17772 > > > Florian Mickler <florian@mickler.org> changed: > > What |Removed |Added > ---------------------------------------------------------------------------- > Status|NEW |RESOLVED > CC| |florian@mickler.org, > | |mingo@elte.hu > Resolution| |PATCH_ALREADY_AVAILABLE > > > > > --- Comment #4 from Florian Mickler <florian@mickler.org> 2010-09-30 > 06:54:07 --- > Patch: https://patchwork.kernel.org/patch/104501/ > > Seems this patch has fallen through the cracks. > I know, it's not an severe issue, but: I couldn't find a patch to this effect in current mainline, is this already queued? Or are there issues with the patch? Poked-By: Florian Mickler <florian@mickler.org> Handled-By : Eric W. Biederman <ebiederm@xmission.com> Sorry for long delay.. Patch works for me, thanks! Is this patch still needed on 2.6.38? Thanks Florian for PINGing! Debian has this patch (still) as "bugfix/x86/Skip-looking-for-ioapic-overrides-when-ioapics-are-not-present.patch" for so long time in its patch-series (months, a year?). Here on my system you need to add "lapic" to boot-parameter to suppress: [ 0.008891] no APIC, boot with the "lapic" boot parameter to force-enable it. (See discussion on "[PATCH] x86, perf, nmi: Disable perf if counters are not accessable" patch.) An additional text to the original patch would be helpful to reflect "lapic". Ok, I sent it out to Andrew and cc'd people.. that should normally get it merged or at least we get some comment as to why it is out of tree. Ingo is probably not reading this bug anymore. :( Good. I tried to convince GRML project to include this patch into 2010-12 release (and its kernel). I got a NOPE as they argued it is not in *mainline*. I contra-argued it is in Debian's patch-series (and GRML based on Debian). NOPE. OK, this is a somehow a "cosmetic" fix. But anyway, it will be nice to see it in mainly. BTW, [1] says "New Linux kernel version based on 2.6.36.2" :-). Might be good to CC stable.k.o? [1] http://grml.org/changelogs/README-grml-2010.12/ Tested on 2.6.39.1 The patch is no longer needed. Problem fixed, thanks! |