[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v9 12/12] migration: Add multifd test
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH v9 12/12] migration: Add multifd test |
Date: |
Mon, 11 Dec 2017 10:40:07 +0100 |
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:
>> We set the x-multifd-page-count and x-multifd-channels.
>>
>> Signed-off-by: Juan Quintela <address@hidden>
>
> <snip>
> We *must* find a way to share all the boiler plate I've snipped
> out; even this test_migrate function is almost identical to
> your other test series with just those few extra parameter sets.
See my migration-test series, I changed this a lot. I think that
without using function pointers, it is not easy to share more code that
what I did there (and I am not sure that using function pointers makes
things easier to understand).
>> + /* 200ms downtime */
>> + rsp = qmp("{ 'execute': 'migrate-set-parameters',"
>> + "'arguments': { 'downtime-limit': 300 } }");
>
> Note 200 vs 300 !
Fixed when I merged with the previous cod.
>> + /* Destination still running, wait for a byte to change */
>> + do {
>> + qtest_memread(to, start_address, &dest_byte_b, 1);
>> + usleep(10 * 1000);
>> + } while (dest_byte_a == dest_byte_b);
>
> Are there any multifd stats we can check? I guess the migration
> wont start until all channels are connected, so we know they're all
> in use in theory, so there's nothing specific.
We can check that all threads have been created, for instance.
Later, Juan.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v9 12/12] migration: Add multifd test,
Juan Quintela <=