public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
* [PATCH v10 00/12] Add vhost-user support to passt. (part 3)
@ 2024-11-08 10:37 Laurent Vivier
  2024-11-08 10:37 ` [PATCH v10 01/12] packet: replace struct desc by struct iovec Laurent Vivier
                   ` (12 more replies)
  0 siblings, 13 replies; 19+ messages in thread
From: Laurent Vivier @ 2024-11-08 10:37 UTC (permalink / raw)
  To: passt-dev; +Cc: Laurent Vivier

This series of patches adds vhost-user support to passt
and then allows passt to connect to QEMU network backend using
virtqueue rather than a socket.

With QEMU, rather than using to connect:

  -netdev stream,id=s,server=off,addr.type=unix,addr.path=/tmp/passt_1.socket

we will use:

  -chardev socket,id=chr0,path=/tmp/passt_1.socket
  -netdev vhost-user,id=netdev0,chardev=chr0
  -device virtio-net,netdev=netdev0
  -object memory-backend-memfd,id=memfd0,share=on,size=$RAMSIZE
  -numa node,memdev=memfd0

The memory backend is needed to share data between passt and QEMU.

Performance comparison between "-netdev stream" and "-netdev vhost-user":

$ iperf3 -c localhost -p 10001  -t 60 -6 -u -b 50G

socket:
[  5]   0.00-60.05  sec  95.6 GBytes  13.7 Gbits/sec  0.017 ms  6998988/10132413 (69%)  receiver
vhost-user:
[  5]   0.00-60.04  sec   237 GBytes  33.9 Gbits/sec  0.006 ms  53673/7813770 (0.69%)  receiver

$ iperf3 -c localhost -p 10001  -t 60 -4 -u -b 50G

socket:
[  5]   0.00-60.05  sec  98.9 GBytes  14.1 Gbits/sec  0.018 ms  6260735/9501832 (66%)  receiver
vhost-user:
[  5]   0.00-60.05  sec   235 GBytes  33.7 Gbits/sec  0.008 ms  37581/7752699 (0.48%)  receiver

$ iperf3 -c localhost -p 10001  -t 60 -6

socket:
[  5]   0.00-60.00  sec  17.3 GBytes  2.48 Gbits/sec    0             sender
[  5]   0.00-60.06  sec  17.3 GBytes  2.48 Gbits/sec                  receiver
vhost-user:
[  5]   0.00-60.00  sec   191 GBytes  27.4 Gbits/sec    0             sender
[  5]   0.00-60.05  sec   191 GBytes  27.3 Gbits/sec                  receiver

$ iperf3 -c localhost -p 10001  -t 60 -4

socket:
[  5]   0.00-60.00  sec  15.6 GBytes  2.24 Gbits/sec    0             sender
[  5]   0.00-60.06  sec  15.6 GBytes  2.24 Gbits/sec                  receiver
vhost-user:
[  5]   0.00-60.00  sec   189 GBytes  27.1 Gbits/sec    0             sender
[  5]   0.00-60.04  sec   189 GBytes  27.0 Gbits/sec                  receiver

v10:
 - rebase v9 on top of my changes
 - remove last 4 patches from v9 as
   'tcp: Pass TCP header and payload separately to tcp_fill_headers[46]()'
   introduces a regression in "make check" for me.
 - addressed comments from David
 - I tried to cleanup iov management, but I was not able to remove the
   update of the first iov to point to the header or to the data
     - upd_vu_hdrlen()/tcp_vu_hdrlen() includes now the size of the
       virtio-net header
 - I didn't address comments from Stefano.
 - I didn't fix the bug with seek_offset_cap

v9: [David Gibson]
 - Rebased on current main
   - Conflicts with v4/v6 buffer merge addressed
   - Conflicts with TCP options construction rework addressed
 - Added several cleanup patches on top
 - Added a number of IOV and buffer cleanups on top
   - The aim is that these should allow more sharing of logic between
     the vhost-user and non-vhost-user pathes, although they've only
     minimally accomplished that so far.
v8:
  - remove iov_size() from vu_collect_one_frame()
  - move vu_packet_check_range() to vu_common.c
  - fix UDP when dlen is 0.

v7:
  - rebase
  - use vu_collect_one_frame() to do vu_collect() (collect multiple frame)
  - add vhost-user tests from Stefano

v6:
  - rebase
  - extract 3 patches from "vhost-user: add vhost-user":
      passt: rename tap_sock_init() to tap_backend_init()
      tcp: Export headers functions
      udp: Prepare udp.c to be shared with vhost-user
  - introduce new functions vu_collect_one_frame(),
    vu_collect(), vu_set_vnethdr(), vu_flush(), vu_send_single()
    to be called from tcp_vu.c, udp_vu.c and ICMP/DHCP where vhost-user
    code was duplicated.

v5:
  - rebase on top of 2024_09_06.6b38f07
  - rework udp_vu.c as ref.udp.v6 has been removed and we need to
    know if we receive IPv4 or IPv6 frame when we prepare the
    guest buffers for recvmsg()
  - remove vnet->hdrlen as the size is always the same with virtio-net v1
  - address comments from David and Stefano

