Subject: Re: [patch 03/13] forcedeth: improve NAPI logic

From: Jeff Garzik <[email-masked]>
Date: Fri, 11 May 2007 03:11:04 -0400

[email-masked] wrote:
> From: Ingo Molnar <[email-masked]>
> 
> Another forcedeth.c thing: i noticed that its NAPI handler does not do
> tx-ring processing.  The patch below implements this - tested on DESC_VER_2
> hardware, with CONFIG_FORCEDETH_NAPI=y.
> 
> 
> Signed-off-by: Ingo Molnar <[email-masked]>
> Cc: Ayaz Abdulla <[email-masked]>
> ---
> Auke said:
> 
> the patch needs to be improved.
> 
> The following needs to be done when NAPI is enabled:
> - remove the tx handling within the ISRs
> - mask off the tx interrupts within the ISRs that handle tx processing
> - re-enable tx interrupts within the NAPI handler
> - add tx handling within the NAPI handler (this patch covers it)
> 
> Signed-off-by: Andrew Morton <[email-masked]>
> ---
> 
>  drivers/net/forcedeth.c |    8 ++++++++
>  1 file changed, 8 insertions(+)

This was NAK'd at least twice?

It's a half-finished project.  You need to do more, if moving TX from 
irq handler to NAPI.

I'm definitely not applying it, and as its incomplete, it definitely 
isn't suitable for upstream.

	Jeff