Re: (amd64) panic: mpacpi_findintr_linkdev: irq mismatch

["Robert P. Thille" <list-netbsd-current-users%rangat.org@localhost>]

D'Arcy J.M. Cain wrote:
> 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?

Yes, that's the change I meant.

With the panic #if 0'd out, I get to where wd0 is accessed:
wd0 at atabus1 drive 0: <ST3400833NS>
viaide0:1:0: lost interrupt
    type: ata tc_bcount: 512 tc_skip: 0
wd0: quirks 2<FORCE_LBA48>
wd0: 372 GB, 775221 cyl, 16 head, 63 sec, 512 bytes/sect x 781422768 sectors
viaide0:1:0: lost interrupt
    type: ata tc_bcount: 512 tc_skip: 0
viaide0:1:0: lost interrupt
    type: ata tc_bcount: 512 tc_skip: 0
wd0d: device timeout reading fsbn 0 (wd0 bn 0; cn 0 tn 0 sn 0), retrying


And that's where it hangs.

The last time I built a kernel was November, and that one works fine.

Robert

> 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?


--
HELP ME FIGHT CANCER!           DONATE HERE: http://tinyurl.com/ynq7uk
--
Robert Thille                7575 Meadowlark Dr.; Sebastopol, CA 95472
Home: 707.824.9753    Office/VOIP: 707.780.1560     Cell: 707.217.7544
Robert.Thille%rangat.org@localhost   YIM:rthille   http://www.rangat.org/rthille