[PATCH v7 00/31] Introduce discontiguous frames management

[Laurent Vivier <lvivier@redhat.com>]

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.

v7:
    - Add a patch to fix comment style of 'Return:'
    - Fix ignore_arp()/accept_arp()
    - Fix coverity error
    - Fix several comments

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 (31):
  style: Fix 'Return' comment style
  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 +-
 checksum.c   |   4 +-
 conf.c       |  12 +--
 dhcp.c       |  48 ++++++-----
 dhcp.h       |   2 +-
 dhcpv6.c     | 223 +++++++++++++++++++++++++++++++-------------------
 dhcpv6.h     |   2 +-
 flow.c       |   4 +-
 flow_table.h |   2 +-
 icmp.c       |  43 ++++++----
 icmp.h       |   2 +-
 inany.c      |   4 +-
 iov.c        | 118 ++++++++++++++++++++++----
 iov.h        |  58 +++++++++----
 ip.c         |  32 ++++----
 ip.h         |   3 +-
 lineread.c   |   2 +-
 ndp.c        |  16 +++-
 ndp.h        |   4 +-
 packet.c     | 227 ++++++++++++++++++++++++++++++++++++++-------------
 packet.h     |  57 ++++++-------
 pcap.c       |   1 +
 tap.c        | 134 +++++++++++++++++-------------
 tap.h        |   8 +-
 tcp.c        |  67 ++++++++++-----
 tcp_buf.c    |   2 +-
 tcp_splice.c |   4 +-
 tcp_vu.c     |   4 +-
 udp.c        |  37 ++++++---
 udp_vu.c     |   6 +-
 util.c       |   6 +-
 util.h       |   4 +-
 vhost_user.c |  72 ++++++++--------
 virtio.c     |   6 +-
 virtio.h     |  24 ++++--
 vu_common.c  |  48 ++++-------
 37 files changed, 870 insertions(+), 504 deletions(-)

-- 
2.49.0