[libvirt] [PATCH 07/11] qemu: blockPivot: Don't pause the VM any more since we don't use drive-reopen

Peter Krempa pkrempa at redhat.com
Thu Apr 9 13:15:52 UTC 2015


On Wed, Apr 08, 2015 at 11:31:18 -0400, John Ferlan wrote:
> 
> 
> On 04/01/2015 01:20 PM, Peter Krempa wrote:
> > Support for drive-reopen was never present in the upstream code so we
> > don't need to pause the VM when doing the block pivot. Kill all the
> > code related to this semi-upstream artifact.
> > ---
> >  src/qemu/qemu_driver.c | 48 +++++-------------------------------------------
> >  1 file changed, 5 insertions(+), 43 deletions(-)
> > 
> 
> ACK - although it seems to be outside the realm of .0 "Refactor the
> block job code"...  IOW - should this have been a separate patch.

Separate as in a separate series? .. well it fixes block job code so I
think it belongs in this series.

I've pushed patches 2,3,4,6 and 7 and will follow up with the rest in
V3.

Thanks for the review.

Peter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150409/9254deba/attachment-0001.sig>


More information about the libvir-list mailing list