[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 12/13] qemu-sockets: update socket_uri() to be consistent
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v6 12/13] qemu-sockets: update socket_uri() to be consistent with socket_parse() |
Date: |
Wed, 13 Jul 2022 18:26:46 +0100 |
User-agent: |
Mutt/2.2.6 (2022-06-05) |
On Wed, Jul 13, 2022 at 06:46:17PM +0200, Laurent Vivier wrote:
> On 12/07/2022 14:05, Dr. David Alan Gilbert wrote:
> > * Laurent Vivier (lvivier@redhat.com) wrote:
> > > Remove 'tcp:' prefix for inet type (because inet can be 'tcp' or 'udp'
> > > and socket_parse() doesn't recognize it), the format is 'host:port'.
> >
> > I don't think I understand why tests/qtest/migration-test.c
> > test_precopy_common is happy with this; it does:
> >
> > if (!args->connect_uri) {
> > g_autofree char *local_connect_uri =
> > migrate_get_socket_address(to, "socket-address");
> > migrate_qmp(from, local_connect_uri, "{}");
> >
> > which hmm, is the code you're changing what was in SocketAddress_to_str
> > which is what migrate_get_socket_address uses; but then the migrate_qmp
> > I don't think will take a migrate uri without the tcp: on the front.
>
> It's a good point.
>
> I think socket_parse() should accept the 'tcp:' prefix, and thus
> socket_uri() should generate it, so it will be usable with the qmp migrate
> command.
>
> I'm going to add 'tcp:' case in socket_parse() and make socket_uri() to
> generate it.
I'd say any code in util/qemu-sockets.c should only work in terms of
generic concepts. If we're formattting/parsing a migration URI, the
helper APIs should live in the migration/ subdir.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [PATCH v6 05/13] qapi: net: add stream and dgram netdevs, (continued)
- [PATCH v6 05/13] qapi: net: add stream and dgram netdevs, Laurent Vivier, 2022/07/06
- [PATCH v6 08/13] net: dgram: make dgram_dst generic, Laurent Vivier, 2022/07/06
- [PATCH v6 02/13] net: remove the @errp argument of net_client_inits(), Laurent Vivier, 2022/07/06
- [PATCH v6 07/13] net: stream: add unix socket, Laurent Vivier, 2022/07/06
- [PATCH v6 10/13] net: dgram: add unix socket, Laurent Vivier, 2022/07/06
- [PATCH v6 09/13] net: dgram: move mcast specific code from net_socket_fd_init_dgram(), Laurent Vivier, 2022/07/06
- [PATCH v6 12/13] qemu-sockets: update socket_uri() to be consistent with socket_parse(), Laurent Vivier, 2022/07/06
[PATCH v6 13/13] net: stream: move to QIO, Laurent Vivier, 2022/07/06
[PATCH v6 11/13] qemu-sockets: move and rename SocketAddress_to_str(), Laurent Vivier, 2022/07/06