Re: [libvirt PATCH v2 4/6] qemu_snapshot: correctly load the saved memory state file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Sep 01, 2023 at 10:32:15 +0200, Pavel Hrdina wrote:
> Original code assumed that the memory state file is only migration
> stream but it has additional metadata stored by libvirt. To correctly
> load the memory state file we need to reuse code that is used when
> restoring domain from saved image.
> 
> This duplicates some necessary parts of qemuSaveImageStartVM() because
> the external snapshot memory state is done by qemuSaveImageCreate().
> 
> Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> ---
>  src/qemu/qemu_snapshot.c | 115 +++++++++++++++++++++++++++++++++------
>  1 file changed, 97 insertions(+), 18 deletions(-)

Hmmm, in the end you were right. This version is more verbose and in
fact doesn't end up having two code paths for external and internal but
both call qemuProcessStart which still does everything together.

I'll re-visit v1. Since my main concern is not aleviated by this one and
I think v1 can be made tolerable by improving some function
documentation.

Regardless of the above:

Reviewed-by: Peter Krempa <pkrempa@xxxxxxxxxx>

as I went through this pathc already.




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux