Re: [PATCH 2/2] qemu: format address wdith on intel-iommu command line

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

 



On 04/06/2020 10:38, Peter Krempa wrote:
On Thu, Jun 04, 2020 at 09:42:43 +0200, Menno Lageman wrote:
Format the address width attribute. Depending on the version of
QEMU it is named 'aw-bits' or 'x-aw-bits'.

Signed-off-by: Menno Lageman <menno.lageman@xxxxxxxxxx>
---
  src/qemu/qemu_capabilities.c                  |  6 ++++
  src/qemu/qemu_capabilities.h                  |  4 +++
  src/qemu/qemu_command.c                       |  6 ++++
  src/qemu/qemu_validate.c                      |  9 ++++-
  .../caps_2.12.0.x86_64.xml                    |  1 +
  .../caps_3.0.0.x86_64.xml                     |  1 +
  .../caps_3.1.0.x86_64.xml                     |  1 +
  .../caps_4.0.0.x86_64.xml                     |  1 +
  .../caps_4.1.0.x86_64.xml                     |  1 +
  .../caps_4.2.0.x86_64.xml                     |  1 +
  .../caps_5.0.0.x86_64.xml                     |  1 +
  .../caps_5.1.0.x86_64.xml                     |  1 +
  .../intel-iommu-aw-bits.x86_64-latest.args    | 33 +++++++++++++++++++
  tests/qemuxml2argvtest.c                      |  1 +
  14 files changed, 66 insertions(+), 1 deletion(-)
  create mode 100644 tests/qemuxml2argvdata/intel-iommu-aw-bits.x86_64-latest.args

diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index f12769635a86..fa305bf67f33 100644
--- a/src/qemu/qemu_capabilities.c
+++ b/src/qemu/qemu_capabilities.c
@@ -582,6 +582,10 @@ VIR_ENUM_IMPL(virQEMUCaps,
                "tcg",
                "virtio-blk-pci.scsi.default.disabled",
                "pvscsi",
+
+              /* 370 */
+              "intel-iommu.x-aw-bits",

We definitely don't want to expose anythign with the experimental
prefix.

+              "intel-iommu.aw-bits",

This is okay.


Ok, I'll remove x-aw-bits in v2.

Is this 'we don't want to expose experimental stuff' documented somewhere I should have read?

Thanks,

Menno




[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