<meta http-equiv="Content-Type" content="text/html; charset=utf-8"><div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:large"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">пт, 12 нояб. 2021 г. в 10:39, Peter Krempa <<a href="mailto:pkrempa@redhat.com">pkrempa@redhat.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Nov 11, 2021 at 11:55:45 +0300, Nikolay Shirokovskiy wrote:<br>
> Hi, everyone.<br>
> <br>
> The details for instant snapshot are given in main patch "qemu: support<br>
> instant snapshots". Here I only want to mention two moments.<br>
> <br>
> First as instant snapshot API is synchronous we need to add an event<br>
> that snapshot moment in time is passed so that client is free to make<br>
> changes to guest. This way client can make use of instant snapshot while<br>
> memory is still being written to disk.<br>
<br>
This is a nice replacement for --live. I had a dummy implementation for<br>
this but I put it aside for the following reasons.<br>
<br>
1) It doesn't work with all VM configs.<br>
<br>
My original approach wanted to use it as a full replacement for the<br>
snapshot code. That wouldn't work.<br>
<br>
Yours uses a new flag which is okay as it sidesteps the issue. (One<br>
detail, docs don't mention the caveat, more on that inline).<br>
<br>
2) The format change for instant load<br>
<br>
Below you mention that you want to change the image format for instant<br>
load. Introducing the feature now, without the image format change means<br>
that there will be a need for yet another flag switching to the new<br>
format.<br>
<br>
I'd strongly suggest you reconsider whether to push this now or just<br>
simply go with the full new format once it's ready in qemu.</blockquote><div><br></div><div><div class="gmail_default" style="font-size:large">Hi, Peter. </div><div class="gmail_default" style="font-size:large"><br></div><div class="gmail_default" style="font-size:large">Thank you for your review. Ok, let's wait for new format in qemu.</div></div><div class="gmail_default" style="font-size:large"><br></div><div class="gmail_default" style="font-size:large">Nikolay</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br>
<br>
> Second there is work in progress on adding asynchronous revert to<br>
> snapshot [1]. The idea is to write memory during snapshot in such a way<br>
> that it is possible to do a revert to snapshot in background. Namely<br>
> start guest on revert with only minimal amount of memory restored from<br>
> disk and then restore memory in background in postcopy migration<br>
> fashion. In order to make it possible snapshot memory should be written to<br>
> disk not just as plain migration stream. Thus the [1] patch adds<br>
> a special tool to write the migration stream during snapshot.<br>
> <br>
> So if/when patch [1] will be merged I plan to add 'instant revert' mode <br>
> to snapshot too.<br>
> <br>
> [1] [RFC PATCH v2 0/7] migration/snapshot: External snapshot utility<br>
> <a href="https://lists.nongnu.org/archive/html/qemu-devel/2021-05/msg03619.html" rel="noreferrer" target="_blank">https://lists.nongnu.org/archive/html/qemu-devel/2021-05/msg03619.html</a><br>
<br>
<br>
I'll add few review points inline.<br>
<br>
Also note that Pavel Hrdina might want to comment too as he is currently<br>
making good progress on making reversion of external snapshots work.<br>
<br>
</blockquote></div></div>