diff options
| author | Alberto Garcia | 2019-10-14 10:15:45 +0200 |
|---|---|---|
| committer | Kevin Wolf | 2019-10-14 17:12:48 +0200 |
| commit | f2208fdc5ba2f4d0cbc023e2530b5e3e110d71c2 (patch) | |
| tree | b182d432c52849da44e25597a2ffbfbd374b4908 /tests/qemu-iotests/268.out | |
| parent | replay: add BH oneshot event for block layer (diff) | |
| download | qemu-f2208fdc5ba2f4d0cbc023e2530b5e3e110d71c2.tar.gz qemu-f2208fdc5ba2f4d0cbc023e2530b5e3e110d71c2.tar.xz qemu-f2208fdc5ba2f4d0cbc023e2530b5e3e110d71c2.zip | |
block: Reject misaligned write requests with BDRV_REQ_NO_FALLBACK
The BDRV_REQ_NO_FALLBACK flag means that an operation should only be
performed if it can be offloaded or otherwise performed efficiently.
However a misaligned write request requires a RMW so we should return
an error and let the caller decide how to proceed.
This hits an assertion since commit c8bb23cbdb if the required
alignment is larger than the cluster size:
qemu-img create -f qcow2 -o cluster_size=2k img.qcow2 4G
qemu-io -c "open -o driver=qcow2,file.align=4k blkdebug::img.qcow2" \
-c 'write 0 512'
qemu-io: block/io.c:1127: bdrv_driver_pwritev: Assertion `!(flags & BDRV_REQ_NO_FALLBACK)' failed.
Aborted
The reason is that when writing to an unallocated cluster we try to
skip the copy-on-write part and zeroize it using BDRV_REQ_NO_FALLBACK
instead, resulting in a write request that is too small (2KB cluster
size vs 4KB required alignment).
Signed-off-by: Alberto Garcia <berto@igalia.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'tests/qemu-iotests/268.out')
| -rw-r--r-- | tests/qemu-iotests/268.out | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/tests/qemu-iotests/268.out b/tests/qemu-iotests/268.out new file mode 100644 index 0000000000..2ed6c68529 --- /dev/null +++ b/tests/qemu-iotests/268.out @@ -0,0 +1,7 @@ +QA output created by 268 + +== Required alignment larger than cluster size == +Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=1048576 +wrote 512/512 bytes at offset 0 +512 bytes, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) +*** done |