[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH v3 00/11] qapi: net: add unix socket type support to netd
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [RFC PATCH v3 00/11] qapi: net: add unix socket type support to netdev backend |
Date: |
Mon, 20 Jun 2022 19:24:19 +0100 |
User-agent: |
Mutt/2.2.5 (2022-05-16) |
* 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).
Had you considered a -netdev chardev?
Dave
> Some examples of CLI syntax:
>
> for TCP:
>
> -netdev stream,id=socket0,addr.type=inet,addr.host=localhost,addr.port=1234
> -netdev
> stream,id=socket0,server=off,addr.type=inet,addr.host=localhost,addr.port=1234
>
> -netdev dgram,id=socket0,\
> local.type=inet,local.host=localhost,local.port=1234,\
> remote.type=inet,remote.host=localhost,remote.port=1235
>
> for UNIX:
>
> -netdev stream,id=socket0,addr.type=unix,addr.path=/tmp/qemu0
> -netdev stream,id=socket0,server=off,addr.type=unix,addr.path=/tmp/qemu0
>
> -netdev dgram,id=socket0,\
> local.type=unix,local.path=/tmp/qemu0,\
> remote.type=unix,remote.path=/tmp/qemu1
>
> for FD:
>
> -netdev stream,id=socket0,addr.type=fd,addr.str=4
> -netdev stream,id=socket0,server=off,addr.type=fd,addr.str=5
>
> -netdev dgram,id=socket0,local.type=fd,addr.str=4
>
> v3:
> - remove support of "-net" for dgram and stream. They are only
> supported with "-netdev" option.
> - use &error_fatal directly in net_client_inits()
> - update qemu-options.hx
> - move to QIO for stream socket
>
> v2:
> - use "stream" and "dgram" rather than "socket-ng,mode=stream"
> and ""socket-ng,mode=dgram"
> - extract code to bypass qemu_opts_parse_noisily() to
> a new patch
> - do not ignore EINVAL (Stefano)
> - fix "-net" option
>
> CC: Ralph Schmieder <ralph.schmieder@gmail.com>
> CC: Stefano Brivio <sbrivio@redhat.com>
> CC: Daniel P. Berrangé <berrange@redhat.com>
> CC: Markus Armbruster <armbru@redhat.com>
>
> Laurent Vivier (10):
> net: introduce convert_host_port()
> net: remove the @errp argument of net_client_inits()
> qapi: net: introduce a way to bypass qemu_opts_parse_noisily()
> qapi: net: add stream and dgram netdevs
> net: stream: add unix socket
> net: dgram: make dgram_dst generic
> net: dgram: move mcast specific code from net_socket_fd_init_dgram()
> net: dgram: add unix socket
> qemu-sockets: introduce socket_uri()
> net: stream: move to QIO
>
> Stefano Brivio (1):
> net: stream: Don't ignore EINVAL on netdev socket connection
>
> hmp-commands.hx | 2 +-
> include/net/net.h | 3 +-
> include/qemu/sockets.h | 4 +-
> monitor/hmp-cmds.c | 23 +-
> net/clients.h | 6 +
> net/dgram.c | 706 +++++++++++++++++++++++++++++++++++++++++
> net/hub.c | 2 +
> net/meson.build | 2 +
> net/net.c | 149 ++++++---
> net/stream.c | 371 ++++++++++++++++++++++
> qapi/net.json | 40 ++-
> qemu-options.hx | 12 +
> softmmu/vl.c | 5 +-
> util/qemu-sockets.c | 20 ++
> 14 files changed, 1277 insertions(+), 68 deletions(-)
> create mode 100644 net/dgram.c
> create mode 100644 net/stream.c
>
> --
> 2.36.1
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
- [RFC PATCH v3 04/11] qapi: net: add stream and dgram netdevs, (continued)
- [RFC PATCH v3 08/11] net: dgram: move mcast specific code from net_socket_fd_init_dgram(), Laurent Vivier, 2022/06/20
- [RFC PATCH v3 10/11] qemu-sockets: introduce socket_uri(), Laurent Vivier, 2022/06/20
- [RFC PATCH v3 07/11] net: dgram: make dgram_dst generic, Laurent Vivier, 2022/06/20
- [RFC PATCH v3 06/11] net: stream: add unix socket, Laurent Vivier, 2022/06/20
- [RFC PATCH v3 09/11] net: dgram: add unix socket, Laurent Vivier, 2022/06/20
- [RFC PATCH v3 11/11] net: stream: move to QIO, Laurent Vivier, 2022/06/20
- Re: [RFC PATCH v3 00/11] qapi: net: add unix socket type support to netdev backend,
Dr. David Alan Gilbert <=