Commitd6acfeb

felix-cavium

authored and

davem330

committed

vxlan: vxlan dev should inherit lowerdev's gso_max_size

vxlan dev currently ignores lowerdev's gso_max_size, which adverselyaffects TSO performance of liquidio if it's the lowerdev. Egress TCPpackets' skb->len often exceed liquidio's advertised gso_max_size. Thismay happen on other NIC drivers.Fix it by assigning lowerdev's gso_max_size to that of vxlan dev. Might aswell do likewise for gso_max_segs.Single flow TSO throughput of liquidio as lowerdev (using iperf3): Before the patch: 139 Mbps After the patch : 8.68 Gbps Percent increase: 6,144 %Signed-off-by: Felix Manlunas <felix.manlunas@cavium.com>Signed-off-by: Satanand Burla <satananda.burla@cavium.com>Signed-off-by: David S. Miller <davem@davemloft.net>

1 parent6c7c98b commitd6acfebCopy full SHA for d6acfeb

File tree

1 file changed

-0

lines changed

drivers/net
- vxlan.c

1 file changed

-0

lines changed

`‎drivers/net/vxlan.c‎`

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -2926,6 +2926,11 @@ static int vxlan_dev_configure(struct net src_net, struct net_device dev,`
`2926`	`2926`	`return-EINVAL;`
`2927`	`2927`	`}`
`2928`	`2928`
	`2929`	`+if (lowerdev) {`
	`2930`	`+dev->gso_max_size=lowerdev->gso_max_size;`
	`2931`	`+dev->gso_max_segs=lowerdev->gso_max_segs;`
	`2932`	`+}`
	`2933`	`+`
`2929`	`2934`	`if (conf->mtu) {`
`2930`	`2935`	`intmax_mtu=ETH_MAX_MTU;`
`2931`	`2936`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitd6acfeb

File tree

1 file changed

1 file changed

`‎drivers/net/vxlan.c‎`

0 commit comments