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 <[email protected]> > --- > 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 >
signature.asc
Description: OpenPGP digital signature
