From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49473) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1csp4A-0007Vc-Bz for qemu-devel@nongnu.org; Tue, 28 Mar 2017 07:13:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1csp49-0004mU-Dw for qemu-devel@nongnu.org; Tue, 28 Mar 2017 07:13:34 -0400 Date: Tue, 28 Mar 2017 12:13:15 +0100 From: "Dr. David Alan Gilbert" Message-ID: <20170328111315.GF2509@work-vm> References: <20170225193155.447462-1-vsementsov@virtuozzo.com> <20170225193155.447462-4-vsementsov@virtuozzo.com> <20170307095323.GB5871@noname.str.redhat.com> <20170328105544.GE2509@work-vm> <20170328110900.GB11725@noname.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170328110900.GB11725@noname.redhat.com> Subject: Re: [Qemu-devel] [PATCH 3/4] savevm: fix savevm after migration List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf Cc: Vladimir Sementsov-Ogievskiy , qemu-block@nongnu.org, qemu-devel@nongnu.org, pbonzini@redhat.com, armbru@redhat.com, eblake@redhat.com, famz@redhat.com, stefanha@redhat.com, quintela@redhat.com, mreitz@redhat.com, peter.maydell@linaro.org, den@openvz.org, jsnow@redhat.com, lirans@il.ibm.com * Kevin Wolf (kwolf@redhat.com) wrote: > Am 28.03.2017 um 12:55 hat Dr. David Alan Gilbert geschrieben: > > * Kevin Wolf (kwolf@redhat.com) wrote: > > > Am 25.02.2017 um 20:31 hat Vladimir Sementsov-Ogievskiy geschrieben: > > > > After migration all drives are inactive and savevm will fail with > > > > > > > > qemu-kvm: block/io.c:1406: bdrv_co_do_pwritev: > > > > Assertion `!(bs->open_flags & 0x0800)' failed. > > > > > > > > Signed-off-by: Vladimir Sementsov-Ogievskiy > > > > > > What's the exact state you're in? I tried to reproduce this, but just > > > doing a live migration and then savevm on the destination works fine for > > > me. > > > > > > Hm... Or do you mean on the source? In that case, I think the operation > > > must fail, but of course more gracefully than now. > > > > > > Actually, the question that you're asking implicitly here is how the > > > source qemu process should be "reactivated" after a failed migration. > > > Currently, as far as I know, this is only with issuing a "cont" command. > > > It might make sense to provide a way to get control without resuming the > > > VM, but I doubt that adding automatic resume to every QMP command is the > > > right way to achieve it. > > > > > > Dave, Juan, what do you think? > > > > I'd only ever really thought of 'cont' or retrying the migration. > > However, it does make sense to me that you might want to do a savevm > > instead; if you can't migrate then perhaps a savevm is the best you > > can do before your machine dies. Are there any other things that > > should be allowed? > > I think we need to ask the other way round: Any reason _not_ to allow > certain operations that you can normally perform on a stopped VM? > > > We would want to be careful not to accidentally reactivate the disks > > on the source after what was actually a succesful migration. > > Yes, that's exactly my concern, even with savevm. That's why I suggested > we could have a 'cont'-like thing that just gets back control of the > images and moves into the normal paused state, but doesn't immediately > resume the actual VM. OK, lets say we had that block-reactivate (for want of a better name), how would we stop everything asserting if the user tried to do it before they'd run block-reactivate? Dave > Kevin -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK