[PATCH 0/3] Intel-IOMMU: Add dma-translation switch

Sandesh Patel posted 3 patches 3 months, 2 weeks ago
docs/formatdomain.rst                         |  7 ++++
src/conf/domain_conf.c                        | 15 ++++++++
src/conf/domain_conf.h                        |  1 +
src/conf/schemas/domaincommon.rng             |  5 +++
src/qemu/qemu_capabilities.c                  |  2 +
src/qemu/qemu_capabilities.h                  |  1 +
src/qemu/qemu_command.c                       |  1 +
src/qemu/qemu_validate.c                      |  6 +++
.../caps_7.1.0_x86_64.xml                     |  1 +
.../caps_7.2.0_x86_64+hvf.xml                 |  1 +
.../caps_7.2.0_x86_64.xml                     |  1 +
.../caps_8.0.0_x86_64.xml                     |  1 +
.../caps_8.1.0_x86_64.xml                     |  1 +
.../caps_8.2.0_x86_64.xml                     |  1 +
.../caps_9.0.0_x86_64.xml                     |  1 +
.../caps_9.1.0_x86_64.xml                     |  1 +
.../migration-out-nbd-bitmaps-in.xml          |  1 +
...l-iommu-dma-translation.x86_64-latest.args | 34 +++++++++++++++++
...el-iommu-dma-translation.x86_64-latest.xml |  1 +
.../intel-iommu-dma-translation.xml           | 37 +++++++++++++++++++
tests/qemuxmlconftest.c                       |  1 +
21 files changed, 120 insertions(+)
create mode 100644 tests/qemuxmlconfdata/intel-iommu-dma-translation.x86_64-latest.args
create mode 120000 tests/qemuxmlconfdata/intel-iommu-dma-translation.x86_64-latest.xml
create mode 100644 tests/qemuxmlconfdata/intel-iommu-dma-translation.xml
[PATCH 0/3] Intel-IOMMU: Add dma-translation switch
Posted by Sandesh Patel 3 months, 2 weeks ago
Since Qemu commit 8646d9c (Support IR-only mode without DMA translation) version 7.1.0,
Qemu support dma-translation attribute for Intel IOMMU. This patch adds a new attribute
dma_translation to configure this.

Sandesh Patel (3):
  Conf: add dma_translation attribute to iommu
  qemu: format dma-translation on intel-iommu command line
  tests: add test for intel-iommu dma-translation attribute

 docs/formatdomain.rst                         |  7 ++++
 src/conf/domain_conf.c                        | 15 ++++++++
 src/conf/domain_conf.h                        |  1 +
 src/conf/schemas/domaincommon.rng             |  5 +++
 src/qemu/qemu_capabilities.c                  |  2 +
 src/qemu/qemu_capabilities.h                  |  1 +
 src/qemu/qemu_command.c                       |  1 +
 src/qemu/qemu_validate.c                      |  6 +++
 .../caps_7.1.0_x86_64.xml                     |  1 +
 .../caps_7.2.0_x86_64+hvf.xml                 |  1 +
 .../caps_7.2.0_x86_64.xml                     |  1 +
 .../caps_8.0.0_x86_64.xml                     |  1 +
 .../caps_8.1.0_x86_64.xml                     |  1 +
 .../caps_8.2.0_x86_64.xml                     |  1 +
 .../caps_9.0.0_x86_64.xml                     |  1 +
 .../caps_9.1.0_x86_64.xml                     |  1 +
 .../migration-out-nbd-bitmaps-in.xml          |  1 +
 ...l-iommu-dma-translation.x86_64-latest.args | 34 +++++++++++++++++
 ...el-iommu-dma-translation.x86_64-latest.xml |  1 +
 .../intel-iommu-dma-translation.xml           | 37 +++++++++++++++++++
 tests/qemuxmlconftest.c                       |  1 +
 21 files changed, 120 insertions(+)
 create mode 100644 tests/qemuxmlconfdata/intel-iommu-dma-translation.x86_64-latest.args
 create mode 120000 tests/qemuxmlconfdata/intel-iommu-dma-translation.x86_64-latest.xml
 create mode 100644 tests/qemuxmlconfdata/intel-iommu-dma-translation.xml

