Re: [PATCH] NETIF_F_LLTX for devices 2

[Jeff Garzik <jgarzik@xxxxxxxxx>]

Andi Kleen wrote:
> On Sun, Sep 12, 2004 at 12:16:05PM -0400, Jeff Garzik wrote:
>
> > Incorrect, you are changing the callsites, which -does- affect every
> > driver.
>
>
> Please read the code before making such claims. 
>
> The new return code is _only_ checked when NETIF_F_LLTX is set. 
> A driver that doesn't set this new flag won't ever recognize 
> any difference.


I read the code :)

The basic premise is that one should be _really_ conservative when 
touching the core RX and TX paths.  Regardless of how safe _you_ feel 
the code is, it is very new, under-analyzed, and untried.

The NAPI-related bug recently fixed in tg3 is an example of the 
unintended consequences of using this new feature.

        Jeff