Re: [PATCH] [Doc] Update 2 items in the formatdomain.rst

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

 



Saying what part of the file is being updated is more useful than
counting the changes. I've changed the commit summary to:

  docs: formatdomain: update hostdev interface section a bit

On a Thursday in 2022, Yalan Zhang wrote:
Update the default "driver" value for hostdev interface since
the default is not "KVM" anymore(refer to "Host device
asssignment" part and by test results). And update the mac
address in one xml example.

Signed-off-by: Yalan Zhang <yalzhang@xxxxxxxxxx>
---
docs/formatdomain.rst | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/docs/formatdomain.rst b/docs/formatdomain.rst
index b6a3495093..19c4aa5e2e 100644
--- a/docs/formatdomain.rst
+++ b/docs/formatdomain.rst
@@ -4917,9 +4917,9 @@ To use VFIO device assignment rather than traditional/legacy KVM device
assignment (VFIO is a new method of device assignment that is compatible with
UEFI Secure Boot), a type='hostdev' interface can have an optional ``driver``
sub-element with a ``name`` attribute set to "vfio". To use legacy KVM device
-assignment you can set ``name`` to "kvm" (or simply omit the ``<driver>``
-element, since "kvm" is currently the default). :since:`Since 1.0.5 (QEMU and
-KVM only, requires kernel 3.6 or newer)`
+assignment you can set ``name`` to "kvm" (the default is "vfio" on systems
+where the VFIO driver is available, and "kvm" on older systems. :since:`Since
+1.1.3` (prior to that the default was always "kvm").


It seems the whole section could use a rewrite with VFIO in mind, but
this change at least gets rid of the incorrect claims.

Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx>

and pushed.

Jano

Attachment: signature.asc
Description: PGP signature


[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