On 13.09.19 10:58, Max Reitz wrote: > On 16.08.19 17:30, Vladimir Sementsov-Ogievskiy wrote: >> Hi all! >> >> Here is an asynchronous scheme for handling fragmented qcow2 >> reads and writes. Both qcow2 read and write functions loops through >> sequential portions of data. The series aim it to parallelize these >> loops iterations. >> It improves performance for fragmented qcow2 images, I've tested it >> as described below. > > Thanks, I’ve changed two things: > - Replaced assert((x & (BDRV_SECTOR_SIZE - 1)) == 0) by > assert(QEMU_IS_ALIGNED(x, BDRV_SECTOR_SIZE)) in patch 3 (conflict with > “block: Use QEMU_IS_ALIGNED”), and > - Replaced the remaining instance of “qcow2_co_do_pwritev()” by > “qcow2_co_pwritev_task()” in a comment in patch 4 > > and applied the series to my block branch: > > https://git.xanclic.moe/XanClic/qemu/commits/branch/block
Unfortunately, I’ll have to unstage the series for now because the fix to 026’s reference output isn’t stable. When running the test in parallel (I can reproduce it with four instances on my machine with two cores + HT), I get failures like: 026 fail [15:21:09] [15:21:37] (last: 18s) output mismatch (see 026.out.bad) --- tests/qemu-iotests/026.out 2019-09-16 14:49:20.720410701 +0200 +++ tests/qemu-iotests/026.out.bad 2019-09-16 15:21:37.180711936 +0200 @@ -563,7 +563,7 @@ qemu-io: Failed to flush the refcount block cache: No space left on device write failed: No space left on device -74 leaked clusters were found on the image. +522 leaked clusters were found on the image. This means waste of disk space, but no harm to data. Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=1073741824 Failures: 026 Failed 1 of 1 iotests Max
signature.asc
Description: OpenPGP digital signature
