Re: [libvirt] [PATCH RESEND] qemu: Remove inactive vm when failed tostart it

2017-07-19 Thread John Ferlan
On 07/12/2017 07:34 AM, wang.y...@zte.com.cn wrote: > Hi John, > > Thanks for your review! > Somehow your response is out of synch with the rest of the series - things like this get lost very quickly. >> This seems to be a strange sequence of operations, but the claim is that > >> by adding

Re: [libvirt] [PATCH RESEND] qemu: Remove inactive vm when failed tostart it

2017-07-12 Thread wang.yi59
Hi John, Thanks for your review! > This seems to be a strange sequence of operations, but the claim is that > by adding this logic to CreateWithFlags, then the problem you're facing > is resolved. However, is adding this to the Create logic the right thing > to do? > IIUC: This condition is