On Mon, 24 Apr 2006 12:02:47 -0700 (PDT)
Linus Torvalds <[email protected]> wrote:
>
>
> On Mon, 24 Apr 2006, Stephen Hemminger wrote:
> >
> > We should fail request_irq() if the SA_SHIRQ but the irq is edge-triggered.
>
> That would be HORRIBLE.
>
> Edge-triggered works perfectly fine for SA_SHIRQ, as long as there is just
> one user and the driver is properly written. Making request_irq() fail
> would break existing and working setups.
Couldn't we at least warn. Because you will loose irq's if two devices
are sharing an edge triggered irq. If A and B are sharing a edge triggered
IRQ; and both cause a transition, then when A clears it's IRQ the
shared IRQ will disappear and B's IRQ will be lost.
> If you have a driver that requires level-triggered interrupts, then your
> driver is arguably buggy. NAPI or no NAPI, doesn't matter. Edge-triggered
> interrupts is a fact of life, and deciding that you don't like them is not
> an excuse for saying "they should not work".
Driver's need to be able to depend on not losing interrupts.
> You can get an edge by having your driver make sure that it clears the
> interrupt source at some point where it requires an edge.
The problem is that IRQ system doesn't tell the driver the trigger status.
If the driver knew that the IRQ was edge triggered, it could do
the necessary workaround.
> For a driver writer, there is one rule above _all_ other rules:
>
> "Reality sucks, deal with it"
>
> That rule is inviolate, and no amount of "I wish", and "it _should_ work
> this way" or "..but the documentation says" matters at all.
The kernel should make the driver writer's problems easier not harder.
-
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/
[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]
