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

Nikolay Shirokovskiy nshirokovskiy at virtuozzo.com
Thu Sep 10 13:20:12 UTC 2015


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(-)




More information about the libvir-list mailing list