[v4,0/6] Add support for ACPI VIOT

Message ID 20210610075130.67517-1-jean-philippe@linaro.org
Headers show
Series
  • Add support for ACPI VIOT
Related show

Message

Jean-Philippe Brucker June 10, 2021, 7:51 a.m.
Add a driver for the ACPI VIOT table, which provides topology
information for para-virtual IOMMUs. Enable virtio-iommu on
non-devicetree platforms, including x86.

Since v3 [1] I fixed a build bug for !CONFIG_IOMMU_API. Joerg offered to
take this series through the IOMMU tree, which requires Acks for patches
1-3.

You can find a QEMU implementation at [2], with extra support for
testing all VIOT nodes including MMIO-based endpoints and IOMMU.
This series is at [3].

[1] https://lore.kernel.org/linux-iommu/20210602154444.1077006-1-jean-philippe@linaro.org/
[2] https://jpbrucker.net/git/qemu/log/?h=virtio-iommu/acpi
[3] https://jpbrucker.net/git/linux/log/?h=virtio-iommu/acpi


Jean-Philippe Brucker (6):
  ACPI: arm64: Move DMA setup operations out of IORT
  ACPI: Move IOMMU setup code out of IORT
  ACPI: Add driver for the VIOT table
  iommu/dma: Pass address limit rather than size to
    iommu_setup_dma_ops()
  iommu/dma: Simplify calls to iommu_setup_dma_ops()
  iommu/virtio: Enable x86 support

 drivers/acpi/Kconfig         |   3 +
 drivers/iommu/Kconfig        |   4 +-
 drivers/acpi/Makefile        |   2 +
 drivers/acpi/arm64/Makefile  |   1 +
 include/acpi/acpi_bus.h      |   3 +
 include/linux/acpi.h         |   3 +
 include/linux/acpi_iort.h    |  14 +-
 include/linux/acpi_viot.h    |  19 ++
 include/linux/dma-iommu.h    |   4 +-
 arch/arm64/mm/dma-mapping.c  |   2 +-
 drivers/acpi/arm64/dma.c     |  50 +++++
 drivers/acpi/arm64/iort.c    | 129 ++-----------
 drivers/acpi/bus.c           |   2 +
 drivers/acpi/scan.c          |  78 +++++++-
 drivers/acpi/viot.c          | 364 +++++++++++++++++++++++++++++++++++
 drivers/iommu/amd/iommu.c    |   9 +-
 drivers/iommu/dma-iommu.c    |  17 +-
 drivers/iommu/intel/iommu.c  |  10 +-
 drivers/iommu/virtio-iommu.c |   8 +
 MAINTAINERS                  |   8 +
 20 files changed, 580 insertions(+), 150 deletions(-)
 create mode 100644 include/linux/acpi_viot.h
 create mode 100644 drivers/acpi/arm64/dma.c
 create mode 100644 drivers/acpi/viot.c

-- 
2.31.1

Comments

Jean-Philippe Brucker June 16, 2021, 6:34 a.m. | #1
Hi Rafael,

On Thu, Jun 10, 2021 at 09:51:27AM +0200, Jean-Philippe Brucker wrote:
> Add a driver for the ACPI VIOT table, which provides topology

> information for para-virtual IOMMUs. Enable virtio-iommu on

> non-devicetree platforms, including x86.

> 

> Since v3 [1] I fixed a build bug for !CONFIG_IOMMU_API. Joerg offered to

> take this series through the IOMMU tree, which requires Acks for patches

> 1-3.


I was wondering if you could take a look at patches 1-3, otherwise we'll
miss the mark for 5.14 since I won't be able to resend next week. The
series adds support for virtio-iommu on QEMU and cloud hypervisor.

Thanks,
Jean

> 

> You can find a QEMU implementation at [2], with extra support for

> testing all VIOT nodes including MMIO-based endpoints and IOMMU.

> This series is at [3].

> 

> [1] https://lore.kernel.org/linux-iommu/20210602154444.1077006-1-jean-philippe@linaro.org/

> [2] https://jpbrucker.net/git/qemu/log/?h=virtio-iommu/acpi

> [3] https://jpbrucker.net/git/linux/log/?h=virtio-iommu/acpi

> 

> 

> Jean-Philippe Brucker (6):

>   ACPI: arm64: Move DMA setup operations out of IORT

>   ACPI: Move IOMMU setup code out of IORT

>   ACPI: Add driver for the VIOT table

>   iommu/dma: Pass address limit rather than size to

>     iommu_setup_dma_ops()

>   iommu/dma: Simplify calls to iommu_setup_dma_ops()

>   iommu/virtio: Enable x86 support