-- 
2.22.3
Re: [PATCH 0/3] Intel-IOMMU: Add dma-translation switch
Posted by Michal Prívozník 3 months, 2 weeks ago
On 8/7/24 09:37, Sandesh Patel wrote:
> Since Qemu commit 8646d9c (Support IR-only mode without DMA translation) version 7.1.0,
> Qemu support dma-translation attribute for Intel IOMMU. This patch adds a new attribute
> dma_translation to configure this.
> 
> Sandesh Patel (3):
>   Conf: add dma_translation attribute to iommu
>   qemu: format dma-translation on intel-iommu command line
>   tests: add test for intel-iommu dma-translation attribute
> 
>  docs/formatdomain.rst                         |  7 ++++
>  src/conf/domain_conf.c                        | 15 ++++++++
>  src/conf/domain_conf.h                        |  1 +
>  src/conf/schemas/domaincommon.rng             |  5 +++
>  src/qemu/qemu_capabilities.c                  |  2 +
>  src/qemu/qemu_capabilities.h                  |  1 +
>  src/qemu/qemu_command.c                       |  1 +
>  src/qemu/qemu_validate.c                      |  6 +++
>  .../caps_7.1.0_x86_64.xml                     |  1 +
>  .../caps_7.2.0_x86_64+hvf.xml                 |  1 +
>  .../caps_7.2.0_x86_64.xml                     |  1 +
>  .../caps_8.0.0_x86_64.xml                     |  1 +
>  .../caps_8.1.0_x86_64.xml                     |  1 +
>  .../caps_8.2.0_x86_64.xml                     |  1 +
>  .../caps_9.0.0_x86_64.xml                     |  1 +
>  .../caps_9.1.0_x86_64.xml                     |  1 +
>  .../migration-out-nbd-bitmaps-in.xml          |  1 +
>  ...l-iommu-dma-translation.x86_64-latest.args | 34 +++++++++++++++++
>  ...el-iommu-dma-translation.x86_64-latest.xml |  1 +
>  .../intel-iommu-dma-translation.xml           | 37 +++++++++++++++++++
>  tests/qemuxmlconftest.c                       |  1 +
>  21 files changed, 120 insertions(+)
>  create mode 100644 tests/qemuxmlconfdata/intel-iommu-dma-translation.x86_64-latest.args
>  create mode 120000 tests/qemuxmlconfdata/intel-iommu-dma-translation.x86_64-latest.xml
>  create mode 100644 tests/qemuxmlconfdata/intel-iommu-dma-translation.xml
> 

We split things a bit differently, but I'll do that before merging.

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

and merged. Congratulations on your first libvirt contribution!

Michal
Re: [PATCH 0/3] Intel-IOMMU: Add dma-translation switch
Posted by Sandesh Patel 3 months, 2 weeks ago

On 9 Aug 2024, at 1:37 PM, Michal Prívozník <mprivozn@redhat.com> wrote:

!-------------------------------------------------------------------|
 CAUTION: External Email

|-------------------------------------------------------------------!

On 8/7/24 09:37, Sandesh Patel wrote:
Since Qemu commit 8646d9c (Support IR-only mode without DMA translation) version 7.1.0,
Qemu support dma-translation attribute for Intel IOMMU. This patch adds a new attribute
dma_translation to configure this.

Sandesh Patel (3):
 Conf: add dma_translation attribute to iommu
 qemu: format dma-translation on intel-iommu command line
 tests: add test for intel-iommu dma-translation attribute

docs/formatdomain.rst                         |  7 ++++
src/conf/domain_conf.c                        | 15 ++++++++
src/conf/domain_conf.h                        |  1 +
src/conf/schemas/domaincommon.rng             |  5 +++
src/qemu/qemu_capabilities.c                  |  2 +
src/qemu/qemu_capabilities.h                  |  1 +
src/qemu/qemu_command.c                       |  1 +
src/qemu/qemu_validate.c                      |  6 +++
.../caps_7.1.0_x86_64.xml                     |  1 +
.../caps_7.2.0_x86_64+hvf.xml                 |  1 +
.../caps_7.2.0_x86_64.xml                     |  1 +
.../caps_8.0.0_x86_64.xml                     |  1 +
.../caps_8.1.0_x86_64.xml                     |  1 +
.../caps_8.2.0_x86_64.xml                     |  1 +
.../caps_9.0.0_x86_64.xml                     |  1 +
.../caps_9.1.0_x86_64.xml                     |  1 +
.../migration-out-nbd-bitmaps-in.xml          |  1 +
...l-iommu-dma-translation.x86_64-latest.args | 34 +++++++++++++++++
...el-iommu-dma-translation.x86_64-latest.xml |  1 +
.../intel-iommu-dma-translation.xml           | 37 +++++++++++++++++++
tests/qemuxmlconftest.c                       |  1 +
21 files changed, 120 insertions(+)
create mode 100644 tests/qemuxmlconfdata/intel-iommu-dma-translation.x86_64-latest.args
create mode 120000 tests/qemuxmlconfdata/intel-iommu-dma-translation.x86_64-latest.xml
create mode 100644 tests/qemuxmlconfdata/intel-iommu-dma-translation.xml


We split things a bit differently, but I'll do that before merging.

Reviewed-by: Michal Privoznik <mprivozn@redhat.com<mailto:mprivozn@redhat.com>>

and merged. Congratulations on your first libvirt contribution!

Michal

Thanks Michal

Regards,
Sandesh