<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr"><br></div>
<br>
<br>
    On 2/3/20, 7:16 PM, "Daniel P. Berrangé" <<a href="mailto:berrange@redhat.com" target="_blank">berrange@redhat.com</a>> wrote:<br>
<br>
        On Mon, Feb 03, 2020 at 10:42:48AM -0300, Daniel Henrique Barboza wrote:<br>
        > Hi Daniel,<br>
        > <br>
        > I am happy that Libvirt is pushing local migration/live patching support, but<br>
        > at the same time I am wondering what changed from what you said here:<br>
<br>
        Err, this isn't libvirt pushing local migration. I'm simply re-posting<br>
        these patches on behalf of Shaju who is unable to post the patches due<br>
        to our broken mail server.  Don't take this as meaning that I approve of<br>
        the patches. They're simply here for discussion as any other patch<br>
        proposal is.<br>
<br>Thank you for forwarding the patch to the list, Danpb.<br><br>        <br>
<br>
        That is largely still my view.</div><div class="gmail_quote"><br></div><div class="gmail_quote">Sure, and we will be happy to discuss this further, as noted below :)<br>
<br>
        > To give you a background, we have live patching enhancements in IBM backlog<br>
        > since a few years ago, and one on the reasons these were being postponed<br>
        > time and time again were the lack of Libvirt support and this direction of<br>
        > "Libvirt is not interested in supporting it". And this message above was being<br>
        > used internally as the rationale for it.<br>
<br>       Hi Daniel HB,</div><div class="gmail_quote">       Thank you for pointing out the fact that this has been in discussion since 2013. While Shaju's patches were independent as an RFC, we will be happy to collaborate to push for a joint solution. The fact that this has been requested time and again, and the fact that most commercial cloud deployments out there already have an in-place upgrade story [1] [2] -- should be good reason we holistically examine the use case once again.</div><div class="gmail_quote">[1] <a href="https://kb.vmware.com/s/article/2005389">https://kb.vmware.com/s/article/2005389</a></div><div class="gmail_quote">[2] <a href="https://dl.acm.org/doi/10.1145/3297858.3304034">https://dl.acm.org/doi/10.1145/3297858.3304034</a></div><div class="gmail_quote"><br></div><div class="gmail_quote">Danpb had explained in much detail as to why mangling file and particularly socket paths can be messy in this patchset. However, even if  libvirtd blocks in-place migrations for such legacy VMs  until apps switch to more stringent XML semantics, it still may help cutting edge apps that intend to leverage this. </div><div class="gmail_quote"><br></div><div class="gmail_quote">I understand the presence of collision-causing file and socket paths can easily be checked as pre-migration checks, and should be trivial to implement. </div><div class="gmail_quote">We can include a revised patchset with this check in place. Support for this feature has been present in qemu for a while for this use-case, and so maybe it is time we pass on the goodness up the stack as well.</div><div class="gmail_quote"><br></div><div class="gmail_quote">Happy to discuss more details on implementation and semantics,</div><div class="gmail_quote"><br></div><div class="gmail_quote">Warm regards,</div><div class="gmail_quote">Prerna Saxena</div></div>