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

Re: [libvirt] [PATCH] New features implemented in hyperv libvirt driver



On 09/25/2014 06:38 AM, Yves Vinter wrote:
> Authors: Simon Rastello (Bull), Adrien Kantcheff (Bull), Yves Vinter (Bull)
> 
> *** Summary of new features added in hyperv driver version 1.2.9 ***
> 
> - Added a mutex to prevent concurrent requests from being run simultaneously [WSMAN libray not thread-safe]
> - Support of default authentication (credentials specified in ~/.config/libvirt/auth.conf)
> - Support of new types of WSMAN requests involving passing complex parameters (simple types, EPR, embedded objects)
>   Required a new auto-generated header file to get the type of objects attributes (hyperv_wmi_classes_attr.generated.h)
> - New functions:
> 	- host management (in hyperv_driver module)
> 		- capabilities (limited)
> 	- domain management (in hyperv_driver module)
> 		- domain creation from an XML description
> 		- domain destruction
> 		- attachment of pre-existing disk images on IDE controller
> 		- attachment of Synthetic network devices
> 		- memory and vcpu management (get and set methods)
> 		- autostart, shutdown, ...
> 	- network management (in hyperv_network_driver module)
> 		- list available networks
> 		- informations for a specified network
>   Required the declaration of new classes (in hyperv_wmi_generator.input)
> - Fixed several memory leak issues

Please split each of these bulleted list items into a separate patch,
for easier review.  No one likes to review a 150k single patch.

Also, it looks like you just missed feature freeze for 1.2.9; by the
time you've split the series, it is probably a better candidate for
inclusion in 1.2.10 around the end of October.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


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