[libvirt] [libvirt-python PATCH 3/3] Implement API binding for virDomainSetIOThreadParams
Pavel Hrdina
phrdina at redhat.com
Tue Nov 20 16:42:52 UTC 2018
On Tue, Nov 20, 2018 at 11:17:12AM -0500, John Ferlan wrote:
> Similar to libvirt_virDomainBlockCopy (and migration API's). Create
> the code for the new API.
>
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
> generator.py | 1 +
> libvirt-override-api.xml | 8 +++++++
> libvirt-override.c | 51 ++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 60 insertions(+)
>
> diff --git a/generator.py b/generator.py
> index bd7ff11..6cb923f 100755
> --- a/generator.py
> +++ b/generator.py
> @@ -440,6 +440,7 @@ skip_impl = (
> 'virDomainPinEmulator',
> 'virDomainGetIOThreadInfo',
> 'virDomainPinIOThread',
> + 'virDomainSetIOThreadParams',
> 'virSecretGetValue',
> 'virSecretSetValue',
> 'virSecretGetUUID',
> diff --git a/libvirt-override-api.xml b/libvirt-override-api.xml
> index 4e8d6c0..0a68d32 100644
> --- a/libvirt-override-api.xml
> +++ b/libvirt-override-api.xml
> @@ -292,6 +292,14 @@
> <arg name='cpumap' type='unsigned char *' info='pointer to a bit map of real CPUs (in 8-bit bytes) (IN) Each bit set to 1 means that corresponding CPU is usable. Bytes are stored in little-endian order: CPU0-7, 8-15... In each byte, lowest CPU number is least significant bit.'/>
> <arg name='flags' type='int' info='an OR'ed set of virDomainModificationImpact'/>
> </function>
> + <function name='virDomainSetIOThreadParams' file='python'>
> + <info>Dynamically allow changing the IOThread performance related parameters. This function requires privileged access to the hypervisor.</info>
> + <return type='int' info='0 in case of success, -1 in case of failure.'/>
> + <arg name='domain' type='virDomainPtr' info='pointer to domain object'/>
> + <arg name='iothread_val' type='unsigned int' info='iothread_id number'/>
> + <arg name='params' type='virTypedParameterPtr' info='pointer to IOThread performance statistic parameter objects'/>
> + <arg name='flags' type='int' info='an OR'ed set of virDomainModificationImpact'/>
> + </function>
> <function name='virDomainSetSchedulerParameters' file='python'>
> <info>Change the scheduler parameters</info>
> <return type='int' info='-1 in case of error, 0 in case of success.'/>
> diff --git a/libvirt-override.c b/libvirt-override.c
> index 0353306..cd03ff1 100644
> --- a/libvirt-override.c
> +++ b/libvirt-override.c
> @@ -1679,6 +1679,54 @@ libvirt_virDomainPinIOThread(PyObject *self ATTRIBUTE_UNUSED,
>
> #endif /* LIBVIR_CHECK_VERSION(1, 2, 14) */
>
> +#if LIBVIR_CHECK_VERSION(4, 10, 4)
> +
> +static virPyTypedParamsHint virPyDomainSetIOThreadParams[] = {
> + { VIR_DOMAIN_IOTHREAD_POLL_MAX_NS, VIR_TYPED_PARAM_ULLONG },
> + { VIR_DOMAIN_IOTHREAD_POLL_GROW, VIR_TYPED_PARAM_UINT },
> + { VIR_DOMAIN_IOTHREAD_POLL_SHRINK, VIR_TYPED_PARAM_ULLONG },
> +};
> +
> +static PyObject *
> +libvirt_virDomainSetIOThreadParams(PyObject *self ATTRIBUTE_UNUSED,
> + PyObject *args)
> +{
> + PyObject *pyobj_dom = NULL;
> + PyObject *pyobj_dict = NULL;
> +
> + virDomainPtr dom;
> + int iothread_val;
> + virTypedParameterPtr params = NULL;
> + int nparams = 0;
> + unsigned int flags;
> + int c_retval;
> +
> + if (!PyArg_ParseTuple(args,
> + (char *)"OiOI:virDomainSetIOThreadParams",
Nitpick: probably everywhere else the format string is on the same line
as 'args'. Feels wrong :).
> + &pyobj_dom, &iothread_val, &pyobj_dict, &flags))
> + return NULL;
> +
> + if (PyDict_Check(pyobj_dict)) {
> + if (virPyDictToTypedParams(pyobj_dict, ¶ms, &nparams,
> + virPyDomainSetIOThreadParams,
> + VIR_N_ELEMENTS(virPyDomainSetIOThreadParams)) < 0) {
> + return NULL;
> + }
> + }
I know that we are not consistent in parsing dictionary in our binding
code which should be eventually fixed.
This will silently ignore the case that params is not dict. I think
that we should error out in that case, for example
libvirt_virDomainSetTime returns correct error.
Pavel
> +
> + dom = (virDomainPtr) PyvirDomain_Get(pyobj_dom);
> +
> + LIBVIRT_BEGIN_ALLOW_THREADS;
> + c_retval = virDomainSetIOThreadParams(dom, iothread_val,
> + params, nparams, flags);
> + LIBVIRT_END_ALLOW_THREADS;
> +
> + virTypedParamsFree(params, nparams);
> +
> + return libvirt_intWrap(c_retval);
> +}
> +#endif /* LIBVIR_CHECK_VERSION(4, 10, 0) */
> +
> /************************************************************************
> * *
> * Global error handler at the Python level *
> @@ -9975,6 +10023,9 @@ static PyMethodDef libvirtMethods[] = {
> {(char *) "virDomainGetIOThreadInfo", libvirt_virDomainGetIOThreadInfo, METH_VARARGS, NULL},
> {(char *) "virDomainPinIOThread", libvirt_virDomainPinIOThread, METH_VARARGS, NULL},
> #endif /* LIBVIR_CHECK_VERSION(1, 2, 14) */
> +#if LIBVIR_CHECK_VERSION(4, 10, 4)
> + {(char *) "virDomainSetIOThreadParams", libvirt_virDomainSetIOThreadParams, METH_VARARGS, NULL},
> +#endif /* LIBVIR_CHECK_VERSION(4, 10, 0) */
> {(char *) "virConnectListStoragePools", libvirt_virConnectListStoragePools, METH_VARARGS, NULL},
> {(char *) "virConnectListDefinedStoragePools", libvirt_virConnectListDefinedStoragePools, METH_VARARGS, NULL},
> #if LIBVIR_CHECK_VERSION(0, 10, 2)
> --
> 2.17.2
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20181120/78d266f6/attachment-0001.sig>
More information about the libvir-list
mailing list