[libvirt] [PATCH] New features implemented in hyperv libvirt driver (Bull)
Eric Blake
eblake at redhat.com
Fri Sep 12 18:55:25 UTC 2014
On 09/12/2014 10:36 AM, Yves Vinter wrote:
> Authors: Simon Rastello (Bull), Adrien Kantcheff (Bull), Yves Vinter (Bull)
>
> Summary of new added features by modules:
>
> hyperv_driver.c
> - hypervDomainDefineXML
> - hypervDomainCreateXML
> - hypervDomainUndefine
> - hypervDomainUndefineFlags
> - hypervDomainShutdown
> - hypervDomainShutdownFlags
> - hypervDomainGetVcpus
> - hypervDomainGetVcpusFlags
> - hypervConnectGetMaxVcpus
> - hypervDomainGetMaxVcpus
> - hypervDomainSetVcpus
> - hypervDomainSetVcpusFlags
> - hypervDomainSetMemory
> - hypervDomainSetMemoryFlags
> - hypervDomainSetMaxMemory
> - hypervNodeGetFreeMemory
> - hypervDomainAttachDevice
> - hypervDomainAttachDeviceFlags
> - hypervDomainGetSchedulerParameters
> - hypervDomainGetSchedulerParametersFlags
> - hypervDomainGetSchedulerType
> - hypervConnectGetCapabilities
> - hypervConnectGetVersion
> - hypervDomainSetAutostart
> - hypervDomainGetAutostart
Thanks for the patch. However, it is far too large to easily review.
It would be much better if you split this into a series of multiple
smaller patches, with each patch focusing on just a few tightly-related
interfaces (for example,
hypervDomain{DefineXML,CreatexML,Undefine,UndefineFlags} could be one
patch, hten hypervDomainShutdown{,Flags} as a second, and so on).
> @@ -58,12 +64,97 @@ hypervFreePrivate(hypervPrivate **priv)
> wsmc_release((*priv)->client);
> }
>
> + /* Bull */
> + pthread_mutex_destroy(&(*priv)->mutex);
> +
What are all the "Bull" comments for?
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 539 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140912/193f5d0c/attachment-0001.sig>
More information about the libvir-list
mailing list