Re: [PATCH -mm] misroute-irq: Don't call desc->chip->end because of edge interrupts

On Tue, 06 Jun 2006 06:50:19 -0400
Steven Rostedt <[email protected]> wrote:

> On Tue, 2006-06-06 at 10:01 +0200, Ingo Molnar wrote:
> > * Steven Rostedt <[email protected]> wrote:
> > 
> > > Hit the following BUG with irqpoll.  The below patch fixes it.
> > 
> > > -		if (work)
> > > +		if (work && disc->chip && desc->chip->end)
> > >  			desc->chip->end(i);
> > 

Why is this change necessary?  2.6.17-rc6 doesn't have it, and it doesn't
oops.  So somebody changed something.  What?  Why?  Was it intentional?  Was
it correct?

> 
> As for eio, could be.  This was part of my whole misroute thing which I
> didn't have time to get to deep in.  I'm leaving today, where I won't
> have any computers or Internet til next Wednesday or Thursday, so I'm
> not going to be able to work on this further till then.

If "could be" == "yes" then what would be the consequences of this
bug-which-were-not-sure-is-there-yet?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: [PATCH -mm] misroute-irq: Don't call desc->chip->end because of edge interrupts
    • From: Ingo Molnar <[email protected]>

• References:
  • Re: [patch, -rc5-mm1] locking validator: special rule: 8390.c disable_irq()
    • From: Alan Cox <[email protected]>
  • Re: [patch, -rc5-mm1] locking validator: special rule: 8390.c disable_irq()
    • From: Steven Rostedt <[email protected]>
  • Re: [patch, -rc5-mm1] locking validator: special rule: 8390.c disable_irq()
    • From: Arjan van de Ven <[email protected]>
  • Re: [patch, -rc5-mm1] locking validator: special rule: 8390.c disable_irq()
    • From: Steven Rostedt <[email protected]>
  • Re: [patch, -rc5-mm1] locking validator: special rule: 8390.c disable_irq()
    • From: Alan Cox <[email protected]>
  • Re: [patch, -rc5-mm1] locking validator: special rule: 8390.c disable_irq()
    • From: Steven Rostedt <[email protected]>
  • Re: [patch, -rc5-mm1] locking validator: special rule: 8390.c disable_irq()
    • From: Alan Cox <[email protected]>
  • Re: [patch, -rc5-mm1] locking validator: special rule: 8390.c disable_irq()
    • From: Steven Rostedt <[email protected]>
  • Re: [patch, -rc5-mm1] locking validator: special rule: 8390.c disable_irq()
    • From: Ingo Molnar <[email protected]>
  • [PATCH -mm] misroute-irq: Don't call desc->chip->end because of edge interrupts
    • From: Steven Rostedt <[email protected]>
  • Re: [PATCH -mm] misroute-irq: Don't call desc->chip->end because of edge interrupts
    • From: Ingo Molnar <[email protected]>
  • Re: [PATCH -mm] misroute-irq: Don't call desc->chip->end because of edge interrupts
    • From: Steven Rostedt <[email protected]>

• Prev by Date: Quick close of all the open files.
• Next by Date: Re: [PATCH -mm] misroute-irq: Don't call desc->chip->end because of edge interrupts
• Previous by thread: Re: [PATCH -mm] misroute-irq: Don't call desc->chip->end because of edge interrupts
• Next by thread: Re: [PATCH -mm] misroute-irq: Don't call desc->chip->end because of edge interrupts
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]