[Qemu-devel] [PATCH] iotests: 122 needs at least two refcount bits now

Max Reitz posted 1 patch 5 years, 11 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180406151759.11223-1-mreitz@redhat.com
Test checkpatch passed
Test docker-build@min-glib passed
Test docker-mingw@fedora passed
Test s390x passed
tests/qemu-iotests/122 | 4 ++++
1 file changed, 4 insertions(+)
[Qemu-devel] [PATCH] iotests: 122 needs at least two refcount bits now
Posted by Max Reitz 5 years, 11 months ago
The new test case for compressed clusters (added in commit
abd3622cc03cf41ed542126a540385f30a4c0175) requires two refcount bits to
succeed, so we need to skip the test when refcount_bits=1 was requested.

Signed-off-by: Max Reitz <mreitz@redhat.com>
---
 tests/qemu-iotests/122 | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tests/qemu-iotests/122 b/tests/qemu-iotests/122
index 6cf4fcb866..95e7c51e72 100755
--- a/tests/qemu-iotests/122
+++ b/tests/qemu-iotests/122
@@ -42,6 +42,10 @@ _supported_fmt qcow2
 _supported_proto file
 _supported_os Linux
 
+# Repairing the compressed image requires qemu-img check to store a
+# refcount up to 3, which requires at least two refcount bits.
+_unsupported_imgopts 'refcount_bits=1[^0-9]'
+
 
 TEST_IMG="$TEST_IMG".base _make_test_img 64M
 $QEMU_IO -c "write -P 0x11 0 64M" "$TEST_IMG".base 2>&1 | _filter_qemu_io | _filter_testdir
-- 
2.14.3


Re: [Qemu-devel] [PATCH] iotests: 122 needs at least two refcount bits now
Posted by Eric Blake 5 years, 11 months ago
On 04/06/2018 10:17 AM, Max Reitz wrote:
> The new test case for compressed clusters (added in commit
> abd3622cc03cf41ed542126a540385f30a4c0175) requires two refcount bits to
> succeed, so we need to skip the test when refcount_bits=1 was requested.

We already questioned whether the changes to 122 were appropriate, given
that it didn't even use qemu-img convert.  Should we instead split this
into two tests (keeping 122 as it was before, and making the new
compressed refcount munging tests be separate), where only the new test
needs this filter?

> 
> Signed-off-by: Max Reitz <mreitz@redhat.com>
> ---
>  tests/qemu-iotests/122 | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/tests/qemu-iotests/122 b/tests/qemu-iotests/122
> index 6cf4fcb866..95e7c51e72 100755
> --- a/tests/qemu-iotests/122
> +++ b/tests/qemu-iotests/122
> @@ -42,6 +42,10 @@ _supported_fmt qcow2
>  _supported_proto file
>  _supported_os Linux
>  
> +# Repairing the compressed image requires qemu-img check to store a
> +# refcount up to 3, which requires at least two refcount bits.
> +_unsupported_imgopts 'refcount_bits=1[^0-9]'
> +

This makes sense, but I'm withholding R-b until we decide if splitting
the tests is wiser.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Re: [Qemu-devel] [PATCH] iotests: 122 needs at least two refcount bits now
Posted by Max Reitz 5 years, 11 months ago
On 2018-04-06 17:32, Eric Blake wrote:
> On 04/06/2018 10:17 AM, Max Reitz wrote:
>> The new test case for compressed clusters (added in commit
>> abd3622cc03cf41ed542126a540385f30a4c0175) requires two refcount bits to
>> succeed, so we need to skip the test when refcount_bits=1 was requested.
> 
> We already questioned whether the changes to 122 were appropriate, given
> that it didn't even use qemu-img convert.  Should we instead split this
> into two tests (keeping 122 as it was before, and making the new
> compressed refcount munging tests be separate), where only the new test
> needs this filter?

Can do, why not.

Max

>> Signed-off-by: Max Reitz <mreitz@redhat.com>
>> ---
>>  tests/qemu-iotests/122 | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/tests/qemu-iotests/122 b/tests/qemu-iotests/122
>> index 6cf4fcb866..95e7c51e72 100755
>> --- a/tests/qemu-iotests/122
>> +++ b/tests/qemu-iotests/122
>> @@ -42,6 +42,10 @@ _supported_fmt qcow2
>>  _supported_proto file
>>  _supported_os Linux
>>  
>> +# Repairing the compressed image requires qemu-img check to store a
>> +# refcount up to 3, which requires at least two refcount bits.
>> +_unsupported_imgopts 'refcount_bits=1[^0-9]'
>> +
> 
> This makes sense, but I'm withholding R-b until we decide if splitting
> the tests is wiser.
>