[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 2/8] tests/qemu-iotests/005: Add a sanity che
From: |
Alex Bennée |
Subject: |
Re: [Qemu-block] [PATCH v2 2/8] tests/qemu-iotests/005: Add a sanity check for large sparse file support |
Date: |
Wed, 01 May 2019 15:23:46 +0100 |
User-agent: |
mu4e 1.3.1; emacs 26.1 |
Thomas Huth <address@hidden> writes:
> On 01/05/2019 13.08, Alex Bennée wrote:
>>
>> Thomas Huth <address@hidden> writes:
>>
>>> "check -raw 005" fails when running on ext4 filesystems - these do not
>>> support such large sparse files. Use the same check as in test 220 to
>>> skip the test in this case.
>>>
>>> Suggested-by: Eric Blake <address@hidden>
>>> Signed-off-by: Thomas Huth <address@hidden>
>>> ---
>>> tests/qemu-iotests/005 | 9 +++++++++
>>> 1 file changed, 9 insertions(+)
>>>
>>> diff --git a/tests/qemu-iotests/005 b/tests/qemu-iotests/005
>>> index 2fef63af88..6136ced5c8 100755
>>> --- a/tests/qemu-iotests/005
>>> +++ b/tests/qemu-iotests/005
>>> @@ -55,6 +55,15 @@ if [ "$IMGPROTO" = "sheepdog" ]; then
>>> _notrun "image protocol $IMGPROTO does not support large image sizes"
>>> fi
>>>
>>> +# Sanity check: For raw, we require a file system that permits the creation
>>> +# of a HUGE (but very sparse) file. tmpfs works, ext4 does not.
>>
>> Is this comment correct? Running on my ext4 home partition this test
>> seems to be running fine and not skipping.
> Maybe it depends on the kernel version (I'm still using a 3.10-based
> kernel here) or the way how the file system was created? It fails at
> least for me on my ext4 partition.
> But I can adjust a little bit for sure.
# Sanity check: For raw, we require a file system that permits the
# creation of a HUGE (but very sparse) file. Check we can create such
# a file before continuing.
?
>
> Thomas
--
Alex Bennée