[PATCH v1] docs: cputune is also supported by the xen driver

Jim Fehlig jfehlig at suse.com
Tue May 4 17:12:00 UTC 2021


On 5/3/21 5:33 AM, Olaf Hering wrote:
> Since commit 68c5b6fb2b5fdabce775e9f8fc761a400e16a9d3 libxl also handles
> a domain/cputune/vcpupin element in domU.xml.

git describe --contains 68c5b6fb2b5fdabce775e9f8fc761a400e16a9d3
v0.9.1~83

> 
> Signed-off-by: Olaf Hering <olaf at aepfle.de>
> ---
>   docs/formatdomain.rst | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/formatdomain.rst b/docs/formatdomain.rst
> index 282176c4f4..0328fe488b 100644
> --- a/docs/formatdomain.rst
> +++ b/docs/formatdomain.rst
> @@ -744,8 +744,8 @@ CPU Tuning
>      of element ``vcpu`` is not specified, the vCPU is pinned to all the physical
>      CPUs by default. It contains two required attributes, the attribute ``vcpu``
>      specifies vCPU id, and the attribute ``cpuset`` is same as attribute
> -   ``cpuset`` of element ``vcpu``. (NB: Only qemu driver support) :since:`Since
> -   0.9.0`
> +   ``cpuset`` of element ``vcpu``. (NB: Only qemu and xen driver support)
> +   :since:`Since 0.9.0`

So this isn't quite right.

IMO we can also improve the wording while at it. E.g. "QEMU driver support since 
0.9.0, Xen driver support since 0.9.1". That would be similar to the period, 
quota, etc descriptions, minus the unnecessary use of "Only".

Jim




More information about the libvir-list mailing list