[pve-devel] [PATCH] remove running from Storage and check it in QemuServer

Dietmar Maurer dietmar at proxmox.com
Thu May 7 06:28:14 CEST 2015

> I wonder if we could replace  qmp  "snapshot-drive" (from
> internal-snapshot-async.patch)
> by upstream qemu qmp method
> blockdev-snapshot-internal-sync
> http://git.qemu.org/?p=qemu.git;a=blob_plain;f=qmp-commands.hx;hb=HEAD
> One advantage is that multiple drive can be snapshotted in a transaction.
> from history we have done this snapshot-drive because no method existed at
> this time.
> I think we just need to keep savevm_start, savevm_stop from
> internal-snapshot-async.patch,
> to be able to save the vmstate to external file/volume.

So we still need the code to save the state asynchronously "process_savevm_co",
and also the code to load it "load_state_from_blockdev". So this would just
a few line of very simple code in qmp_snapshot_drive? And we still need to
external snapshots, so we do not gain much from that 
"multiple drive can be snapshotted in a transaction"
feature (we just need to add more special cases, which make code more complex)?

More information about the pve-devel mailing list