- dev - passt

[PATCH v2 0/6] Improve IPv4 address endian handling and related bug fixes
by David Gibson 04 Nov '22

04 Nov '22

While working on allowing us to use IPv4/IPv6 combined listening sockets, I made several endian mistakes. So, I took a diversion to make it harder to make such mistakes. Along the way I found some small existing endian and other bugs in IPv4 address handling. Changes since v1: * Added extra patch making more safety conversion * Added extra patch making some tangentially related comment fixes David Gibson (6): Correct some missing endian conversions of IPv4 addresses Minor improvements to IPv4 netmask handling Use IPV4_IS_LOOPBACK more widely Use typing to reduce chances of IPv4 endianness errors Use endian-safer typing in struct tap4_l4_t tcp: Correct function comments for address types checksum.c | 7 ++-- checksum.h | 3 +- conf.c | 118 ++++++++++++++++++++++++++++++----------------------- dhcp.c | 15 ++++--- icmp.c | 5 +-- passt.c | 2 +- passt.h | 16 ++++---- pasta.c | 7 +--- tap.c | 49 +++++++++++----------- tap.h | 8 ++-- tcp.c | 60 +++++++++++++-------------- tcp.h | 2 +- udp.c | 30 +++++++------- udp.h | 2 +- util.h | 12 +++++- 15 files changed, 182 insertions(+), 154 deletions(-) -- 2.38.1

1 6

[PATCH 0/3] Prepare for native AF_UNIX socket support in qemu 7.2
by Stefano Brivio 04 Nov '22

04 Nov '22

Patches 1/3 and 2/3 deal with the tests, patch 3/3 updates usage notes and documentation bits. Indeed, we'll need to keep qrap around for a little longer, but it's probably time to start telling users about this. Stefano Brivio (3): test: Switch to qemu -netdev stream option instead of using qrap test/perf: Finally drop workaround for virtio_net TX stall passt, qrap, README: Update notes and documentation for AF_UNIX support in qemu README.md | 10 ++++----- passt.1 | 8 ++----- qrap.1 | 13 +++++------- qrap.c | 5 +++-- tap.c | 10 ++++----- test/README.md | 5 +++++ test/demo/passt | 9 +++----- test/distro/debian | 24 ++++++++++----------- test/distro/fedora | 38 ++++++++++++++++----------------- test/distro/opensuse | 14 ++++++------- test/distro/ubuntu | 12 +++++------ test/lib/setup | 50 ++++++++++++++++++++------------------------ test/perf/passt_tcp | 15 ------------- test/perf/passt_udp | 15 ------------- 14 files changed, 93 insertions(+), 135 deletions(-) -- 2.35.1

1 3

[PATCH v4 0/5] Test cases to display memory usage in kernel and userspace
by Stefano Brivio 04 Nov '22

04 Nov '22

Note that 2/5 is not actually needed for the purposes of these "test cases" (which always succeed, at the moment), it just fixes a condition I hit while writing them. v4: Fold changes by David: use a dummy interface for the purpose of starting passt, and drop setup and teardown steps related to the "outer" instance of passt, now gone. Install passt in /bin instead of /usr/bin, but also add a link v3: In 5/5, drop useless conditionals and teardown steps as pointed out by David v2: Actually include passt.mem.mbuto in 5/5, and move the introduction of INITRAMFS_MEM for the setup function there Stefano Brivio (5): test/lib: Move screen-scraping setup and layout functions to _ugly files tap: Support for detection of existing sockets on ramfs test/lib/perf_report: Use own flag to track initialisation test/lib: Add "td" directive, handled by table_value() test: Add memory/passt test cases tap.c | 3 +- test/Makefile | 5 +- test/lib/layout | 69 ---------------- test/lib/layout_ugly | 113 ++++++++++++++++++++++++++ test/lib/perf_report | 9 ++- test/lib/setup | 23 ------ test/lib/setup_ugly | 58 ++++++++++++++ test/lib/term | 28 +++++++ test/lib/test | 3 + test/memory/passt | 187 +++++++++++++++++++++++++++++++++++++++++++ test/passt.mem.mbuto | 44 ++++++++++ test/run | 6 ++ 12 files changed, 451 insertions(+), 97 deletions(-) create mode 100644 test/lib/layout_ugly create mode 100755 test/lib/setup_ugly create mode 100644 test/memory/passt create mode 100755 test/passt.mem.mbuto -- 2.35.1

