From: David Gibson <david@gibson.dropbear.id.au>
To: Stefano Brivio <sbrivio@redhat.com>, passt-dev@passt.top
Cc: David Gibson <david@gibson.dropbear.id.au>
Subject: [PATCH v2 0/9] Improve handling of MTU limits
Date: Wed, 12 Mar 2025 13:18:30 +1100 [thread overview]
Message-ID: <20250312021839.2405877-1-david@gibson.dropbear.id.au> (raw)
After considerable lead up, this corrects the handling of the --mtu
option so that it will respect limits imposed by the backend tap
layers, correctly accounting for L2 headers.
This incorporates the earlier mode setting patches, on which the rest
depend. There's been no change in those patches, there just included
here for self-containedness.
v2:
* More static_assert() validation of frame limits in 6/9
* Cosmetic fixups in 6/9
David Gibson (9):
conf: Use the same optstring for passt and pasta modes
conf: Move mode detection into helper function
conf: Detect vhost-user mode earlier
packet: Give explicit name to maximum packet size
packet: Remove redundant TAP_BUF_BYTES define
tap: Use explicit defines for maximum length of L2 frame
Simplify sizing of pkt_buf
pcap: Correctly set snaplen based on tap backend type
conf: Limit maximum MTU based on backend frame size
conf.c | 98 +++++++++++++++++++++++++++++++++++++++++---------------
conf.h | 1 +
packet.c | 4 +--
packet.h | 3 ++
passt.c | 16 ++-------
passt.h | 7 ++--
pcap.c | 46 +++++++++++++-------------
tap.c | 44 ++++++++++++++++++++++---
tap.h | 26 +++++++++++++++
util.h | 3 --
10 files changed, 173 insertions(+), 75 deletions(-)
--
2.48.1
next reply other threads:[~2025-03-12 3:07 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-12 2:18 David Gibson [this message]
2025-03-12 2:18 ` [PATCH v2 1/9] conf: Use the same optstring for passt and pasta modes David Gibson
2025-03-12 2:18 ` [PATCH v2 2/9] conf: Move mode detection into helper function David Gibson
2025-03-12 2:18 ` [PATCH v2 3/9] conf: Detect vhost-user mode earlier David Gibson
2025-03-12 2:18 ` [PATCH v2 4/9] packet: Give explicit name to maximum packet size David Gibson
2025-03-12 2:18 ` [PATCH v2 5/9] packet: Remove redundant TAP_BUF_BYTES define David Gibson
2025-03-12 2:18 ` [PATCH v2 6/9] tap: Use explicit defines for maximum length of L2 frame David Gibson
2025-03-12 2:18 ` [PATCH v2 7/9] Simplify sizing of pkt_buf David Gibson
2025-03-12 2:18 ` [PATCH v2 8/9] pcap: Correctly set snaplen based on tap backend type David Gibson
2025-03-12 2:18 ` [PATCH v2 9/9] conf: Limit maximum MTU based on backend frame size David Gibson
2025-03-12 23:04 ` [PATCH v2 0/9] Improve handling of MTU limits Stefano Brivio
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250312021839.2405877-1-david@gibson.dropbear.id.au \
--to=david@gibson.dropbear.id.au \
--cc=passt-dev@passt.top \
--cc=sbrivio@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://passt.top/passt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).