qemu-devel
[Top][All Lists]
Advanced

[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.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]