Re: [PATCH] docs: Update documentation of CPU models in domain caps

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

 



On Tue, Mar 05, 2024 at 06:15:46PM +0100, Jiri Denemark wrote:
>     using the CPU model and no additional feature elements. Models marked as
>     usable (``usable='yes'``) can be safely used in domain XMLs with
>     ``check='none'`` as the hypervisor guarantees the model can be used on the
> -   current host and additional checks done by libvirt are redundant. In fact,
> -   disabling libvirt checks via ``check='none'`` for such models is recommended
> -   to avoid needless issues with starting domains when libvirt's definition of
> -   a particular model differs from hypervisor's definition. The
> +   current host and additional checks done by libvirt are redundant.
> +   :since:`Since 10.2.0` libvirt automatically detects this situation and
> +   avoids the redundant checks, with older releases disabling libvirt checks
> +   via ``check='none'`` for such models is recommended to avoid needless
> +   issues with starting domains when libvirt's definition of a particular model
> +   differs from hypervisor's definition. The
>     ``deprecated`` attribute reflects the hypervisor's policy on usage of this
>     model :since:`(since 7.1.0)`. The ``vendor`` attribute :since:`(since 8.9.0)`
>     contains the vendor of the CPU model for users who want to use CPU models

I suggest adding

  ... avoids the redundant checks even when ``check='partial'`` is
  used, with older releases...

Either way

  Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>

-- 
Andrea Bolognani / Red Hat / Virtualization
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux