On 03.06.2016 11:07, Fam Zheng wrote:
> Somehow in my locking series, I missed this case where concurrent access
> to an image is performed, perhaps we can remove this case independently.

The patch itself is good, but this commit message will not make much
sense in the git log. I'd be fine with removing it completely because
the title itself should be sufficient. (Which is something I can do in
case you agree.)

Max

> 
> Signed-off-by: Fam Zheng <f...@redhat.com>
> ---
>  tests/qemu-iotests/095 | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/qemu-iotests/095 b/tests/qemu-iotests/095
> index dad04b9..030adb2 100755
> --- a/tests/qemu-iotests/095
> +++ b/tests/qemu-iotests/095
> @@ -74,6 +74,8 @@ _send_qemu_cmd $h "{ 'execute': 'block-commit',
>                                   'arguments': { 'device': 'test',
>                                   'top': '"${TEST_IMG}.snp1"' } }" 
> "BLOCK_JOB_COMPLETED"
>  
> +_cleanup_qemu
> +
>  echo
>  echo "=== Base image info after commit and resize ==="
>  TEST_IMG="${TEST_IMG}.base" _img_info | _filter_img_info
> 


Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to