[libvirt] [PATCH v2 0/12] migration: support all toURI and proto combos

Daniel P. Berrange berrange at redhat.com
Thu Sep 17 12:11:59 UTC 2015

On Thu, Sep 10, 2015 at 04:20:12PM +0300, Nikolay Shirokovskiy wrote:
> Current implementation of 'toURI' migration interfaces does not support all
> combinations of interface versions and protocol versions. For example 'toURI2'
> with p2p flag will not migrate if driver supports only v3params proto.
> This is not convinient as drivers that starts to support migration have to
> manually support older versions of protocol. I guess this should be done in
> one place, namely here.
> Another issue is that there are a lot of code duplication in implementation of
> toURI interfaces and it is not obvious from code how are they related.
> This implementation uses extensible parameters as intermediate parameters
> representation. This is possible as interfaces are done backward compatible in
> terms of parameters and later versions supports all parameters of former
> versions.
> = Changes from version1
> Patch is splitted into a set. Quite a big one as a result of the following strategy:
> 1. each change in behaviour even subtle one deserves a separate patch. One
>    patch changes one aspect in behaviour.
> 2. separate pure refactoring steps into patches too as rather simple refactor
>    steps could introduce many line changes. Mark such patches with 'refactor:'
> Now every patch is easy to grasp I think.
> The resulted cumulative patch is slightly different from first in behaviour but
> I'm not going to describe the differece here as original patch was not reviewed
> in details by anyone anyway )
>  src/libvirt-domain.c |  520 +++++++++++++++++++++-----------------------------
>  1 files changed, 216 insertions(+), 304 deletions(-)

Just a quick note to say that I haven't forgotten about this patch
series. I'm looking to review it today/tomorrow I hope.

|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

More information about the libvir-list mailing list