Re: [libvirt PATCH 3/7] qemuSaveImageStartProcess: add snapshot argument
On Thu, Aug 31, 2023 at 16:55:02 +0200, Pavel Hrdina wrote:
When called from snapshot code we will need to pass snapshot object
in
order to make internal snapshots work correctly.
Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com>
---
src/qemu/qemu_saveimage.c | 6 ++++--
src/qemu/qemu_saveimage.h | 1 +
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_saveimage.c b/src/qemu/qemu_saveimage.c
index 1eedc900b9..73115af42d 100644
--- a/src/qemu/qemu_saveimage.c
+++ b/src/qemu/qemu_saveimage.c
@@ -572,6 +572,7 @@ qemuSaveImageOpen(virQEMUDriver *driver,
* @vm: domain object
* @fd: FD pointer of memory state file
* @path: path to memory state file
+ * @snapshot: snapshot to load when starting QEMU process or NULL
At this point the function comment must include what happens if you
provide it. And more specifically that user must not do that if they are
reloading an image.
Reviewed-by: Peter Krempa <pkrempa(a)redhat.com>