[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 5/6] tests: Add migration compress threads tests
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH 5/6] tests: Add migration compress threads tests |
Date: |
Wed, 18 Oct 2017 13:57:36 +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:
>> For some reason, compression is not working at the moment, test is
>> disabled until I found why.
>>
>> Signed-off-by: Juan Quintela <address@hidden>
>> ---
>> tests/migration-test.c | 51
>> ++++++++++++++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 51 insertions(+)
>>
>> diff --git a/tests/migration-test.c b/tests/migration-test.c
>> index f0734e4ba0..a8abc3d007 100644
>> --- a/tests/migration-test.c
>> +++ b/tests/migration-test.c
>> @@ -621,6 +621,54 @@ static void test_xbzrle_unix(void)
>> g_free(uri);
>> }
>>
>> +static void test_compress(const char *uri)
>> +{
>> + QTestState *from, *to;
>> +
>> + test_migrate_start(&from, &to, uri);
>> +
>> + /* We want to pick a speed slow enough that the test completes
>> + * quickly, but that it doesn't complete precopy even on a slow
>> + * machine, so also set the downtime.
>> + */
>> + /* 100 ms */
>> + migrate_set_parameter(from, "downtime-limit", "100");
>> + /* 1MB/s slow*/
>> + migrate_set_parameter(from, "max-bandwidth", "100000000");
>> +
>> + migrate_set_parameter(from, "compress-threads", "4");
>> + migrate_set_parameter(from, "decompress-threads", "3");
>
> Should that be 'to' ?
Yes.
> I worry about having this many threads in a loaded test environment.
Suggestion is?
I am not getting they working reliabely without having it loaded.
Later, Juan.
- Re: [Qemu-devel] [PATCH 1/6] tests: Add basic migration precopy test, (continued)