| To: | Alan Cox <alan@xxxxxxxxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [announce] [patch] limiting IRQ load, irq-rewrite-2.4.11-B5 |
| From: | Pavel Machek <pavel@xxxxxxx> |
| Date: | Wed, 10 Oct 2001 16:26:52 +0000 |
| Cc: | jamal <hadi@xxxxxxxxxx>, Jeff Garzik <jgarzik@xxxxxxxxxxxxxxxx>, Andrea Arcangeli <andrea@xxxxxxx>, Ingo Molnar <mingo@xxxxxxx>, Linux-Kernel <linux-kernel@xxxxxxxxxxxxxxx>, netdev@xxxxxxxxxxx, Linus Torvalds <torvalds@xxxxxxxxxxxxx> |
| In-reply-to: | <E15qczg-00011N-00@xxxxxxxxxxxxxxxxx>; from alan@xxxxxxxxxxxxxxxxxxx on Mon, Oct 08, 2001 at 05:11:03PM +0100 |
| References: | <Pine.GSO.4.30.0110081146050.5473-100000@xxxxxxxxxxxxxxxx> <E15qczg-00011N-00@xxxxxxxxxxxxxxxxx> |
| Sender: | owner-netdev@xxxxxxxxxxx |
Hi!
> > Agreed if you add the polling cardbus bit.
> > Note polling cardbus would require more changes than the above.
>
> I don't think it does. There are two pieces to the problem
>
> a) Not dying horribly
> b) Handling it elegantly
>
> b) is driver specific (NAPI etc) and I think well understood to the point
> its being used already for performance reasons
>
> a) is as simple as
>
> if(stuck_in_irq(foo) && irq_shared(foo))
> {
> disable_real_irq(foo);
> timer_fake_irq_foo();
> }
I'd kill irq_shared() test, and added a printk :-).
Pavel
--
Philips Velo 1: 1"x4"x8", 300gram, 60, 12MB, 40bogomips, linux, mutt,
details at http://atrey.karlin.mff.cuni.cz/~pavel/velo/index.html.
|
| Previous by Date: | Re: [announce] [patch] limiting IRQ load, irq-rewrite-2.4.11-B5, Pavel Machek |
|---|---|
| Next by Date: | Retrieving in.terface statistics frequently and without pain, Richard Sharpe |
| Previous by Thread: | Re: [announce] [patch] limiting IRQ load, irq-rewrite-2.4.11-B5, jamal |
| Next by Thread: | Re: [announce] [patch] limiting IRQ load, irq-rewrite-2.4.11-B5, Pavel Machek |
| Indexes: | [Date] [Thread] [Top] [All Lists] |