[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 6/6] iotests: Test committing to short backing file
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [PATCH v2 6/6] iotests: Test committing to short backing file |
Date: |
Thu, 21 Nov 2019 09:36:27 +0000 |
21.11.2019 0:27, Eric Blake wrote:
> On 11/20/19 12:45 PM, Kevin Wolf wrote:
>> Signed-off-by: Kevin Wolf <address@hidden>
>> ---
>> tests/qemu-iotests/274 | 141 +++++++++++++++++++++
>> tests/qemu-iotests/274.out | 227 ++++++++++++++++++++++++++++++++++
>> tests/qemu-iotests/group | 1 +
>> tests/qemu-iotests/iotests.py | 2 +-
>> 4 files changed, 370 insertions(+), 1 deletion(-)
>> create mode 100755 tests/qemu-iotests/274
>> create mode 100644 tests/qemu-iotests/274.out
>>
>
>
>> + iotests.log('=== Testing QMP active commit (top -> mid) ===')
>> +
>> + create_chain()
>> + with create_vm() as vm:
>> + vm.launch()
>> + vm.qmp_log('block-commit', device='top', base_node='mid',
>> + job_id='job0', auto_dismiss=False)
>> + vm.run_job('job0', wait=5)
>> +
>> + iotests.img_info_log(mid)
>> + iotests.qemu_io_log('-c', 'read -P 1 0 %d' % size_short, mid)
>> + iotests.qemu_io_log('-c', 'read -P 0 %d %d' % (size_short, size_diff),
>> mid)
>> +
>
> Would it also be worth testing a commit of mid -> base, and showing that top
> still sees the same contents afterwards?
>
This is just still broken, see may "[PATCH 0/4] fix & merge block_status_above
and is_allocated_above" series.
--
Best regards,
Vladimir
- Re: [PATCH v2 2/6] block: truncate: Don't make backing file data visible, (continued)
Re: [PATCH for-4.2? v2 0/6] block: Fix resize (extending) of short overlays, Stefan Hajnoczi, 2019/11/21