[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH for-2.11] qcow2: fix image corruption on commit

From: Kevin Wolf
Subject: Re: [Qemu-devel] [PATCH for-2.11] qcow2: fix image corruption on commit with persistent snapshot
Date: Fri, 17 Nov 2017 18:24:04 +0100
User-agent: Mutt/1.9.1 (2017-09-22)

Am 17.11.2017 um 18:07 hat Vladimir Sementsov-Ogievskiy geschrieben:
> 17.11.2017 19:47, Eric Blake wrote:
> > If an image contains persistent snapshots, we cannot use the
> bitmaps
> > fast path of bdrv_make_empty() to clear the image during
> > qemu-img commit, because that will lose the clusters related
> > to the bitmaps.
> > 
> > Also leave a comment in qcow2_read_extensions to remind future
> > feature additions to think about fast-path removal, since we
> > just barely fixed the same bug for LUKS encryption.
> > 
> > It's a pain that qemu-img has not yet been taught to manipulate,
> > or even at a very minimum display, information about persistent
> > bitmaps; instead, we have to use QMP commands.  It's also a
> > pain that only qeury-block and x-debug-block-dirty-bitmap-sha256
> > will allow bitmap introspection; but the former requires the
> > node to be hooked to a block device, and the latter is experimental.
> sorry for that pain =(.
> Honestly, I don't understand why such a simple and obvious fix needs an
> additional test.

Because we could otherwise accidentally break it again in the future.

If there is a feature that you care about, make sure to have a test for
everything in it, and that you add a regression test for every bug that
you fix.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]