* [Qemu-devel] [PATCH] iotests: fix 254 for qcow (1)
@ 2019-06-05 15:26 Vladimir Sementsov-Ogievskiy
2019-06-05 15:33 ` Max Reitz
0 siblings, 1 reply; 3+ messages in thread
From: Vladimir Sementsov-Ogievskiy @ 2019-06-05 15:26 UTC (permalink / raw)
To: qemu-devel, qemu-block; +Cc: kwolf, mreitz
Qcow default cluster size is 4k, but default format of created overlay
image on snapshot operation is qcow2 with it's default cluster of 64k.
This leads to block-dirty-bitmap-merge fail when test run for qcow
format, as it can't merge bitmaps with different granularities.
Let's fix it by specifying strict granularity.
Reported-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
---
tests/qemu-iotests/254 | 5 +++--
tests/qemu-iotests/254.out | 3 ++-
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/tests/qemu-iotests/254 b/tests/qemu-iotests/254
index 33cb80a512..a8ea5818ff 100755
--- a/tests/qemu-iotests/254
+++ b/tests/qemu-iotests/254
@@ -29,7 +29,8 @@ qemu_img_create('-f', iotests.imgfmt, disk, str(size))
vm = iotests.VM().add_drive(disk, opts='node-name=base')
vm.launch()
-vm.qmp_log('block-dirty-bitmap-add', node='drive0', name='bitmap0')
+vm.qmp_log('block-dirty-bitmap-add', node='drive0', name='bitmap0',
+ granularity=65536)
vm.hmp_qemu_io('drive0', 'write 0 512K')
@@ -38,7 +39,7 @@ vm.qmp_log('transaction', indent=2, actions=[
'data': {'device': 'drive0', 'snapshot-file': top,
'snapshot-node-name': 'snap'}},
{'type': 'block-dirty-bitmap-add',
- 'data': {'node': 'snap', 'name': 'bitmap0'}},
+ 'data': {'node': 'snap', 'name': 'bitmap0', 'granularity': 65536}},
{'type': 'block-dirty-bitmap-merge',
'data': {'node': 'snap', 'target': 'bitmap0',
'bitmaps': [{'node': 'base', 'name': 'bitmap0'}]}}
diff --git a/tests/qemu-iotests/254.out b/tests/qemu-iotests/254.out
index d7394cf002..dc899016da 100644
--- a/tests/qemu-iotests/254.out
+++ b/tests/qemu-iotests/254.out
@@ -1,4 +1,4 @@
-{"execute": "block-dirty-bitmap-add", "arguments": {"name": "bitmap0", "node": "drive0"}}
+{"execute": "block-dirty-bitmap-add", "arguments": {"granularity": 65536, "name": "bitmap0", "node": "drive0"}}
{"return": {}}
{
"execute": "transaction",
@@ -14,6 +14,7 @@
},
{
"data": {
+ "granularity": 65536,
"name": "bitmap0",
"node": "snap"
},
--
2.18.0
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [Qemu-devel] [PATCH] iotests: fix 254 for qcow (1)
2019-06-05 15:26 [Qemu-devel] [PATCH] iotests: fix 254 for qcow (1) Vladimir Sementsov-Ogievskiy
@ 2019-06-05 15:33 ` Max Reitz
2019-06-05 15:47 ` Vladimir Sementsov-Ogievskiy
0 siblings, 1 reply; 3+ messages in thread
From: Max Reitz @ 2019-06-05 15:33 UTC (permalink / raw)
To: Vladimir Sementsov-Ogievskiy, qemu-devel, qemu-block; +Cc: kwolf
[-- Attachment #1: Type: text/plain, Size: 2930 bytes --]
On 05.06.19 17:26, Vladimir Sementsov-Ogievskiy wrote:
> Qcow default cluster size is 4k, but default format of created overlay
> image on snapshot operation is qcow2 with it's default cluster of 64k.
Then I wonder why we run this test even for anything but qcow2.
I forgot to mention that this test also fails for all image formats that
do not support image creation. So we need some form of
iotests.verify_image_format() anyway. If half of this test requires
qcow2, and the other half does not really care about the image format,
then why not just restrict it to qcow2 altogether.
(Or restrict it to all formats that support backing files, and then
explicitly specify the format when creating the snapshot?)
Max
> This leads to block-dirty-bitmap-merge fail when test run for qcow
> format, as it can't merge bitmaps with different granularities.
> Let's fix it by specifying strict granularity.
>
> Reported-by: Max Reitz <mreitz@redhat.com>
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
> ---
> tests/qemu-iotests/254 | 5 +++--
> tests/qemu-iotests/254.out | 3 ++-
> 2 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/tests/qemu-iotests/254 b/tests/qemu-iotests/254
> index 33cb80a512..a8ea5818ff 100755
> --- a/tests/qemu-iotests/254
> +++ b/tests/qemu-iotests/254
> @@ -29,7 +29,8 @@ qemu_img_create('-f', iotests.imgfmt, disk, str(size))
> vm = iotests.VM().add_drive(disk, opts='node-name=base')
> vm.launch()
>
> -vm.qmp_log('block-dirty-bitmap-add', node='drive0', name='bitmap0')
> +vm.qmp_log('block-dirty-bitmap-add', node='drive0', name='bitmap0',
> + granularity=65536)
>
> vm.hmp_qemu_io('drive0', 'write 0 512K')
>
> @@ -38,7 +39,7 @@ vm.qmp_log('transaction', indent=2, actions=[
> 'data': {'device': 'drive0', 'snapshot-file': top,
> 'snapshot-node-name': 'snap'}},
> {'type': 'block-dirty-bitmap-add',
> - 'data': {'node': 'snap', 'name': 'bitmap0'}},
> + 'data': {'node': 'snap', 'name': 'bitmap0', 'granularity': 65536}},
> {'type': 'block-dirty-bitmap-merge',
> 'data': {'node': 'snap', 'target': 'bitmap0',
> 'bitmaps': [{'node': 'base', 'name': 'bitmap0'}]}}
> diff --git a/tests/qemu-iotests/254.out b/tests/qemu-iotests/254.out
> index d7394cf002..dc899016da 100644
> --- a/tests/qemu-iotests/254.out
> +++ b/tests/qemu-iotests/254.out
> @@ -1,4 +1,4 @@
> -{"execute": "block-dirty-bitmap-add", "arguments": {"name": "bitmap0", "node": "drive0"}}
> +{"execute": "block-dirty-bitmap-add", "arguments": {"granularity": 65536, "name": "bitmap0", "node": "drive0"}}
> {"return": {}}
> {
> "execute": "transaction",
> @@ -14,6 +14,7 @@
> },
> {
> "data": {
> + "granularity": 65536,
> "name": "bitmap0",
> "node": "snap"
> },
>
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [Qemu-devel] [PATCH] iotests: fix 254 for qcow (1)
2019-06-05 15:33 ` Max Reitz
@ 2019-06-05 15:47 ` Vladimir Sementsov-Ogievskiy
0 siblings, 0 replies; 3+ messages in thread
From: Vladimir Sementsov-Ogievskiy @ 2019-06-05 15:47 UTC (permalink / raw)
To: Max Reitz, qemu-devel@nongnu.org, qemu-block@nongnu.org; +Cc: kwolf@redhat.com
05.06.2019 18:33, Max Reitz wrote:
> On 05.06.19 17:26, Vladimir Sementsov-Ogievskiy wrote:
>> Qcow default cluster size is 4k, but default format of created overlay
>> image on snapshot operation is qcow2 with it's default cluster of 64k.
>
> Then I wonder why we run this test even for anything but qcow2.
>
> I forgot to mention that this test also fails for all image formats that
> do not support image creation. So we need some form of
> iotests.verify_image_format() anyway. If half of this test requires
> qcow2, and the other half does not really care about the image format,
> then why not just restrict it to qcow2 altogether.
Hmm, then better restrict to qcow2 ofcourse.
>
> (Or restrict it to all formats that support backing files, and then
> explicitly specify the format when creating the snapshot?)
>
> Max
>
>> This leads to block-dirty-bitmap-merge fail when test run for qcow
>> format, as it can't merge bitmaps with different granularities.
>> Let's fix it by specifying strict granularity.
>>
>> Reported-by: Max Reitz <mreitz@redhat.com>
>> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
>> ---
>> tests/qemu-iotests/254 | 5 +++--
>> tests/qemu-iotests/254.out | 3 ++-
>> 2 files changed, 5 insertions(+), 3 deletions(-)
>>
>> diff --git a/tests/qemu-iotests/254 b/tests/qemu-iotests/254
>> index 33cb80a512..a8ea5818ff 100755
>> --- a/tests/qemu-iotests/254
>> +++ b/tests/qemu-iotests/254
>> @@ -29,7 +29,8 @@ qemu_img_create('-f', iotests.imgfmt, disk, str(size))
>> vm = iotests.VM().add_drive(disk, opts='node-name=base')
>> vm.launch()
>>
>> -vm.qmp_log('block-dirty-bitmap-add', node='drive0', name='bitmap0')
>> +vm.qmp_log('block-dirty-bitmap-add', node='drive0', name='bitmap0',
>> + granularity=65536)
>>
>> vm.hmp_qemu_io('drive0', 'write 0 512K')
>>
>> @@ -38,7 +39,7 @@ vm.qmp_log('transaction', indent=2, actions=[
>> 'data': {'device': 'drive0', 'snapshot-file': top,
>> 'snapshot-node-name': 'snap'}},
>> {'type': 'block-dirty-bitmap-add',
>> - 'data': {'node': 'snap', 'name': 'bitmap0'}},
>> + 'data': {'node': 'snap', 'name': 'bitmap0', 'granularity': 65536}},
>> {'type': 'block-dirty-bitmap-merge',
>> 'data': {'node': 'snap', 'target': 'bitmap0',
>> 'bitmaps': [{'node': 'base', 'name': 'bitmap0'}]}}
>> diff --git a/tests/qemu-iotests/254.out b/tests/qemu-iotests/254.out
>> index d7394cf002..dc899016da 100644
>> --- a/tests/qemu-iotests/254.out
>> +++ b/tests/qemu-iotests/254.out
>> @@ -1,4 +1,4 @@
>> -{"execute": "block-dirty-bitmap-add", "arguments": {"name": "bitmap0", "node": "drive0"}}
>> +{"execute": "block-dirty-bitmap-add", "arguments": {"granularity": 65536, "name": "bitmap0", "node": "drive0"}}
>> {"return": {}}
>> {
>> "execute": "transaction",
>> @@ -14,6 +14,7 @@
>> },
>> {
>> "data": {
>> + "granularity": 65536,
>> "name": "bitmap0",
>> "node": "snap"
>> },
>>
>
>
--
Best regards,
Vladimir
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-06-05 15:48 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-06-05 15:26 [Qemu-devel] [PATCH] iotests: fix 254 for qcow (1) Vladimir Sementsov-Ogievskiy
2019-06-05 15:33 ` Max Reitz
2019-06-05 15:47 ` Vladimir Sementsov-Ogievskiy
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).