[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v12 08/21] migration: Transmit initial package t
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH v12 08/21] migration: Transmit initial package through the multifd channels |
Date: |
Wed, 09 May 2018 10:34:57 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
"Dr. David Alan Gilbert" <address@hidden> wrote:
> * Juan Quintela (address@hidden) wrote:
>> Signed-off-by: Juan Quintela <address@hidden>
>> Reviewed-by: Daniel P. Berrangé <address@hidden>
>
> Please print the magic with %x.
done.
>
>> + return -1;
>> + }
>> +
>> + if (msg.version != MULTIFD_VERSION) {
>> + error_setg(errp, "multifd: received packet version %d "
>> + "expected %d", msg.version, MULTIFD_VERSION);
>> + return -1;
>> + }
>> +
>> + if (memcmp(msg.uuid, &qemu_uuid, sizeof(qemu_uuid))) {
>> + char *uuid = qemu_uuid_unparse_strdup(&qemu_uuid);
>> + error_setg(errp, "multifd: received uuid '%s' and expected "
>> + "uuid '%s' for channel %hhd", msg.uuid, uuid, msg.id);
>
> I don't think you can just print msg.uuid there; it's the raw 16 bytes
> isn't it rather than a string?
You ae right.
> I suspect you need to do the unprase on both of them.
Done.
Thanks, Juan.