netdev
[Top] [All Lists]

Re: [announce] [patch] limiting IRQ load, irq-rewrite-2.4.11-B5

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.


<Prev in Thread] Current Thread [Next in Thread>