Re: [LEDE-DEV] [PATCH] ar71xx: Add GRO support to ag71xx

2017-10-17 Thread Dave Taht
Felix Fietkau writes: > On 2017-09-03 20:35, Rosen Penev wrote: >> On a TL-WN710N, this patch increases iperf performance from ~92.5 to ~93.5 >> mbps.> Keep in mind the WN710N is a 100mbps device. I expect greater numbers > from gigabit devices. >> >> Signed-off-by: Rosen Penev > Hi Rosen, > >

Re: [LEDE-DEV] [PATCH] ar71xx: Add GRO support to ag71xx

2017-10-17 Thread Felix Fietkau
On 2017-09-03 20:35, Rosen Penev wrote: > On a TL-WN710N, this patch increases iperf performance from ~92.5 to ~93.5 > mbps.> Keep in mind the WN710N is a 100mbps device. I expect greater numbers from gigabit devices. > > Signed-off-by: Rosen Penev Hi Rosen, Sorry about that, but I will have to

Re: [LEDE-DEV] [PATCH] ar71xx: Add GRO support to ag71xx

2017-09-04 Thread rosenp
After a bit more research, it turns out this change is really boring. Eric Dumazet did this for many ethernet drivers early this year. As far as TSO goes, no idea how to even go about that. As an interesting side note, TSO support broke mv643xx_eth for a while. On Sun, 2017-09-03 at 19:10 -0700,

Re: [LEDE-DEV] [PATCH] ar71xx: Add GRO support to ag71xx

2017-09-03 Thread Florian Fainelli
Le 09/03/17 à 15:46, ros...@gmail.com a écrit : > That's...way better than I expected given how minimal my changes are. > > For some context, the ag71xx driver is special in that it does not seem > to do any hardware offloading to the NIC. > > As far as I understand this change, GRO takes 1500 MT

Re: [LEDE-DEV] [PATCH] ar71xx: Add GRO support to ag71xx

2017-09-03 Thread rosenp
That's...way better than I expected given how minimal my changes are. For some context, the ag71xx driver is special in that it does not seem to do any hardware offloading to the NIC. As far as I understand this change, GRO takes 1500 MTU packets and packs then into 64Kb blocks which the kernel t

Re: [LEDE-DEV] [PATCH] ar71xx: Add GRO support to ag71xx

2017-09-03 Thread Christian Lamparter via Lede-dev
The sender domain has a DMARC Reject/Quarantine policy which disallows sending mailing list messages using the original "From" header. To mitigate this problem, the original message has been wrapped automatically by the mailing list software.--- Begin Message --- On Sunday, September 3, 2017 11:35

[LEDE-DEV] [PATCH] ar71xx: Add GRO support to ag71xx

2017-09-03 Thread Rosen Penev
On a TL-WN710N, this patch increases iperf performance from ~92.5 to ~93.5 mbps. Keep in mind the WN710N is a 100mbps device. I expect greater numbers from gigabit devices. Signed-off-by: Rosen Penev --- .../ar71xx/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c | 6 +++--- 1 file cha