v4:
  - rebase on top of 2024_08_21.1d6142f
    (rebasing on top of 620e19a1b48a ("udp: Merge udp[46]_mh_recv arrays")
     introduces a regression in the measure of the latency with UDP
     because I think I don't replace correctly ref.udp.v6 that is removed
     by this commit)
  - Addressed most of the comments from David and Stefano
    (I didn't want to postpone this version to next week,
     so I'll address the remaining comments in the next version).

v3:
  - rebase on top of flow table
  - update tcp_vu.c to look like udp_vu.c (recv()/prepare()/send_frame())
  - address comments from Stefano and David on version 2

v2:
  - remove PATCH 4
  - rewrite PATCH 2 and 3 to follow passt coding style
  - move some code from PATCH 3 to PATCH 4 (previously PATCH 5)
  - partially addressed David's comment on PATCH 5

David Gibson (4):
  tcp: Use only netinet/tcp.h instead of linux/tcp.h
  tcp_vu: Share more header construction between IPv4 and IPv6 paths
  tcp: Move tcp_l2_buf_fill_headers() to tcp_buf.c
  tcp: Adjust iov_len before filling headers

Laurent Vivier (7):
  packet: replace struct desc by struct iovec
  vhost-user: introduce virtio API
  vhost-user: introduce vhost-user API
  udp: Prepare udp.c to be shared with vhost-user
  tcp: Export headers functions
  passt: rename tap_sock_init() to tap_backend_init()
  vhost-user: add vhost-user

Stefano Brivio (1):
  test: Add tests for passt in vhost-user mode

 Makefile               |   9 +-
 conf.c                 |  21 +-
 epoll_type.h           |   4 +
 iov.c                  |   1 -
 isolation.c            |  17 +-
 packet.c               |  91 ++--
 packet.h               |  22 +-
 passt.1                |  10 +-
 passt.c                |  11 +-
 passt.h                |   7 +
 pcap.c                 |   1 -
 tap.c                  | 129 ++++--
 tap.h                  |   7 +-
 tcp.c                  |  76 +---
 tcp_buf.c              |  41 +-
 tcp_internal.h         |  19 +-
 tcp_vu.c               | 498 +++++++++++++++++++++
 tcp_vu.h               |  12 +
 test/lib/perf_report   |  15 +
 test/lib/setup         |  77 +++-
 test/lib/setup_ugly    |   2 +-
 test/passt_vu          |   1 +
 test/passt_vu_in_ns    |   1 +
 test/perf/passt_vu_tcp | 211 +++++++++
 test/perf/passt_vu_udp | 159 +++++++
 test/run               |  25 ++
 test/two_guests_vu     |   1 +
 udp.c                  |  85 ++--
 udp_internal.h         |  34 ++
 udp_vu.c               | 336 ++++++++++++++
 udp_vu.h               |  13 +
 util.h                 |   8 +
 vhost_user.c           | 981 +++++++++++++++++++++++++++++++++++++++++
 vhost_user.h           | 206 +++++++++
 virtio.c               | 660 +++++++++++++++++++++++++++
 virtio.h               | 184 ++++++++
 vu_common.c            | 285 ++++++++++++
 vu_common.h            |  60 +++
 38 files changed, 4116 insertions(+), 204 deletions(-)
 create mode 100644 tcp_vu.c
 create mode 100644 tcp_vu.h
 create mode 120000 test/passt_vu
 create mode 120000 test/passt_vu_in_ns
 create mode 100644 test/perf/passt_vu_tcp
 create mode 100644 test/perf/passt_vu_udp
 create mode 120000 test/two_guests_vu
 create mode 100644 udp_internal.h
 create mode 100644 udp_vu.c
 create mode 100644 udp_vu.h
 create mode 100644 vhost_user.c
 create mode 100644 vhost_user.h
 create mode 100644 virtio.c
 create mode 100644 virtio.h
 create mode 100644 vu_common.c
 create mode 100644 vu_common.h

-- 
2.47.0



^ permalink raw reply	[flat|nested] 19+ messages in thread

end of thread, other threads:[~2024-11-14  4:40 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-11-08 10:37 [PATCH v10 00/12] Add vhost-user support to passt. (part 3) Laurent Vivier
2024-11-08 10:37 ` [PATCH v10 01/12] packet: replace struct desc by struct iovec Laurent Vivier
2024-11-08 10:37 ` [PATCH v10 02/12] vhost-user: introduce virtio API Laurent Vivier
2024-11-08 10:37 ` [PATCH v10 03/12] vhost-user: introduce vhost-user API Laurent Vivier
2024-11-08 10:37 ` [PATCH v10 04/12] udp: Prepare udp.c to be shared with vhost-user Laurent Vivier
2024-11-08 10:37 ` [PATCH v10 05/12] tcp: Export headers functions Laurent Vivier
2024-11-08 10:37 ` [PATCH v10 06/12] passt: rename tap_sock_init() to tap_backend_init() Laurent Vivier
2024-11-08 10:37 ` [PATCH v10 07/12] vhost-user: add vhost-user Laurent Vivier
2024-11-11  2:47   ` David Gibson
2024-11-13  7:13     ` Laurent Vivier
2024-11-14  4:33       ` David Gibson
2024-11-08 10:37 ` [PATCH v10 08/12] test: Add tests for passt in vhost-user mode Laurent Vivier
2024-11-11  2:48   ` David Gibson
2024-11-08 10:37 ` [PATCH v10 09/12] tcp: Use only netinet/tcp.h instead of linux/tcp.h Laurent Vivier
2024-11-08 10:37 ` [PATCH v10 10/12] tcp_vu: Share more header construction between IPv4 and IPv6 paths Laurent Vivier
2024-11-08 10:37 ` [PATCH v10 11/12] tcp: Move tcp_l2_buf_fill_headers() to tcp_buf.c Laurent Vivier
2024-11-08 10:37 ` [PATCH v10 12/12] tcp: Adjust iov_len before filling headers Laurent Vivier
2024-11-08 11:27 ` [PATCH v10 00/12] Add vhost-user support to passt. (part 3) Stefano Brivio
2024-11-08 12:51   ` Laurent Vivier

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).