[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 13/20] iotests: 129: prepare for backup over block-copy

From: Vladimir Sementsov-Ogievskiy
Subject: Re: [PATCH v2 13/20] iotests: 129: prepare for backup over block-copy
Date: Mon, 9 Nov 2020 15:16:32 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0

04.11.2020 20:30, Max Reitz wrote:
On 22.10.20 23:10, Vladimir Sementsov-Ogievskiy wrote:
23.07.2020 11:03, Max Reitz wrote:
On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote:
After introducing parallel async copy requests instead of plain
cluster-by-cluster copying loop, backup job may finish earlier than
final assertion in do_test_stop. Let's require slow backup explicitly
by specifying speed parameter.

Isn’t the problem really that block_set_io_throttle does absolutely
nothing?  (Which is a long-standing problem with 129.  I personally just
never run it, honestly.)

Hmm.. is it better to drop test_drive_backup() from here ?

I think the best would be to revisit this:



I've checked, no, that doesn't help. I suppose that throttling has same defect 
original block-job speed: it calculates throttling after request, and if we 
simultaneously many requests they all will be handled (and after it we'll have 
throttling pause). New solution for backup in these series works better with 
requests (see patch [PATCH v2 07/20] block/block-copy: add ratelimit to 

So, I'd keep this patch for now.

Best regards,

reply via email to

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