1. Patchew
  2. Libvirt
  3. [libvirt] [libvirt-python PATCH 0/3] Couple of fixes and impl for virDomainSetIOThreadParams
  4. View patch

[libvirt] [libvirt-python PATCH 3/3] Implement API binding for virDomainSetIOThreadParams

John Ferlan posted 3 patches 7 years, 2 months ago
There is a newer version of this series
[libvirt] [libvirt-python PATCH 3/3] Implement API binding for virDomainSetIOThreadParams
Posted by John Ferlan 7 years, 2 months ago 
Similar to libvirt_virDomainBlockCopy (and migration API's). Create
the code for the new API.

Signed-off-by: John Ferlan <jferlan@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&apos;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&apos;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",
+                          &pyobj_dom, &iothread_val, &pyobj_dict, &flags))
+        return NULL;
+
+    if (PyDict_Check(pyobj_dict)) {
+        if (virPyDictToTypedParams(pyobj_dict, &params, &nparams,
+                                   virPyDomainSetIOThreadParams,
+                                   VIR_N_ELEMENTS(virPyDomainSetIOThreadParams)) < 0) {
+            return NULL;
+        }
+    }
+
+    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@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [libvirt-python PATCH 3/3] Implement API binding for virDomainSetIOThreadParams
Posted by Pavel Hrdina 7 years, 2 months ago 
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@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&apos;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&apos;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, &params, &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@redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Patchew 2.1-devel-b67e4c2

© 2016 - 2026 Red Hat, Inc.