Re: [libvirt] [PATCH v2 8/8] qemu: Fix some corner cases in persistent migration

2015-09-18 Thread Jiri Denemark
On Thu, Sep 17, 2015 at 18:40:59 -0400, John Ferlan wrote: > > > On 09/11/2015 09:26 AM, Jiri Denemark wrote: > > When persistently migrating a domain to a destination host where the > > same domain already exists (i.e., it is persistent and shutdown at the > > destination), we would happily thro

Re: [libvirt] [PATCH v2 8/8] qemu: Fix some corner cases in persistent migration

2015-09-17 Thread John Ferlan
On 09/11/2015 09:26 AM, Jiri Denemark wrote: > When persistently migrating a domain to a destination host where the > same domain already exists (i.e., it is persistent and shutdown at the > destination), we would happily through away the original persistent s/through/throw > definition without

[libvirt] [PATCH v2 8/8] qemu: Fix some corner cases in persistent migration

2015-09-11 Thread Jiri Denemark
When persistently migrating a domain to a destination host where the same domain already exists (i.e., it is persistent and shutdown at the destination), we would happily through away the original persistent definition without properly freeing it. And when updating the definition fails for some rea