[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters
From: |
Pavel Dovgalyuk |
Subject: |
Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters |
Date: |
Wed, 28 Sep 2016 12:32:52 +0300 |
> From: Kevin Wolf [mailto:address@hidden
> Am 27.09.2016 um 16:06 hat Pavel Dovgalyuk geschrieben:
> > > From: Kevin Wolf [mailto:address@hidden
> > > Am 26.09.2016 um 11:51 hat Pavel Dovgalyuk geschrieben:
> > > > > From: Kevin Wolf [mailto:address@hidden
> > > > > Am 26.09.2016 um 10:08 hat Pavel Dovgalyuk geschrieben:
> > > Originally, we only called bdrv_goto_snapshot() for all _top level_
> > > BDSes, and this is still what you normally get. However, if you
> > > explicitly create a BDS (e.g. with its own -drive option), it is
> > > considered a top level BDS without actually being top level for the
> > > guest, and therefore the snapshotting function is called for it.
> > >
> > > Of course, this is highly inefficient because the goto_snapshot request
> > > is passed by the filter driver and then called another time for the
> > > lower node, effectively loading the snapshot a second time.
Maybe double-saving/loading does the smallest damage then?
And we should just document how to use blkreplay effectively?
> > >
> > > On the other hand if you use a single -drive option to create both the
> > > qcow2 BDS and the blkreplay filter, we do need to pass down the
> > > goto_snapshot request because it won't be called for the qcow2 layer
> > > otherwise.
> >
> > How this can be specified in command line?
> > I believed that separate -drive option is required.
>
> Something like this:
>
> -drive driver=blkreplay,image.driver=file,image.filename=test.img
>
I tried the following command line, but VM does not detect the hard drive
and cannot boot.
-drive
driver=blkreplay,if=none,image.driver=file,image.filename=testdisk.qcow,id=img-blkreplay
-device ide-hd,drive=img-blkreplay
Pavel Dovgalyuk
- [Qemu-devel] [PATCH v5 4/9] record/replay: add network support, (continued)
- [Qemu-devel] [PATCH v5 4/9] record/replay: add network support, Pavel Dovgalyuk, 2016/09/26
- [Qemu-devel] [PATCH v5 5/9] savevm: add public save_vmstate function, Pavel Dovgalyuk, 2016/09/26
- [Qemu-devel] [PATCH v5 6/9] replay: save/load initial state, Pavel Dovgalyuk, 2016/09/26
- [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters, Pavel Dovgalyuk, 2016/09/26
- Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters, Kevin Wolf, 2016/09/26
- Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters, Pavel Dovgalyuk, 2016/09/26
- Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters, Kevin Wolf, 2016/09/26
- Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters, Pavel Dovgalyuk, 2016/09/27
- Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters, Kevin Wolf, 2016/09/28
- Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters,
Pavel Dovgalyuk <=
- Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters, Kevin Wolf, 2016/09/28
- Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters, Pavel Dovgalyuk, 2016/09/28
- Re: [Qemu-devel] [PATCH v5 7/9] block: don't make snapshots for filters, Kevin Wolf, 2016/09/29
[Qemu-devel] [PATCH v5 8/9] block: implement bdrv_recurse_is_first_non_filter for blkreplay, Pavel Dovgalyuk, 2016/09/26
[Qemu-devel] [PATCH v5 9/9] integratorcp: adding vmstate for save/restore, Pavel Dovgalyuk, 2016/09/26
Re: [Qemu-devel] [PATCH v5 0/9] replay additions, Paolo Bonzini, 2016/09/26