public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
From: Laurent Vivier <lvivier@redhat.com>
To: passt-dev@passt.top
Cc: Laurent Vivier <lvivier@redhat.com>
Subject: [PATCH v6 00/30] Introduce discontiguous frames management
Date: Wed,  4 Jun 2025 15:08:04 +0200	[thread overview]
Message-ID: <20250604130834.3868010-1-lvivier@redhat.com> (raw)

This series introduces iov_tail to convey frame information
between functions.

This is only an API change, for the moment the memory pool
is only able to store contiguous buffer, so, except for
vhost-user in a special case, we only play with iovec array
with only one entry.

v6:
    - Replaced iov_slice() with the clearer iov_tail_clone()
      for creating iovec subsets.
    - Standardized local header variable names (to *_storage suffix).
    - Renamed functions for better semantics (e.g., ignore_arp to
      accept_arp, packet_data to packet_get).
    - Corrected OPTLEN_MAX definition in TCP.
    - Addressed minor logic issues (e.g., DHCPv6 FQDN flags, NDP null check).
    - Updated ipv6_l4hdr() return type to boolean.
    - Improved comments and documentation across several modules.

v5:
    - store in the pool iovec array with several entries

v4:
  Prepare to introduce iovec array in the pool:
    - passe iov_tail rather than pool to ndp,icmp, dhcp, dhcpv6 and arp
    - remove unused pool macros
    - add memory regions in the pool structure, this will allow us to use
      the buf pointer to store the iovec array for vhost-user

v3:
  Address comments from David

Laurent Vivier (30):
  arp: Don't mix incoming and outgoing buffers
  iov: Introduce iov_tail_clone() and iov_tail_drop().
  iov: Update IOV_REMOVE_HEADER() and IOV_PEEK_HEADER()
  tap: Use iov_tail with tap_add_packet()
  packet: Use iov_tail with packet_add()
  packet: Add packet_data()
  arp: Convert to iov_tail
  ndp: Convert to iov_tail
  icmp: Convert to iov_tail
  udp: Convert to iov_tail
  tcp: Convert tcp_tap_handler() to use iov_tail
  tcp: Convert tcp_data_from_tap() to use iov_tail
  dhcpv6: move offset initialization out of dhcpv6_opt()
  dhcpv6: Extract sending of NotOnLink status
  dhcpv6: Convert to iov_tail
  dhcpv6: Use iov_tail in dhcpv6_opt()
  dhcp: Convert to iov_tail
  ip: Use iov_tail in ipv6_l4hdr()
  tap: Convert tap4_handler() to iov_tail
  tap: Convert tap6_handler() to iov_tail
  packet: rename packet_data() to packet_get()
  arp: use iov_tail rather than pool
  dhcp: use iov_tail rather than pool
  dhcpv6: use iov_tail rather than pool
  icmp: use iov_tail rather than pool
  ndp: use iov_tail rather than pool
  packet: remove PACKET_POOL() and PACKET_POOL_P()
  packet: remove unused parameter from PACKET_POOL_DECL()
  packet: add memory regions information into pool
  packet: use buf to store iovec array

 arp.c        |  86 ++++++++++++-------
 arp.h        |   2 +-
 dhcp.c       |  47 ++++++-----
 dhcp.h       |   2 +-
 dhcpv6.c     | 222 +++++++++++++++++++++++++++++--------------------
 dhcpv6.h     |   2 +-
 icmp.c       |  41 ++++++----
 icmp.h       |   2 +-
 iov.c        | 102 +++++++++++++++++++++--
 iov.h        |  58 +++++++++----
 ip.c         |  32 ++++----
 ip.h         |   3 +-
 ndp.c        |  16 +++-
 ndp.h        |   4 +-
 packet.c     | 227 ++++++++++++++++++++++++++++++++++++++-------------
 packet.h     |  57 ++++++-------
 pcap.c       |   1 +
 tap.c        | 134 +++++++++++++++++-------------
 tap.h        |   4 +-
 tcp.c        |  61 ++++++++++----
 tcp_buf.c    |   2 +-
 udp.c        |  33 +++++---
 vhost_user.c |  28 +++----
 virtio.c     |   4 +-
 virtio.h     |  18 +++-
 vu_common.c  |  48 ++++-------
 26 files changed, 799 insertions(+), 437 deletions(-)

-- 
2.49.0



             reply	other threads:[~2025-06-04 13:08 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-04 13:08 Laurent Vivier [this message]
2025-06-04 13:08 ` [PATCH v6 01/30] arp: Don't mix incoming and outgoing buffers Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 02/30] iov: Introduce iov_tail_clone() and iov_tail_drop() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 03/30] iov: Update IOV_REMOVE_HEADER() and IOV_PEEK_HEADER() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 04/30] tap: Use iov_tail with tap_add_packet() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 05/30] packet: Use iov_tail with packet_add() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 06/30] packet: Add packet_data() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 07/30] arp: Convert to iov_tail Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 08/30] ndp: " Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 09/30] icmp: " Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 10/30] udp: " Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 11/30] tcp: Convert tcp_tap_handler() to use iov_tail Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 12/30] tcp: Convert tcp_data_from_tap() " Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 13/30] dhcpv6: move offset initialization out of dhcpv6_opt() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 14/30] dhcpv6: Extract sending of NotOnLink status Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 15/30] dhcpv6: Convert to iov_tail Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 16/30] dhcpv6: Use iov_tail in dhcpv6_opt() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 17/30] dhcp: Convert to iov_tail Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 18/30] ip: Use iov_tail in ipv6_l4hdr() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 19/30] tap: Convert tap4_handler() to iov_tail Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 20/30] tap: Convert tap6_handler() " Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 21/30] packet: rename packet_data() to packet_get() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 22/30] arp: use iov_tail rather than pool Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 23/30] dhcp: " Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 24/30] dhcpv6: " Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 25/30] icmp: " Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 26/30] ndp: " Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 27/30] packet: remove PACKET_POOL() and PACKET_POOL_P() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 28/30] packet: remove unused parameter from PACKET_POOL_DECL() Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 29/30] packet: add memory regions information into pool Laurent Vivier
2025-06-04 13:08 ` [PATCH v6 30/30] packet: use buf to store iovec array Laurent Vivier

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=20250604130834.3868010-1-lvivier@redhat.com \
    --to=lvivier@redhat.com \
    --cc=passt-dev@passt.top \
    /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).