2 6

[PATCH v3 0/3] Fixes and workarounds for pasta with Podman in Google Cloud
by Stefano Brivio 03 Nov '22

03 Nov '22

These patches work around or fix some issues found while testing the Podman integration for pasta in Google Compute Engine environments. v3: Fix up botched rebase, especially in 2/3, further reword 2/3, fix up 1/3 to match the recent patch from David that stores IPv4 netmasks as prefix lengths instead of masks v2: Rephrase commit messages of 1/3 and 2/3 adding examples with addresses. In 2/3, set host addresses (dns4 and dns6) and increase the pointer regardless of what we're doing with dns4_send and dns6_send. Stefano Brivio (3): conf: Adjust netmask on mismatch between IPv4 address/netmask and gateway conf: Split the notions of read DNS addresses and offered ones udp: Check for answers to forwarded DNS queries before handling local redirects conf.c | 74 ++++++++++++++++++++++++++++++++++++++++++++------------ dhcp.c | 4 +-- dhcpv6.c | 5 ++-- ndp.c | 6 ++--- passt.h | 8 ++++-- udp.c | 22 ++++++++--------- 6 files changed, 84 insertions(+), 35 deletions(-) -- 2.35.1

1 3

[PATCH v2 0/3] Fixes and workarounds for pasta with Podman in Google Cloud
by Stefano Brivio 03 Nov '22

03 Nov '22

These patches work around or fix some issues found while testing the Podman integration for pasta in Google Compute Engine environments. v2: Rephrase commit messages of 1/3 and 2/3 adding examples with addresses. In 2/3, set host addresses (dns4 and dns6) and increase the pointer regardless of what we're doing with dns4_send and dns6_send. Stefano Brivio (3): conf: Consistency check between configured IPv4 netmask and gateway conf: Split the notions of read DNS addresses and offered ones udp: Check for answers to forwarded DNS queries before handling local redirects conf.c | 64 +++++++++++++++++++++++++++++++++++++++++++------------- dhcp.c | 4 ++-- dhcpv6.c | 5 +++-- ndp.c | 6 +++--- passt.h | 8 +++++-- udp.c | 22 +++++++++---------- 6 files changed, 74 insertions(+), 35 deletions(-) -- 2.35.1

1 4

[PATCH v3 0/5] Test cases to display memory usage in kernel and userspace
by Stefano Brivio 03 Nov '22

03 Nov '22

Note that 2/5 is not actually needed for the purposes of these "test cases" (which always succeed, at the moment), it just fixes a condition I hit while writing them. v3: In 5/5, drop useless conditionals and teardown steps as pointed out by David v2: Actually include passt.mem.mbuto in 5/5, and move the introduction of INITRAMFS_MEM for the setup function there Stefano Brivio (5): test/lib: Move screen-scraping setup and layout functions to _ugly files tap: Support for detection of existing sockets on ramfs test/lib/perf_report: Use own flag to track initialisation test/lib: Add "td" directive, handled by table_value() test: Add memory/passt test cases tap.c | 3 +- test/Makefile | 5 +- test/lib/layout | 69 ---------------- test/lib/layout_ugly | 117 +++++++++++++++++++++++++++ test/lib/perf_report | 9 ++- test/lib/setup | 23 ------ test/lib/setup_ugly | 72 +++++++++++++++++ test/lib/term | 28 +++++++ test/lib/test | 3 + test/memory/passt | 187 +++++++++++++++++++++++++++++++++++++++++++ test/passt.mem.mbuto | 42 ++++++++++ test/run | 6 ++ 12 files changed, 467 insertions(+), 97 deletions(-) create mode 100644 test/lib/layout_ugly create mode 100755 test/lib/setup_ugly create mode 100644 test/memory/passt create mode 100755 test/passt.mem.mbuto -- 2.35.1

2 7

[PATCH 0/4] Improve IPv4 address endian handling and related bug fixes
by David Gibson 03 Nov '22

03 Nov '22

