From: Stefano Brivio <sbrivio@redhat.com>
To: Laurent Vivier <lvivier@redhat.com>
Cc: qemu-devel@nongnu.org, "Jason Wang" <jasowang@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Gregory Kurz" <gregory.kurz@free.fr>,
"Daniel P. Berrangé" <berrange@redhat.com>,
"Eric Blake" <eblake@redhat.com>,
"Markus Armbruster" <armbru@redhat.com>,
"Thomas Huth" <thuth@redhat.com>,
"David Gibson" <david@gibson.dropbear.id.au>,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>,
"Ralph Schmieder" <ralph.schmieder@gmail.com>
Subject: Re: [PATCH v8 00/14] qapi: net: add unix socket type support to netdev backend
Date: Tue, 13 Sep 2022 11:24:31 +0100 [thread overview]
Message-ID: <20220913112431.12d164de@elisabeth> (raw)
In-Reply-To: <20220913064000.79353-1-lvivier@redhat.com>
On Tue, 13 Sep 2022 08:39:46 +0200
Laurent Vivier <lvivier@redhat.com> wrote:
> "-netdev socket" only supports inet sockets.
>
> It's not a complex task to add support for unix sockets, but
> the socket netdev parameters are not defined to manage well unix
> socket parameters.
>
> As discussed in:
>
> "socket.c added support for unix domain socket datagram transport"
> https://lore.kernel.org/qemu-devel/1C0E1BC5-904F-46B0-8044-68E43E67BE60@gmail.com/
>
> This series adds support of unix socket type using SocketAddress QAPI structure.
>
> Two new netdev backends, "stream" and "dgram" are added, that are barely a copy of "socket"
> backend but they use the SocketAddress QAPI to provide socket parameters.
> And then they also implement unix sockets (TCP and UDP).
For the series,
Tested-by: Stefano Brivio <sbrivio@redhat.com>
...specifically, same as v7, the stream-oriented AF_UNIX support, with
passt (https://passt.top) and its test suite, running qemu (x86_64 KVM,
and armv6l, armv7l, aarch64, ppc64, ppc64le, s390x TCG) with something
on the lines of:
-device virtio-net-pci,netdev=socket0 -netdev stream,id=socket0,server=off,addr.type=unix,addr.path=...
--
Stefano
prev parent reply other threads:[~2022-09-13 10:28 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-13 6:39 [PATCH v8 00/14] qapi: net: add unix socket type support to netdev backend Laurent Vivier
2022-09-13 6:39 ` [PATCH v8 01/14] net: introduce convert_host_port() Laurent Vivier
2022-09-14 2:26 ` David Gibson
2022-09-13 6:39 ` [PATCH v8 02/14] net: remove the @errp argument of net_client_inits() Laurent Vivier
2022-09-14 2:28 ` David Gibson
2022-09-13 6:39 ` [PATCH v8 03/14] net: simplify net_client_parse() error management Laurent Vivier
2022-09-14 2:29 ` David Gibson
2022-09-13 6:39 ` [PATCH v8 04/14] qapi: net: introduce a way to bypass qemu_opts_parse_noisily() Laurent Vivier
2022-09-13 6:39 ` [PATCH v8 05/14] qapi: net: add stream and dgram netdevs Laurent Vivier
2022-09-14 5:03 ` David Gibson
2022-09-13 6:39 ` [PATCH v8 06/14] net: stream: Don't ignore EINVAL on netdev socket connection Laurent Vivier
2022-09-14 5:05 ` David Gibson
2022-09-23 14:58 ` Laurent Vivier
2022-09-13 6:39 ` [PATCH v8 07/14] net: stream: add unix socket Laurent Vivier
2022-09-14 5:16 ` David Gibson
2022-09-26 12:15 ` Laurent Vivier
2022-09-13 6:39 ` [PATCH v8 08/14] net: dgram: make dgram_dst generic Laurent Vivier
2022-09-13 6:39 ` [PATCH v8 09/14] net: dgram: move mcast specific code from net_socket_fd_init_dgram() Laurent Vivier
2022-09-13 6:39 ` [PATCH v8 10/14] net: dgram: add unix socket Laurent Vivier
2022-09-13 6:39 ` [PATCH v8 11/14] qemu-sockets: move and rename SocketAddress_to_str() Laurent Vivier
2022-09-14 5:23 ` David Gibson
2022-09-14 19:08 ` Dr. David Alan Gilbert
2022-09-13 6:39 ` [PATCH v8 12/14] qemu-sockets: update socket_uri() and socket_parse() to be consistent Laurent Vivier
2022-09-14 19:06 ` Dr. David Alan Gilbert
2022-09-21 15:01 ` Markus Armbruster
2022-09-13 6:39 ` [PATCH v8 13/14] net: stream: move to QIO Laurent Vivier
2022-09-21 14:58 ` Markus Armbruster
2022-09-22 9:13 ` Laurent Vivier
2022-09-22 9:40 ` Markus Armbruster
2022-09-13 6:40 ` [PATCH v8 14/14] tests/qtest: netdev: test stream and dgram backends Laurent Vivier
2022-09-13 10:24 ` Stefano Brivio [this message]
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=20220913112431.12d164de@elisabeth \
--to=sbrivio@redhat.com \
--cc=armbru@redhat.com \
--cc=berrange@redhat.com \
--cc=david@gibson.dropbear.id.au \
--cc=dgilbert@redhat.com \
--cc=eblake@redhat.com \
--cc=gregory.kurz@free.fr \
--cc=jasowang@redhat.com \
--cc=lvivier@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=ralph.schmieder@gmail.com \
--cc=thuth@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.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.