> 

>  drivers/acpi/Kconfig         |   3 +

>  drivers/iommu/Kconfig        |   4 +-

>  drivers/acpi/Makefile        |   2 +

>  drivers/acpi/arm64/Makefile  |   1 +

>  include/acpi/acpi_bus.h      |   3 +

>  include/linux/acpi.h         |   3 +

>  include/linux/acpi_iort.h    |  14 +-

>  include/linux/acpi_viot.h    |  19 ++

>  include/linux/dma-iommu.h    |   4 +-

>  arch/arm64/mm/dma-mapping.c  |   2 +-

>  drivers/acpi/arm64/dma.c     |  50 +++++

>  drivers/acpi/arm64/iort.c    | 129 ++-----------

>  drivers/acpi/bus.c           |   2 +

>  drivers/acpi/scan.c          |  78 +++++++-

>  drivers/acpi/viot.c          | 364 +++++++++++++++++++++++++++++++++++

>  drivers/iommu/amd/iommu.c    |   9 +-

>  drivers/iommu/dma-iommu.c    |  17 +-

>  drivers/iommu/intel/iommu.c  |  10 +-

>  drivers/iommu/virtio-iommu.c |   8 +

>  MAINTAINERS                  |   8 +

>  20 files changed, 580 insertions(+), 150 deletions(-)

>  create mode 100644 include/linux/acpi_viot.h

>  create mode 100644 drivers/acpi/arm64/dma.c

>  create mode 100644 drivers/acpi/viot.c

> 

> -- 

> 2.31.1

>
Eric Auger June 16, 2021, 12:40 p.m. | #2
Hi Jean,

On 6/10/21 9:51 AM, Jean-Philippe Brucker wrote:
> Add a driver for the ACPI VIOT table, which provides topology

> information for para-virtual IOMMUs. Enable virtio-iommu on

> non-devicetree platforms, including x86.

>

> Since v3 [1] I fixed a build bug for !CONFIG_IOMMU_API. Joerg offered to

> take this series through the IOMMU tree, which requires Acks for patches

> 1-3.

>

> You can find a QEMU implementation at [2], with extra support for

> testing all VIOT nodes including MMIO-based endpoints and IOMMU.

> This series is at [3].

>

> [1] https://lore.kernel.org/linux-iommu/20210602154444.1077006-1-jean-philippe@linaro.org/

> [2] https://jpbrucker.net/git/qemu/log/?h=virtio-iommu/acpi

> [3] https://jpbrucker.net/git/linux/log/?h=virtio-iommu/acpi


I tested the series on both aarch64 and x86_64 with qemu. It works for me.
Feel free to add my T-b.

Tested-by: Eric Auger <eric.auger@redhat.com>


Thanks

Eric

>

>

> Jean-Philippe Brucker (6):

>   ACPI: arm64: Move DMA setup operations out of IORT

>   ACPI: Move IOMMU setup code out of IORT

>   ACPI: Add driver for the VIOT table

>   iommu/dma: Pass address limit rather than size to

>     iommu_setup_dma_ops()

>   iommu/dma: Simplify calls to iommu_setup_dma_ops()

>   iommu/virtio: Enable x86 support

>

>  drivers/acpi/Kconfig         |   3 +

>  drivers/iommu/Kconfig        |   4 +-

>  drivers/acpi/Makefile        |   2 +

>  drivers/acpi/arm64/Makefile  |   1 +

>  include/acpi/acpi_bus.h      |   3 +

>  include/linux/acpi.h         |   3 +

>  include/linux/acpi_iort.h    |  14 +-

>  include/linux/acpi_viot.h    |  19 ++

>  include/linux/dma-iommu.h    |   4 +-

>  arch/arm64/mm/dma-mapping.c  |   2 +-

>  drivers/acpi/arm64/dma.c     |  50 +++++

>  drivers/acpi/arm64/iort.c    | 129 ++-----------

>  drivers/acpi/bus.c           |   2 +

>  drivers/acpi/scan.c          |  78 +++++++-

>  drivers/acpi/viot.c          | 364 +++++++++++++++++++++++++++++++++++

>  drivers/iommu/amd/iommu.c    |   9 +-

>  drivers/iommu/dma-iommu.c    |  17 +-

>  drivers/iommu/intel/iommu.c  |  10 +-

>  drivers/iommu/virtio-iommu.c |   8 +

>  MAINTAINERS                  |   8 +

>  20 files changed, 580 insertions(+), 150 deletions(-)

>  create mode 100644 include/linux/acpi_viot.h

>  create mode 100644 drivers/acpi/arm64/dma.c

>  create mode 100644 drivers/acpi/viot.c

>