On Wed, Oct 01, 2025 at 05:23:46PM -0700, Nathan Chen via Devel wrote:
> Introduce support for "smmuv3Dev" IOMMU model and
> its "parentIdx" driver attribute. The "parentIdx"
> attribute indicates the index of the controller that a
> smmuv3Dev IOMMU device is attached to.
> 
> Signed-off-by: Nathan Chen <[email protected]>
> ---
>  docs/formatdomain.rst             |  9 +++-
>  src/conf/domain_conf.c            | 17 ++++++++
>  src/conf/domain_conf.h            |  2 +
>  src/conf/domain_validate.c        | 26 +++++++++--
>  src/conf/schemas/domaincommon.rng |  6 +++
>  src/qemu/qemu_command.c           | 72 +++++++++++++++++++++++++++++--
>  src/qemu/qemu_domain_address.c    |  2 +
>  src/qemu/qemu_validate.c          | 16 +++++++
>  8 files changed, 141 insertions(+), 9 deletions(-)


> @@ -9212,6 +9213,10 @@ Example:
>        Enable x2APIC mode. Useful for higher number of guest CPUs.
>        :since:`Since 11.5.0` (QEMU/KVM and ``amd`` model only)
>  
> +   ``parentIdx``
> +      The ``parentIdx`` attribute notes the index of the controller that an
> +      IOMMU device is attached to. (QEMU/KVM and ``smmuv3Dev`` model only)
> +

In the individual devices, for the <address> element we use 'bus' to refer
to the index of the PCI controller. I wonder if we shouldn't keep that
terminology similar, such that we get a 'pciBus' attribute instead
of 'parentIdx'. eg 

  <iommu .... pciBus='2'/>



With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

Reply via email to