Re: regression caused by: genirq: do not leave interupts enabled on free_irq

From: Yinghai Lu
Date: Fri Apr 11 2008 - 12:24:26 EST


On Fri, Apr 11, 2008 at 8:19 AM, Rafael J. Wysocki <rjw@xxxxxxx> wrote:
>
> On Friday, 11 of April 2008, Thomas Gleixner wrote:
> > On Fri, 11 Apr 2008, Eric W. Biederman wrote:
> > > On Fri, 2008-04-11 at 00:17 -0700, Yinghai Lu wrote:
> > > > On Fri, Apr 11, 2008 at 12:14 AM, Yinghai Lu <yhlu.kernel@xxxxxxxxx> wrote:
> > > > > On Fri, Apr 11, 2008 at 12:01 AM, Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
> > > > > > On Thu, 10 Apr 2008, Yinghai Lu wrote:
> > > > > > > last week found:
> > > > > > > after latest kernel kexec RHEL 5.1 or other stack kernel, the nvidia
> > > > > > > forcedeth doesn't work anymore.
> > > > > > >
> > > > > > > I stared at forcedeth.c two days. and revert every patches about that
> > > > > > > doesn't help.
> > > > > >
> > > > > > So forcedeth does not come up again, when you kexec from linus.git
> > > > > > into an older distro kernel. Or is it the other way round ?
> > > > > RHEL 5.1 kexec RHEL 5.1 : works
> > > > > RHEL 5.1 kexec linus kernel: works
> > > > > linus (after -rc2) kexec linus tree: works:
> > > > > linus (after -rc2) kexec RHEL 5.1 : forcedeth will not come up
> > > > > linus ( before -rc2 include rc2) kexec RHEL 5.1 works
> > > > >
> > > > the forcedeth can not get IP address...
> > >
> > > Sounds like you are not getting any interrupts when you receive a
> > > packet. (i.e. The interrupt line is staying disabled).
> > >
> > > Is MSI an option here? I'm wondering if we disable the MSI and
> > > something is not enabling it.
> >
> > Hmm, we disable the interrupt on free_irq(), but we reenable it in
> > request_irq()/setup_irq(), which is called when the forcedeth driver
> > initializes in the kexeced kernel. So there is some other deeper down
> > problem lurking.
> >
> > Yinghai, can you apply that patch to RHEL 5.1 and check, what happens
> > if you do:
> >
> > modprobe forcedeth
> > ifup ...
> > ifdown ...
> > rmmod forcedeth
> > modprobe forcedeth
> > ifup ...
> >
> > This should result in the same problem, but probably simpler to debug.
>
> I've just got a report from a forcedeth user that it doesn't work after a
> resume from RAM for him any more. Can it be related?

then another path that missing enable irq..

YH
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/