While working on allowing us to use IPv4/IPv6 combined listening sockets, I made several endian mistakes. So, I took a diversion to make it harder to make such mistakes. Along the way I found some small existing endian and other bugs in IPv4 address handling. David Gibson (4): Correct some missing endian conversions of IPv4 addresses Minor improvements to IPv4 netmask handling Use IPV4_IS_LOOPBACK more widely Use typing to reduce chances of IPv4 endianness errors checksum.c | 7 ++-- checksum.h | 3 +- conf.c | 118 ++++++++++++++++++++++++++++++----------------------- dhcp.c | 15 ++++--- icmp.c | 5 +-- passt.c | 2 +- passt.h | 16 ++++---- pasta.c | 7 +--- tap.c | 18 ++++---- tap.h | 8 ++-- tcp.c | 48 +++++++++++----------- tcp.h | 2 +- udp.c | 30 +++++++------- udp.h | 2 +- util.h | 12 +++++- 15 files changed, 160 insertions(+), 133 deletions(-) -- 2.38.1

1 4

[PATCH 0/2] Refinements to memory usage tests
by David Gibson 01 Nov '22

01 Nov '22

Here are some patches on top of Stefano's series which adds tests showing userspace and kernel memory usage for passt. The first I needed to get the tests running on my system. The second just removes some unneccesary things. David Gibson (2): test: Correct passt binary path for memory usage tests test: Don't use "outer" passt for memory usage tests test/lib/layout_ugly | 9 --------- test/lib/setup_ugly | 21 ++------------------- test/passt.mem.mbuto | 6 ++++-- 3 files changed, 6 insertions(+), 30 deletions(-) -- 2.38.1

2 3

[PATCH v2 0/5] Test cases to display memory usage in kernel and userspace
by Stefano Brivio 01 Nov '22

01 Nov '22

Note that 2/5 is not actually needed for the purposes of these "test cases" (which always succeed, at the moment), it just fixes a condition I hit while writing them. v2: Actually include passt.mem.mbuto in 5/5, and move the introduction of INITRAMFS_MEM for the setup function there Stefano Brivio (5): test/lib: Move screen-scraping setup and layout functions to _ugly files tap: Support for detection of existing sockets on ramfs test/lib/perf_report: Use own flag to track initialisation test/lib: Add "td" directive, handled by table_value() test: Add memory/passt test cases tap.c | 3 +- test/Makefile | 5 +- test/lib/layout | 69 ---------------- test/lib/layout_ugly | 122 ++++++++++++++++++++++++++++ test/lib/perf_report | 9 ++- test/lib/setup | 23 ------ test/lib/setup_ugly | 75 +++++++++++++++++ test/lib/term | 28 +++++++ test/lib/test | 3 + test/memory/passt | 187 +++++++++++++++++++++++++++++++++++++++++++ test/passt.mem.mbuto | 42 ++++++++++ test/run | 6 ++ 12 files changed, 475 insertions(+), 97 deletions(-) create mode 100644 test/lib/layout_ugly create mode 100755 test/lib/setup_ugly create mode 100644 test/memory/passt create mode 100755 test/passt.mem.mbuto -- 2.35.1

2 11

[PATCH 0/5] Test cases to display memory usage in kernel and userspace
by Stefano Brivio 31 Oct '22

31 Oct '22

Note that 2/5 is not actually needed for the purposes of these "test cases" (which always succeed, at the moment), it just fixes a condition I hit while writing them. Stefano Brivio (5): test/lib: Move screen-scraping setup and layout functions to _ugly files tap: Support for detection of existing sockets on ramfs test/lib/perf_report: Use own flag to track initialisation test/lib: Add "td" directive, handled by table_value() test: Add memory/passt test cases tap.c | 3 +- test/Makefile | 5 +- test/lib/layout | 69 ---------------- test/lib/layout_ugly | 122 ++++++++++++++++++++++++++++ test/lib/perf_report | 9 ++- test/lib/setup | 24 +----- test/lib/setup_ugly | 73 +++++++++++++++++ test/lib/term | 28 +++++++ test/lib/test | 3 + test/memory/passt | 187 +++++++++++++++++++++++++++++++++++++++++++ test/run | 6 ++ 11 files changed, 432 insertions(+), 97 deletions(-) create mode 100644 test/lib/layout_ugly create mode 100755 test/lib/setup_ugly create mode 100644 test/memory/passt -- 2.35.1

1 6