[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 4/8] replay: save/load initial state
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v3 4/8] replay: save/load initial state |
Date: |
Tue, 20 Sep 2016 14:52:12 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 |
On 20/09/2016 14:39, Pavel Dovgalyuk wrote:
> > > + replay_snapshot = g_strdup(qemu_opt_get(opts, "rrsnapshot"));
> > > +
> > > replay_enable(fname, mode);
> > >
> >
> > Should you set snapshot = 1 here if there is no rrsnapshot option?
>
> No, because there is default snapshot name for the case when user
> specifies overlay for the drives.
There are three possibilities:
a) these patches:
with implicit overlay:
-drive file=disk.raw,if=none,id=img-direct
-drive driver=blkreplay,if=none,image=img-direct,id=img-blkreplay
with explicit overlay:
-drive file=disk.raw,if=none,id=img-direct
-drive
driver=blkreplay,if=none,image=img-direct,id=img-blkreplay,overlay=foo.qcow2
Advantages:
- does the right thing in the "implicit overlay" case.
Disadvantages:
- no need really to specify disk.raw in the "explicit overlay" case, since
it's already specified when you create the overlay with qemu-img.
Vote for implicit overlay: excellent
Vote for explicit overlay: bad (need to track two file names)
b) always specify -snapshot manually:
with implicit overlay:
-drive file=disk.raw,if=none,id=img-direct -snapshot
-drive driver=blkreplay,if=none,image=img-direct,id=img-blkreplay
with explicit overlay:
-drive file=foo.qcow2,if=none,id=img-direct
-drive driver=blkreplay,if=none,image=img-direct,id=img-blkreplay
Advantages:
- can use default snapshot name in the "explicit overlay" case
Disadvantages:
- need to specify -snapshot explicitly for the "implicit overlay" case
Vote for implicit overlay: awful (disk.raw destroy if you forget -snapshot)
Vote for explicit overlay: excellent
c) no rrsnapshot implies -snapshot:
without overlay:
-drive file=disk.raw,if=none,id=img-direct
-drive driver=blkreplay,if=none,image=img-direct,id=img-blkreplay
with overlay:
-drive file=foo.qcow2,if=none,id=img-direct
-drive driver=blkreplay,if=none,image=img-direct,id=img-blkreplay
-icount ...,rrsnapshot=snapname
Advantages:
- does the right thing in the "implicit overlay" case.
Disadvantages:
- always have to specify snapshot name in the "explicit overlay" case
Vote for implicit overlay: excellent (same as a)
Vote for explicit overlay: decent
We have to choose between (a) and (c) I think, because the risk of corruption
for (b) is too high. I prefer (c). Kevin, what do you think?
Thanks,
Paolo
- [Qemu-devel] [PATCH v3 0/8] replay additions, Pavel Dovgalyuk, 2016/09/20
- [Qemu-devel] [PATCH v3 3/8] block: don't make snapshots for filters, Pavel Dovgalyuk, 2016/09/20
- [Qemu-devel] [PATCH v3 4/8] replay: save/load initial state, Pavel Dovgalyuk, 2016/09/20
- Re: [Qemu-devel] [PATCH v3 4/8] replay: save/load initial state, Paolo Bonzini, 2016/09/20
- Re: [Qemu-devel] [PATCH v3 4/8] replay: save/load initial state, Pavel Dovgalyuk, 2016/09/20
- Re: [Qemu-devel] [PATCH v3 4/8] replay: save/load initial state,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH v3 4/8] replay: save/load initial state, Pavel Dovgalyuk, 2016/09/20
- Re: [Qemu-devel] [PATCH v3 4/8] replay: save/load initial state, Paolo Bonzini, 2016/09/20
- Re: [Qemu-devel] [PATCH v3 4/8] replay: save/load initial state, Pavel Dovgalyuk, 2016/09/20
- Re: [Qemu-devel] [PATCH v3 4/8] replay: save/load initial state, Paolo Bonzini, 2016/09/20
[Qemu-devel] [PATCH v3 5/8] replay: move internal data to the structure, Pavel Dovgalyuk, 2016/09/20
[Qemu-devel] [PATCH v3 6/8] replay: vmstate for replay module, Pavel Dovgalyuk, 2016/09/20
[Qemu-devel] [PATCH v3 7/8] replay: allow replay stopping and restarting, Pavel Dovgalyuk, 2016/09/20
[Qemu-devel] [PATCH v3 8/8] integratorcp: adding vmstate for save/restore, Pavel Dovgalyuk, 2016/09/20
Re: [Qemu-devel] [PATCH v3 0/8] replay additions, no-reply, 2016/09/20