]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commit
Merge branch 'vhost-skb-leaks'
authorDavid S. Miller <davem@davemloft.net>
Sun, 3 Dec 2017 02:31:04 +0000 (21:31 -0500)
committerDavid S. Miller <davem@davemloft.net>
Sun, 3 Dec 2017 02:31:04 +0000 (21:31 -0500)
commit7344ba039f4fc5fb18be5618199827e246b0c697
treedc7a2620476bf2716201f858f342f3b389cdfae2
parentfa935ca224b43031ee10c857d7535aaec85c83b2
parent61d78537843e676e7f56ac6db333db0c0529b892
Merge branch 'vhost-skb-leaks'

Wei Xu says:

====================
vhost: fix a few skb leaks

Matthew found a roughly 40% tcp throughput regression with commit
c67df11f(vhost_net: try batch dequing from skb array) as discussed
in the following thread:
https://www.mail-archive.com/netdev@vger.kernel.org/msg187936.html

v4:
- fix zero iov iterator count in tap/tap_do_read()(Jason)
- don't put tun in case of EBADFD(Jason)
- Replace msg->msg_control with new 'skb' when calling tun/tap_do_read()

v3:
- move freeing skb from vhost to tun/tap recvmsg() to not
  confuse the callers.

v2:
- add Matthew as the reporter, thanks matthew.
- moving zero headcount check ahead instead of defer consuming skb
  due to jason and mst's comment.
- add freeing skb in favor of recvmsg() fails.
====================

Acked-by: Michael S. Tsirkin <mst@redhat.com>
Tested-by: Matthew Rosato <mjrosato@linux.vnet.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>