[PATCH] gianfar: Fall back to software tcp/udp checksum on older controllers

Alex Dubov oakad at yahoo.com
Fri Jan 28 13:43:44 EST 2011


> > -       
> gfar_tx_checksum(skb, fcb);
> > +        switch
> (unlikely(gfar_has_errata(priv, GFAR_ERRATA_12))
> > +       
>     ? 1 : 0) {
> 
> The switch construction is quite bizarre. ;-) Why not
> 

My testing shows that even on broken chips unaligned packets are quite rare
(only some SYN packets seem to be affected). So I tried to leave the
checksum offload path as the most preferred one.



      


More information about the Linuxppc-dev mailing list