Re: (amd64) panic: mpacpi_findintr_linkdev: irq mismatch

["D'Arcy J.M. Cain" <darcy%NetBSD.org@localhost>]

On Fri, 28 Mar 2008 17:10:19 -0700
"Robert P. Thille" <list-netbsd-current-users%rangat.org@localhost> wrote:
> Trying D'Arcy's change lets me get past that panic, but it just results 
> in a hang later on when viaide reports lost interrupts.

By "D'Arcy's change" do you mean this?

Index: sys/arch/x86/x86/mpacpi.c
===================================================================
RCS file: /cvsroot/src/sys/arch/x86/x86/mpacpi.c,v
retrieving revision 1.56
diff -u -p -u -r1.56 mpacpi.c
--- sys/arch/x86/x86/mpacpi.c   12 Dec 2007 23:33:22 -0000      1.56
+++ sys/arch/x86/x86/mpacpi.c   30 Mar 2008 14:22:57 -0000
@@ -1139,12 +1139,10 @@ mpacpi_findintr_linkdev(struct mp_intr_m
        irq = acpi_pci_link_route_interrupt(mip->linkdev, mip->sourceindex, 
&line, &pol, &trig);
        if (mp_verbose)
-           printf("linkdev %s returned ACPI global int %d\n",
-               acpi_pci_link_name(mip->linkdev), line);
+           printf("linkdev %s returned ACPI global irq %d, line %d\n",
+               acpi_pci_link_name(mip->linkdev), irq, line);
        if (irq == X86_PCI_INTERRUPT_LINE_NO_CONNECTION)
            return ENOENT;
-       if (irq != line)
-           panic("mpacpi_findintr_linkdev: irq mismatch");
 
        /*
         * Convert ACPICA values to MPS values

The second was jmcneil's suggestion to remove the actual panic call but
is not a final fix.  However, that first part is mine and does seem
like a reasonable change regardless.  Does anyone have a problem if I
commit that part?

-- 
D'Arcy J.M. Cain <darcy%NetBSD.org@localhost>
http://www.NetBSD.org/