[PATCH v6 0/7] virtio-iommu: Add ACPI support (x86 part)

Jean-Philippe Brucker posted 7 patches 2 years, 6 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211026182024.2642038-1-jean-philippe@linaro.org
Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Igor Mammedov <imammedo@redhat.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, "Michael S. Tsirkin" <mst@redhat.com>, Ani Sinha <ani@anisinha.ca>, Eduardo Habkost <ehabkost@redhat.com>, Peter Xu <peterx@redhat.com>, Jason Wang <jasowang@redhat.com>, Richard Henderson <richard.henderson@linaro.org>
hw/acpi/viot.h                 |  13 ++++
include/hw/i386/pc.h           |   1 +
include/hw/i386/x86-iommu.h    |  12 ----
hw/acpi/viot.c                 | 114 +++++++++++++++++++++++++++++++++
hw/i386/acpi-build.c           |  28 ++++----
hw/i386/amd_iommu.c            |   2 -
hw/i386/intel_iommu.c          |   3 -
hw/i386/pc.c                   |  26 +++++++-
hw/i386/x86-iommu-stub.c       |   5 --
hw/i386/x86-iommu.c            |  31 +++------
tests/qtest/bios-tables-test.c |  21 ++++++
hw/acpi/Kconfig                |   4 ++
hw/acpi/meson.build            |   1 +
hw/i386/Kconfig                |   1 +
tests/data/acpi/q35/DSDT.viot  | Bin 0 -> 9398 bytes
tests/data/acpi/q35/VIOT.viot  | Bin 0 -> 112 bytes
16 files changed, 205 insertions(+), 57 deletions(-)
create mode 100644 hw/acpi/viot.h
create mode 100644 hw/acpi/viot.c
create mode 100644 tests/data/acpi/q35/DSDT.viot
create mode 100644 tests/data/acpi/q35/VIOT.viot
[PATCH v6 0/7] virtio-iommu: Add ACPI support (x86 part)
Posted by Jean-Philippe Brucker 2 years, 6 months ago
Allow instantiating a virtio-iommu device on ACPI systems by adding a
Virtual I/O Translation table (VIOT). Enable x86 support for VIOT.

Changes since v5 [1]:
* Split the series into x86 and arm parts. This patchset contains only
  the VIOT table (patch 1) and the x86 support.
* Rebased onto master to fix a merge conflict
* Added review and acks, thanks!

The boot-bypass series [2] isn't a hard requirement for this series but
will be needed for complete support. In the meantime, storage accessed
by firmware needs to bypass the IOMMU some other way, using either a
virtio-blk device without the iommu_platform property, or a bypass
bridge (docs/iommu-bypass.txt).

You can find a description of the VIOT table, which will be included in
next ACPI version, here: https://jpbrucker.net/virtio-iommu/viot/viot-v9.pdf

[1] https://lore.kernel.org/qemu-devel/20211020172745.620101-1-jean-philippe@linaro.org/
[2] https://lore.kernel.org/qemu-devel/20210930185050.262759-1-jean-philippe@linaro.org/

Jean-Philippe Brucker (7):
  hw/acpi: Add VIOT table
  hw/i386/pc: Remove x86_iommu_get_type()
  hw/i386/pc: Move IOMMU singleton into PCMachineState
  hw/i386/pc: Allow instantiating a virtio-iommu device
  tests/acpi: allow updates of VIOT expected data files on q35 machine
  tests/acpi: add test case for VIOT on q35 machine
  tests/acpi: add expected blobs for VIOT test on q35 machine

 hw/acpi/viot.h                 |  13 ++++
 include/hw/i386/pc.h           |   1 +
 include/hw/i386/x86-iommu.h    |  12 ----
 hw/acpi/viot.c                 | 114 +++++++++++++++++++++++++++++++++
 hw/i386/acpi-build.c           |  28 ++++----
 hw/i386/amd_iommu.c            |   2 -
 hw/i386/intel_iommu.c          |   3 -
 hw/i386/pc.c                   |  26 +++++++-
 hw/i386/x86-iommu-stub.c       |   5 --
 hw/i386/x86-iommu.c            |  31 +++------
 tests/qtest/bios-tables-test.c |  21 ++++++
 hw/acpi/Kconfig                |   4 ++
 hw/acpi/meson.build            |   1 +
 hw/i386/Kconfig                |   1 +
 tests/data/acpi/q35/DSDT.viot  | Bin 0 -> 9398 bytes
 tests/data/acpi/q35/VIOT.viot  | Bin 0 -> 112 bytes
 16 files changed, 205 insertions(+), 57 deletions(-)
 create mode 100644 hw/acpi/viot.h
 create mode 100644 hw/acpi/viot.c
 create mode 100644 tests/data/acpi/q35/DSDT.viot
 create mode 100644 tests/data/acpi/q35/VIOT.viot

-- 
2.33.0