|
From: | Anton Ivanov |
Subject: | Re: [Qemu-devel] [PATCH 2/3] Unified Datagram Socket Transport - GRE support |
Date: | Sat, 22 Jul 2017 08:52:50 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
## If I read this right port is mandatory, correct?Okay, so it sounds like reusing InetSocket directly may not be possible. But there's still the interface question of whether we want dual 'ipv4' and 'ipv6' switches to allow finer-grain control over which (or both) families to be used.
I have that in the new version ready for submission. Behavior is identical with other arguments which have a v4 and v6 switch.
Concur - there are places where people rely that it has the port mandatory. Changing it to optional will break too much unrelated code.We may be able to do it if the port portion if InetSocketAddress becomes optional. There is no such thing as port for the protocols which use the raw families.We can always create a new QAPI type that expresses only the fields we need; I don't think InetSocketAddress should be changed to have an optional port just for your code additions.
-- Anton R. Ivanov Cambridge Greys Limited, England and Wales company No 10273661 http://www.cambridgegreys.com/
[Prev in Thread] | Current Thread | [Next in Thread] |