On Fri, Jul 09, 2021 at 09:33:50AM +0300, Vladimir Sementsov-Ogievskiy wrote: > > +++ b/tests/qemu-iotests/tests/qemu-img-bitmaps
> > +echo > > +echo "=== Check handling of inconsistent bitmap ===" > > +echo > > + > > +$QEMU_IO -c abort "$TEST_IMG" 2>/dev/null > > +$QEMU_IMG bitmap --add "$TEST_IMG" b4 > > +$QEMU_IMG bitmap --remove "$TEST_IMG" b1 > > +_img_info --format-specific | _filter_irrelevant_img_info > > +$QEMU_IMG convert --bitmaps -O qcow2 "$TEST_IMG" "$TEST_IMG.copy" > > Worth then removing remaining inconsistent bitmaps and try again? > > I think you should now remove $TEST_IMG.copy in _cleanup $TEST_IMG.copy isn't created on failure (or if it is, that in itself is a problem we should be avoiding), so as written, there was nothing that should have needed cleaning up until patch 2. But your idea (here and in patch 2) of demonstrating manual cleanup for recovery (in addition to the goal of patch 2 of skipping broken bitmaps in the first place) is reasonable, so I'll incorporate that into v2. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
