Re: [PATCH v4 0/12] migration: support all toURI and proto combos

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Oct 02, 2015 at 10:52:40 +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 version 3
> 
> * fix spellings
> * make error messages more specific as suggested
> * in 'migration: refactor: introduce params version of unmanaged':
>   fix conversion for null values of explicit string params into virTypedParameterPtr.
> * squash 'migration: reuse parameters check in toURI2 and toURI3' into
>   'migration: introduce parameter checking function'
> * 'migration: refactor: one return in forURI family functions' is dropped as suggested
> 
>  src/libvirt-domain.c |  509 +++++++++++++++++++++++---------------------------
>  1 files changed, 237 insertions(+), 272 deletions(-)

ACK series and pushed with the three cosmetics issues fixed.

Thanks for your work.

Jirka

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]