- dev - passt

[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

[PATCH] passt.1: Fix typo: "addressses", reported by Lintian
by Stefano Brivio 28 Oct '22

28 Oct '22

Signed-off-by: Stefano Brivio <sbrivio(a)redhat.com> --- passt.1 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/passt.1 b/passt.1 index d121050..2c93907 100644 --- a/passt.1 +++ b/passt.1 @@ -718,7 +718,7 @@ root@localhost's password: .SH NOTES -.SS Handling of traffic with local destination and source addressses +.SS Handling of traffic with local destination and source addresses Both \fBpasst\fR and \fBpasta\fR can bind on ports with a local address, depending on the configuration. Local destination or source addresses need to be -- 2.35.1

2 1

new bugs, new features
by Stefano Brivio 27 Oct '22

27 Oct '22

Hi, I finally took the chance to turn some sparse to-do lists I had into bugs and feature requests. I linked the most obvious Bugzilla queries at: https://passt.top/#contribute feel free to take up items if you're interested. Thanks, -- Stefano

1 0

new version 2022_10_26.f212044 available
by Stefano Brivio 27 Oct '22

27 Oct '22

The new version with tag 2022_10_26.f212044 includes the following changes: f212044 icmp: Don't discard first reply sequence for a given echo ID b062ee4 icmp: Add debugging messages for handled replies and requests 947d756 tap: Trace received (outbound) ICMP packets in debug mode, too 7402951 conf, passt.1: Don't imply --foreground with --debug https://passt.top/passt/log/?qt=range&q=2022_10_26.e4df8b0..2022_10_26.f212… Packages: - Copr (CentOS Stream, EPEL, Fedora, Mageia, openSUSE): https://copr.fedorainfracloud.org/coprs/sbrivio/passt/build/4991606/ permanent mirror: https://passt.top/builds/copr/0^20221026.ge4df8b0/ - Debian package, from x86_64 static build: https://passt.top/builds/latest/x86_64/passt_0f212044-1_all.deb - Fedora updates: https://bodhi.fedoraproject.org/updates/?packages=passt - Package for other RPM-based distributions, from x86_64 static build: https://passt.top/builds/latest/x86_64/passt-gf212044-1.x86_64.rpm - x86_64 static binaries: https://passt.top/builds/latest/x86_64/ -- Stefano

1 2

[PATCH] test/run: Temporarily disable distribution tests
by Stefano Brivio 27 Oct '22

27 Oct '22

They're too slow to cope with current release cycles, and they haven't found bugs in months, also because clang-tidy and cppcheck would find most of them earlier. Disable them for the moment. We should pre-install gcc and make in non-x86 images, as those run on my test machine with qemu TCG, and that's the real slow-down here. Then we can re-enable them. Signed-off-by: Stefano Brivio <sbrivio(a)redhat.com> --- test/run | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/test/run b/test/run index d61b2d3..cb3b6c0 100755 --- a/test/run +++ b/test/run @@ -110,12 +110,16 @@ run() { test passt_in_ns/shutdown teardown passt_in_ns + # TODO: Make those faster by at least pre-installing gcc and make on + # non-x86 images, then re-enable. +skip_distro() { setup distro test distro/debian test distro/fedora test distro/opensuse test distro/ubuntu teardown distro +} perf_finish [ ${CI} -eq 1 ] && video_stop -- 2.35.1

2 1

[PATCH 0/4] Debugging improvements, fix for 0 sequence echo reply
by Stefano Brivio 26 Oct '22

26 Oct '22

Patches 1/4 to 3/4 make debugging of the issue solved in 4/4 more viable. Stefano Brivio (4): conf, passt.1: Don't imply --foreground with --debug tap: Trace received (outbound) ICMP packets in debug mode, too icmp: Add debugging messages for handled replies and requests icmp: Don't discard first reply sequence for a given echo ID conf.c | 7 +++---- icmp.c | 46 +++++++++++++++++++++++++++++++++++++++------- icmp.h | 1 + passt.1 | 5 ++--- passt.c | 3 +++ tap.c | 2 ++ 6 files changed, 50 insertions(+), 14 deletions(-) -- 2.35.1

2 8

[PATCH] checksum: Fix calculation for ICMP checksum on IPv4
by Stefano Brivio 26 Oct '22

26 Oct '22

We need to zero out the checksum field before calculating the checksum, of course. I have no idea how this passed the "icmp" test set, looking into it. Reported-by: Paul Holzinger <pholzing(a)redhat.com> Fixes: 67ab6171729c ("Add csum_icmp4() helper for calculating ICMP checksums") Signed-off-by: Stefano Brivio <sbrivio(a)redhat.com> --- checksum.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/checksum.c b/checksum.c index 09d2c7c..c59869c 100644 --- a/checksum.c +++ b/checksum.c @@ -160,10 +160,13 @@ void csum_udp4(struct udphdr *udp4hr, in_addr_t saddr, in_addr_t daddr, */ void csum_icmp4(struct icmphdr *icmp4hr, const void *payload, size_t len) { - /* Partial checksum for ICMP header alone */ - uint32_t psum = sum_16b(icmp4hr, sizeof(*icmp4hr)); + uint32_t psum; icmp4hr->checksum = 0; + + /* Partial checksum for ICMP header alone */ + psum = sum_16b(icmp4hr, sizeof(*icmp4hr)); + icmp4hr->checksum = csum_unaligned(payload, len, psum); } -- 2.35.1

3 7

2024

2023

2022

2021

dev