qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v5 05/12] qapi: net: add stream and dgram netdevs


From: Laurent Vivier
Subject: Re: [PATCH v5 05/12] qapi: net: add stream and dgram netdevs
Date: Thu, 30 Jun 2022 17:09:24 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.10.0

On 29/06/2022 13:20, Markus Armbruster wrote:
Laurent Vivier <lvivier@redhat.com> writes:

Copied from socket netdev file and modified to use SocketAddress
to be able to introduce new features like unix socket.

"udp" and "mcast" are squashed into dgram netdev, multicast is detected
according to the IP address type.
"listen" and "connect" modes are managed by stream netdev. An optional
parameter "server" defines the mode (server by default)

Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Reviewed-by: Stefano Brivio <sbrivio@redhat.com>
---

[...]
...
+# @server: create server socket (default: true)
+#
+# Since: 7.1
+##
+{ 'struct': 'NetdevStreamOptions',
+  'data': {
+    'addr':   'SocketAddress',
+    '*server': 'bool' } }
+
+##
+# @NetdevDgramOptions:
+#
+# Configuration info for datagram socket netdev.
+#
+# @remote: remote address
+# @local: local address
+#
+# The code checks there is at least one of these options and reports an error
+# if not. If remote address is present and it's a multicast address, local
+# address is optional. Otherwise local address is required and remote address
+# is optional.
+#
+# Since: 7.1
+##
+{ 'struct': 'NetdevDgramOptions',
+  'data': {
+    '*local':  'SocketAddress',
+    '*remote': 'SocketAddress' } }

Hard to see, but the space in "} }" is funny: it's U+00A0
(NO-BREAK-SPACE) encoded in UTF-8.  Make it a plain old space, please.


I'm sorry, this happens sometime because I use a french macintosh keyboard, and to do '}' I have to press alt+) and if I don't release fast enough 'alt' I produce an alt+space that seems to generate this (invisible) character code...
(I've the same problem with '|' = alt+shift+l, but bash doesn't like it)

I'm updating the series with all your comments.

Thank you for the reviews.

Laurent




reply via email to

[Prev in Thread] Current Thread [Next in Thread]