[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 10/21] migration: Create socket-address parameter
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PULL 10/21] migration: Create socket-address parameter |
Date: |
Wed, 06 Mar 2019 06:21:51 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 3/5/19 12:15 PM, Dr. David Alan Gilbert (git) wrote:
>> From: Juan Quintela <address@hidden>
>>
>> It will be used to store the uri parameters. We want this only for
>> tcp, so we don't set it for other uris. We need it to know what port
>> is migration running.
>>
>> Reviewed-by: Dr. David Alan Gilbert <address@hidden>
>> Signed-off-by: Juan Quintela <address@hidden>
>>
>> --
>>
>
> This was not the usual '---' divider, and hence:
>
>> This used to be uri parameter, but it has so many troubles to
>> reproduce that it don't just make sense.
>>
>> This used to be a port parameter. I was asked to move to
>> SocketAddress, done.
>> I also merged the setting of the migration tcp port in this one
>> because now I need to free the address, and this makes it easier.
>> This used to be x-socket-address with a single direction, now it is a
>> list of addresses.
>> Move SocketAddress_to_str here. I used to try to generalize the one
>> in chardev/char-socket.c, but it is not worth it.
>>
>> Free string (eric)
>> Handle VSOCK address nicely (not that migration can use them yet).
>> Remove useless breaks (dave)
>> rename socket_address to socket_address_list to avoid confusion
>> Update to 4.0 (eric)
>> Put a comment indicating that there is a problem on the qapi
>> generator (markus).
>
> ...all this got included, even if it was perhaps not intended.
>
>> Message-Id: <address@hidden>
>>
>> Signed-off-by: Dr. David Alan Gilbert <address@hidden>
>
> At any rate, it certainly looks odd to split *-by: tags by so much text.
>
>> +++ b/qapi/sockets.json
>> @@ -152,3 +152,21 @@
>> 'unix': 'UnixSocketAddress',
>> 'vsock': 'VsockSocketAddress',
>> 'fd': 'String' } }
>> +
>> +##
>> +# @DummyStruct:
>> +#
>> +# Both block-core and migration needs SocketAddressList
>> +# I am open to comments about how to share it
>> +#
>> +# @dummy-list: A dummy list
>> +#
>> +# FIXME: This shouldn't be needed, but this struct has two users, and
>> +# current qapi generator generates it on the 1st place that uses it,
>> +# so the second user don't see it. Putting it here it is seen in both
>> +# sides.
>
> If Markus' pull request lands first, we don't need this.
> https://lists.gnu.org/archive/html/qemu-devel/2019-03/msg01185.html
It did: merge commit a3e3b0a7bd5. Please drop this hunk, either in a
respin or in a follow-up patch.
A respin would give you a chance to clean up the commit message.
- [Qemu-devel] [PULL 08/21] migration: Add capabilities validation, (continued)
- [Qemu-devel] [PULL 08/21] migration: Add capabilities validation, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 07/21] tests/migration-test: Add a test for ignore-shared capability, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 11/21] tests: Add basic migration precopy tcp test, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 09/21] tests: Add migration xbzrle test, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 13/21] bitmap: bitmap_count_one_with_offset, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 12/21] bitmap: fix bitmap_count_one, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 14/21] migration: use bitmap_mutex in migration_bitmap_clear_dirty, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 15/21] migration: API to clear bits of guest free pages from the dirty bitmap, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 10/21] migration: Create socket-address parameter, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 16/21] migration/ram.c: add a notifier chain for precopy, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 17/21] migration/ram.c: add the free page optimization enable flag, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 19/21] Migration/colo.c: Fix double close bug when occur COLO failover, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 18/21] virtio-balloon: VIRTIO_BALLOON_F_FREE_PAGE_HINT, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 20/21] Migration/colo.c: Make COLO node running after failover, Dr. David Alan Gilbert (git), 2019/03/05
- [Qemu-devel] [PULL 21/21] qapi/migration.json: Remove a variable that doesn't exist in example, Dr. David Alan Gilbert (git), 2019/03/05
- Re: [Qemu-devel] [PULL 00/21] migration queue, no-reply, 2019/03/05
- Re: [Qemu-devel] [PULL 00/21] migration queue, Eric Blake, 2019/03/05
- Re: [Qemu-devel] [PULL 00/21] migration queue, no-reply, 2019/03/05