May 2024 - dev

[PATCH v2 0/7] Rework UDP buffers
by David Gibson 02 May '24

02 May '24

Laurent recently reworked the TCP buffer handling to be split into various pieces tracked by iovecs. We'll want that for various future changes. This series makes a similar split for UDP buffers, which we'll want in order to allow dual-stack UDP sockets, amongst other things. This is based on my earlier series of cleanups for the TCP buffer handling. Changes since v1: * Rebase on v2 of the TCP iov and buffer handling cleanups * Rename udp_payload and udp_meta structures to _t for more consistency with tcp.c * Other minor changes based on Stefano's review David Gibson (7): test: Allow sftp via vsock-ssh in tests udp: Split tap-bound UDP packets into multiple buffers using io vector udp: Combine initialisation of IPv4 and IPv6 iovs udp: Explicitly set checksum in guest-bound UDP headers udp: Share payload buffers between IPv4 and IPv6 udp: Use the same buffer for the L2 header for all frames udp: Single buffer for IPv4, IPv6 headers and metadata tap.h | 38 ------ test/passt.mbuto | 6 +- udp.c | 299 +++++++++++++++++++++++------------------------ 3 files changed, 152 insertions(+), 191 deletions(-) -- 2.44.0

2 8

[PATCH v2 00/10] Small improvements to IOV handling
by David Gibson 02 May '24

02 May '24

Laurent's changes to split TCP buffers into various components with IOVs is now merged. This series has a batch of small cleanups to make the handling of this slightly nicer. These are preliminaries to doing something similar with the UDP buffers. Note that patch 10/10 might interfere with the experiments to work out what is going wrong with the odd batching / performance issues we've seen. We can leave it off for the time being if that's a problem. Changes since v1: * Added new patch removing some unused structures * Added two new patches cleaning up some endianness confusion * Added a bunch of missing cases to standardisation of length variable names * Assorted minor revisions based on Stefano's review David Gibson (10): checksum: Use proto_ipv6_header_psum() for ICMPv6 as well tap: Split tap specific and L2 (ethernet) headers tap: Remove unused structs tap_msg, tap_l4_msg treewide: Remove misleading and redundant endianness notes checksum: Make csum_ip4_header() take a host endian length treewide: Standardise variable names for various packet lengths tcp: Simplify packet length calculation when preparing headers tap, tcp: (Re-)abstract TAP specific header handling iov: Helper macro to construct iovs covering existing variables or fields tcp: Update tap specific header too in tcp_fill_headers[46]() arp.c | 6 +- checksum.c | 68 ++++++++++----------- checksum.h | 12 ++-- dhcp.c | 6 +- icmp.c | 12 ++-- iov.h | 3 + ndp.c | 6 +- passt.h | 28 ++------- pcap.c | 14 ++--- pcap.h | 2 +- tap.c | 146 ++++++++++++++++++++++---------------------- tap.h | 60 ++++++++++++------ tcp.c | 176 +++++++++++++++++++++++------------------------------ udp.c | 52 ++++++++-------- 14 files changed, 290 insertions(+), 301 deletions(-) -- 2.44.0

2 11

[PATCH 0/2] netlink: Don't duplicate routes with mismatching interfaces
by Stefano Brivio 01 May '24

01 May '24

Stefano Brivio (2): netlink: Fix iterations over nexthop objects netlink: Don't duplicate routes referring to unrelated host interfaces netlink.c | 53 ++++++++++++++++++++++++++++++++++++++++++++--------- 1 file changed, 44 insertions(+), 9 deletions(-) -- 2.43.0

2 4

[PATCH 0/7] Rework UDP buffers
by David Gibson 01 May '24

01 May '24

Laurent recently reworked the TCP buffer handling to be split into various pieces tracked by iovecs. We'll want that for various future changes. This series makes a similar split for UDP buffers, which we'll want in order to allow dual-stack UDP sockets, amongst other things. This is based on my earlier series of cleanups for the TCP buffer handling. David Gibson (7): test: Allow sftp via vsock-ssh in tests udp: Split tap-bound UDP packets into multiple buffers using io vector udp: Combine initialisation of IPv4 and IPv6 iovs udp: Explicitly set checksum in guest-bound UDP headers udp: Share payload buffers between IPv4 and IPv6 udp: Use the same buffer for the L2 header for all frames udp: Single buffer for IPv4, IPv6 headers and metadata tap.h | 38 ------ test/passt.mbuto | 6 +- udp.c | 304 +++++++++++++++++++++++------------------------ 3 files changed, 153 insertions(+), 195 deletions(-) -- 2.44.0

2 13

[PATCH 0/7] Small improvements to IOV handling
by David Gibson 01 May '24

01 May '24

Laurent's changes to split TCP buffers into various components with IOVs is now merged. This series has a batch of small cleanups to make the handling of this slightly nicer. These are preliminaries to doing something similar with the UDP buffers. Note that patch 7/7 might interfere with the experiments to work out what is going wrong with the odd batching / performance issues we've seen. We can leave it off for the time being if that's a problem. David Gibson (7): checksum: Use proto_ipv6_header_psum() for ICMPv6 as well tap: Split tap specific and L2 (ethernet) headers treewide: Standardise variable names for various packet lengths tcp: Simplify packet length calculation when preparing headers tap, tcp: (Re-)abstract TAP specific header handling iov: Helper macro to construct iovs covering existing variables or fields tcp: Update tap specific header too in tcp_fill_headers[46]() checksum.c | 46 ++++++++--------- checksum.h | 10 ++-- icmp.c | 8 +-- iov.h | 3 ++ passt.h | 4 +- tap.c | 51 ++++++++++--------- tap.h | 46 +++++++++++++---- tcp.c | 144 +++++++++++++++++++++++------------------------------ udp.c | 50 ++++++++++--------- 9 files changed, 184 insertions(+), 178 deletions(-) -- 2.44.0

2 17