Re: [PATCH 3/4] qemu snapshot: use QMP snapshot-save/delete for internal snapshots

On 7/16/24 00:42, Nikolai Barybin wrote: > The usage of HMP commands are highly discouraged by qemu. Moreover, > current snapshot creation routine does not provide flexibility in > choosing target device for VM state snapshot. > > This patch makes use of QMP commands snapshot-save/delete and by > default chooses first writable disk (if present) as target for VM > state, NVRAM - otherwise. > > Signed-off-by: Nikolai Barybin <nikolai.barybin(a)virtuozzo.com&gt; > --- > src/qemu/qemu_snapshot.c | 158 ++++++++++++++++++++++++++++++++++++--- > 1 file changed, 148 insertions(+), 10 deletions(-)

> @@ -342,15 +432,17 @@ qemuSnapshotCreateActiveInternal(virQEMUDriver *driver, > } > } > - if (qemuDomainObjEnterMonitorAsync(vm, VIR_ASYNC_JOB_SNAPSHOT) < 0) { > + if ((ret = qemuSnapshotCreateActiveInternalStart(vm, snap->def->name)) < 0) { > resume = false; > goto cleanup; > } > - ret = qemuMonitorCreateSnapshot(priv->mon, snap->def->name); > - qemuDomainObjExitMonitor(vm); > - if (ret < 0) > - goto cleanup; 'snapshot-save' has been added in QEMU 6.0. Right now we are at 8.2 Should we still support pre 6.0 versions, i.e. check the availability of the command via proper capability?