[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/19] Migration patches
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PULL 00/19] Migration patches |
Date: |
Mon, 15 Jul 2019 16:10:44 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) |
Peter Maydell <address@hidden> wrote:
> On Mon, 15 Jul 2019 at 14:44, Juan Quintela <address@hidden> wrote:
>>
>> Peter Maydell <address@hidden> wrote:
>> > On Fri, 12 Jul 2019 at 17:33, Peter Maydell <address@hidden> wrote:
>> >> Still fails on aarch32 host, I'm afraid:
>>
>> Hi
>>
>> dropping the multifd test patch from now. For "some" reason, having a
>> packed struct and 32bits is getting ugly, not sure yet _why_.
>
> IMHO 'packed' structs are usually a bad idea. They have a bunch
> of behaviours you may not be expecting (for instance they're
> also not naturally aligned, and arrays of them won't be the
> size you expect).
I can't get everything happy O:-)
For the multifd initial packet, I used to have that I wrote the fields
by hand. Then danp asked that I used a packed struct, and converted the
values inside it. So ..... Imposible to have everybody happy.
Anyways, the struct is packed, both sides are i386 32bits, and it should
be exactly the same, but it appears that there is where your valgrind
problems appear. Still investigating _where_ the problem is. What is
even weirder is that there is no error at all on 64bits.
Thanks, Juan.
PS. BTW, did you launched by hand the guests with valgrind, or there is
a trick that I am missing for launching a qtest with valgrind?
- [Qemu-devel] [PULL 15/19] kvm: Persistent per kvmslot dirty bitmap, (continued)
- [Qemu-devel] [PULL 15/19] kvm: Persistent per kvmslot dirty bitmap, Juan Quintela, 2019/07/12
- [Qemu-devel] [PULL 16/19] kvm: Introduce slots lock for memory listener, Juan Quintela, 2019/07/12
- [Qemu-devel] [PULL 17/19] kvm: Support KVM_CLEAR_DIRTY_LOG, Juan Quintela, 2019/07/12
- [Qemu-devel] [PULL 18/19] migration: Split log_clear() into smaller chunks, Juan Quintela, 2019/07/12
- [Qemu-devel] [PULL 19/19] migration: allow private destination ram with x-ignore-shared, Juan Quintela, 2019/07/12
- Re: [Qemu-devel] [PULL 00/19] Migration patches, Peter Maydell, 2019/07/12
- Re: [Qemu-devel] [PULL 00/19] Migration patches, Dr. David Alan Gilbert, 2019/07/12
- Re: [Qemu-devel] [PULL 00/19] Migration patches, Peter Maydell, 2019/07/15
- Re: [Qemu-devel] [PULL 00/19] Migration patches, Juan Quintela, 2019/07/15
- Re: [Qemu-devel] [PULL 00/19] Migration patches, Peter Maydell, 2019/07/15
- Re: [Qemu-devel] [PULL 00/19] Migration patches,
Juan Quintela <=
- Re: [Qemu-devel] [PULL 00/19] Migration patches, Peter Maydell, 2019/07/15
- Re: [Qemu-devel] [PULL 00/19] Migration patches, Daniel P . Berrangé, 2019/07/15
- Re: [Qemu-devel] [PULL 00/19] Migration patches, Peter Maydell, 2019/07/15