Thread (41 messages) 41 messages, 7 authors, 2009-04-30

Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment

From: Andrew Gallatin <hidden>
Date: 2009-04-15 21:04:55

Andrew Gallatin wrote:
David Miller wrote:
quoted
From: Brice Goglin <redacted>
quoted
Is there a good summary somewhere of why GRO is better,
Transparent forwarding/bridging support, easier driver port.
quoted
and how to
actually convert drivers?
Step 1: Remove all of your LRO support code, every last line
Step 2: netif_receive_skb() --> napi_gro_receive()
        vlan_hwaccel_rx() --> vlan_gro_receive()

It couldn't be any easier.

And it would also behoove you to look at the commits that converted or
added GRO support to other drivers.  That's how I learned it :-)
Unfortunately, it doesn't appear that GRO is able to handle frags
(like lro_receive_frags()), so I anticipate its overhead would
Ah, I missed napi_gro_frags()!   I've got quick and dirty test
patch which uses that, but I need to fix a few things. I also need
to figure out why it seems to be a bit slower than LRO
(varies from 8.5 to 9.2 Gb/s, rather than always 9.4Gb/s)
on my old, weak 2.0GHz athlon64.

Thanks for the pointer,

Drew
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help