* [PATCH v7 1/8] hw/arm/virt-acpi-build: Add VIOT table for virtio-iommu
2021-12-10 17:04 [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Jean-Philippe Brucker
@ 2021-12-10 17:04 ` Jean-Philippe Brucker
2021-12-10 17:04 ` [PATCH v7 2/8] hw/arm/virt: Remove device tree restriction " Jean-Philippe Brucker
` (8 subsequent siblings)
9 siblings, 0 replies; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-10 17:04 UTC (permalink / raw)
To: mst, imammedo, peter.maydell, shannon.zhaosl
Cc: ani, eric.auger, qemu-arm, qemu-devel, Jean-Philippe Brucker
When a virtio-iommu is instantiated, describe it using the ACPI VIOT
table.
Acked-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
hw/arm/virt-acpi-build.c | 7 +++++++
hw/arm/Kconfig | 1 +
2 files changed, 8 insertions(+)
diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
index 674f902652..d0f4867fdf 100644
--- a/hw/arm/virt-acpi-build.c
+++ b/hw/arm/virt-acpi-build.c
@@ -55,6 +55,7 @@
#include "kvm_arm.h"
#include "migration/vmstate.h"
#include "hw/acpi/ghes.h"
+#include "hw/acpi/viot.h"
#define ARM_SPI_BASE 32
@@ -1011,6 +1012,12 @@ void virt_acpi_build(VirtMachineState *vms, AcpiBuildTables *tables)
}
#endif
+ if (vms->iommu == VIRT_IOMMU_VIRTIO) {
+ acpi_add_table(table_offsets, tables_blob);
+ build_viot(ms, tables_blob, tables->linker, vms->virtio_iommu_bdf,
+ vms->oem_id, vms->oem_table_id);
+ }
+
/* XSDT is pointed to by RSDP */
xsdt = tables_blob->len;
build_xsdt(tables_blob, tables->linker, table_offsets, vms->oem_id,
diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
index 2d37d29f02..e652590943 100644
--- a/hw/arm/Kconfig
+++ b/hw/arm/Kconfig
@@ -27,6 +27,7 @@ config ARM_VIRT
select DIMM
select ACPI_HW_REDUCED
select ACPI_APEI
+ select ACPI_VIOT
config CHEETAH
bool
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v7 2/8] hw/arm/virt: Remove device tree restriction for virtio-iommu
2021-12-10 17:04 [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Jean-Philippe Brucker
2021-12-10 17:04 ` [PATCH v7 1/8] hw/arm/virt-acpi-build: Add VIOT table for virtio-iommu Jean-Philippe Brucker
@ 2021-12-10 17:04 ` Jean-Philippe Brucker
2021-12-10 17:04 ` [PATCH v7 3/8] hw/arm/virt: Reject instantiation of multiple IOMMUs Jean-Philippe Brucker
` (7 subsequent siblings)
9 siblings, 0 replies; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-10 17:04 UTC (permalink / raw)
To: mst, imammedo, peter.maydell, shannon.zhaosl
Cc: ani, eric.auger, qemu-arm, qemu-devel, Jean-Philippe Brucker
virtio-iommu is now supported with ACPI VIOT as well as device tree.
Remove the restriction that prevents from instantiating a virtio-iommu
device under ACPI.
Acked-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
hw/arm/virt.c | 10 ++--------
hw/virtio/virtio-iommu-pci.c | 12 ++----------
2 files changed, 4 insertions(+), 18 deletions(-)
diff --git a/hw/arm/virt.c b/hw/arm/virt.c
index 30da05dfe0..53941e4aac 100644
--- a/hw/arm/virt.c
+++ b/hw/arm/virt.c
@@ -2614,16 +2614,10 @@ static HotplugHandler *virt_machine_get_hotplug_handler(MachineState *machine,
MachineClass *mc = MACHINE_GET_CLASS(machine);
if (device_is_dynamic_sysbus(mc, dev) ||
- (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM))) {
+ object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM) ||
+ object_dynamic_cast(OBJECT(dev), TYPE_VIRTIO_IOMMU_PCI)) {
return HOTPLUG_HANDLER(machine);
}
- if (object_dynamic_cast(OBJECT(dev), TYPE_VIRTIO_IOMMU_PCI)) {
- VirtMachineState *vms = VIRT_MACHINE(machine);
-
- if (!vms->bootinfo.firmware_loaded || !virt_is_acpi_enabled(vms)) {
- return HOTPLUG_HANDLER(machine);
- }
- }
return NULL;
}
diff --git a/hw/virtio/virtio-iommu-pci.c b/hw/virtio/virtio-iommu-pci.c
index a160ae6b41..6a1df7fe50 100644
--- a/hw/virtio/virtio-iommu-pci.c
+++ b/hw/virtio/virtio-iommu-pci.c
@@ -48,16 +48,8 @@ static void virtio_iommu_pci_realize(VirtIOPCIProxy *vpci_dev, Error **errp)
VirtIOIOMMU *s = VIRTIO_IOMMU(vdev);
if (!qdev_get_machine_hotplug_handler(DEVICE(vpci_dev))) {
- MachineClass *mc = MACHINE_GET_CLASS(qdev_get_machine());
-
- error_setg(errp,
- "%s machine fails to create iommu-map device tree bindings",
- mc->name);
- error_append_hint(errp,
- "Check your machine implements a hotplug handler "
- "for the virtio-iommu-pci device\n");
- error_append_hint(errp, "Check the guest is booted without FW or with "
- "-no-acpi\n");
+ error_setg(errp, "Check your machine implements a hotplug handler "
+ "for the virtio-iommu-pci device");
return;
}
for (int i = 0; i < s->nb_reserved_regions; i++) {
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v7 3/8] hw/arm/virt: Reject instantiation of multiple IOMMUs
2021-12-10 17:04 [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Jean-Philippe Brucker
2021-12-10 17:04 ` [PATCH v7 1/8] hw/arm/virt-acpi-build: Add VIOT table for virtio-iommu Jean-Philippe Brucker
2021-12-10 17:04 ` [PATCH v7 2/8] hw/arm/virt: Remove device tree restriction " Jean-Philippe Brucker
@ 2021-12-10 17:04 ` Jean-Philippe Brucker
2021-12-10 17:04 ` [PATCH v7 4/8] hw/arm/virt: Use object_property_set instead of qdev_prop_set Jean-Philippe Brucker
` (6 subsequent siblings)
9 siblings, 0 replies; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-10 17:04 UTC (permalink / raw)
To: mst, imammedo, peter.maydell, shannon.zhaosl
Cc: ani, eric.auger, qemu-arm, qemu-devel, Jean-Philippe Brucker
We do not support instantiating multiple IOMMUs. Before adding a
virtio-iommu, check that no other IOMMU is present. This will detect
both "iommu=smmuv3" machine parameter and another virtio-iommu instance.
Fixes: 70e89132c9 ("hw/arm/virt: Add the virtio-iommu device tree mappings")
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
hw/arm/virt.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/hw/arm/virt.c b/hw/arm/virt.c
index 53941e4aac..608a0a78f0 100644
--- a/hw/arm/virt.c
+++ b/hw/arm/virt.c
@@ -2494,6 +2494,11 @@ static void virt_machine_device_pre_plug_cb(HotplugHandler *hotplug_dev,
hwaddr db_start = 0, db_end = 0;
char *resv_prop_str;
+ if (vms->iommu != VIRT_IOMMU_NONE) {
+ error_setg(errp, "virt machine does not support multiple IOMMUs");
+ return;
+ }
+
switch (vms->msi_controller) {
case VIRT_MSI_CTRL_NONE:
return;
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v7 4/8] hw/arm/virt: Use object_property_set instead of qdev_prop_set
2021-12-10 17:04 [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Jean-Philippe Brucker
` (2 preceding siblings ...)
2021-12-10 17:04 ` [PATCH v7 3/8] hw/arm/virt: Reject instantiation of multiple IOMMUs Jean-Philippe Brucker
@ 2021-12-10 17:04 ` Jean-Philippe Brucker
2021-12-10 17:04 ` [PATCH v7 5/8] tests/acpi: allow updates of VIOT expected data files Jean-Philippe Brucker
` (5 subsequent siblings)
9 siblings, 0 replies; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-10 17:04 UTC (permalink / raw)
To: mst, imammedo, peter.maydell, shannon.zhaosl
Cc: ani, eric.auger, qemu-arm, qemu-devel, Jean-Philippe Brucker
To propagate errors to the caller of the pre_plug callback, use the
object_poperty_set*() functions directly instead of the qdev_prop_set*()
helpers.
Suggested-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
hw/arm/virt.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/hw/arm/virt.c b/hw/arm/virt.c
index 608a0a78f0..6b51e8ad10 100644
--- a/hw/arm/virt.c
+++ b/hw/arm/virt.c
@@ -2518,8 +2518,9 @@ static void virt_machine_device_pre_plug_cb(HotplugHandler *hotplug_dev,
db_start, db_end,
VIRTIO_IOMMU_RESV_MEM_T_MSI);
- qdev_prop_set_uint32(dev, "len-reserved-regions", 1);
- qdev_prop_set_string(dev, "reserved-regions[0]", resv_prop_str);
+ object_property_set_uint(OBJECT(dev), "len-reserved-regions", 1, errp);
+ object_property_set_str(OBJECT(dev), "reserved-regions[0]",
+ resv_prop_str, errp);
g_free(resv_prop_str);
}
}
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v7 5/8] tests/acpi: allow updates of VIOT expected data files
2021-12-10 17:04 [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Jean-Philippe Brucker
` (3 preceding siblings ...)
2021-12-10 17:04 ` [PATCH v7 4/8] hw/arm/virt: Use object_property_set instead of qdev_prop_set Jean-Philippe Brucker
@ 2021-12-10 17:04 ` Jean-Philippe Brucker
2021-12-10 17:04 ` [PATCH v7 6/8] tests/acpi: add test case for VIOT Jean-Philippe Brucker
` (4 subsequent siblings)
9 siblings, 0 replies; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-10 17:04 UTC (permalink / raw)
To: mst, imammedo, peter.maydell, shannon.zhaosl
Cc: ani, eric.auger, qemu-arm, qemu-devel, Jean-Philippe Brucker
Create empty data files and allow updates for the upcoming VIOT tests.
Acked-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
tests/qtest/bios-tables-test-allowed-diff.h | 3 +++
tests/data/acpi/q35/DSDT.viot | 0
tests/data/acpi/q35/VIOT.viot | 0
tests/data/acpi/virt/VIOT | 0
4 files changed, 3 insertions(+)
create mode 100644 tests/data/acpi/q35/DSDT.viot
create mode 100644 tests/data/acpi/q35/VIOT.viot
create mode 100644 tests/data/acpi/virt/VIOT
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..29b5b1eabc 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1 +1,4 @@
/* List of comma-separated changed AML files to ignore */
+"tests/data/acpi/virt/VIOT",
+"tests/data/acpi/q35/DSDT.viot",
+"tests/data/acpi/q35/VIOT.viot",
diff --git a/tests/data/acpi/q35/DSDT.viot b/tests/data/acpi/q35/DSDT.viot
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/q35/VIOT.viot b/tests/data/acpi/q35/VIOT.viot
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/virt/VIOT b/tests/data/acpi/virt/VIOT
new file mode 100644
index 0000000000..e69de29bb2
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v7 6/8] tests/acpi: add test case for VIOT
2021-12-10 17:04 [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Jean-Philippe Brucker
` (4 preceding siblings ...)
2021-12-10 17:04 ` [PATCH v7 5/8] tests/acpi: allow updates of VIOT expected data files Jean-Philippe Brucker
@ 2021-12-10 17:04 ` Jean-Philippe Brucker
2021-12-10 17:04 ` [PATCH v7 7/8] tests/acpi: add expected blobs for VIOT test on q35 machine Jean-Philippe Brucker
` (3 subsequent siblings)
9 siblings, 0 replies; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-10 17:04 UTC (permalink / raw)
To: mst, imammedo, peter.maydell, shannon.zhaosl
Cc: ani, eric.auger, qemu-arm, qemu-devel, Jean-Philippe Brucker
Add two test cases for VIOT, one on the q35 machine and the other on
virt. To test complex topologies the q35 test has two PCIe buses that
bypass the IOMMU (and are therefore not described by VIOT), and two
buses that are translated by virtio-iommu.
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
tests/qtest/bios-tables-test.c | 38 ++++++++++++++++++++++++++++++++++
1 file changed, 38 insertions(+)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index 258874167e..58df53b15b 100644
--- a/tests/qtest/bios-tables-test.c
+++ b/tests/qtest/bios-tables-test.c
@@ -1465,6 +1465,42 @@ static void test_acpi_virt_tcg(void)
free_test_data(&data);
}
+static void test_acpi_q35_viot(void)
+{
+ test_data data = {
+ .machine = MACHINE_Q35,
+ .variant = ".viot",
+ };
+
+ /*
+ * To keep things interesting, two buses bypass the IOMMU.
+ * VIOT should only describes the other two buses.
+ */
+ test_acpi_one("-machine default_bus_bypass_iommu=on "
+ "-device virtio-iommu-pci "
+ "-device pxb-pcie,bus_nr=0x10,id=pcie.100,bus=pcie.0 "
+ "-device pxb-pcie,bus_nr=0x20,id=pcie.200,bus=pcie.0,bypass_iommu=on "
+ "-device pxb-pcie,bus_nr=0x30,id=pcie.300,bus=pcie.0",
+ &data);
+ free_test_data(&data);
+}
+
+static void test_acpi_virt_viot(void)
+{
+ test_data data = {
+ .machine = "virt",
+ .uefi_fl1 = "pc-bios/edk2-aarch64-code.fd",
+ .uefi_fl2 = "pc-bios/edk2-arm-vars.fd",
+ .cd = "tests/data/uefi-boot-images/bios-tables-test.aarch64.iso.qcow2",
+ .ram_start = 0x40000000ULL,
+ .scan_len = 128ULL * 1024 * 1024,
+ };
+
+ test_acpi_one("-cpu cortex-a57 "
+ "-device virtio-iommu-pci", &data);
+ free_test_data(&data);
+}
+
static void test_oem_fields(test_data *data)
{
int i;
@@ -1639,6 +1675,7 @@ int main(int argc, char *argv[])
qtest_add_func("acpi/q35/kvm/xapic", test_acpi_q35_kvm_xapic);
qtest_add_func("acpi/q35/kvm/dmar", test_acpi_q35_kvm_dmar);
}
+ qtest_add_func("acpi/q35/viot", test_acpi_q35_viot);
} else if (strcmp(arch, "aarch64") == 0) {
if (has_tcg) {
qtest_add_func("acpi/virt", test_acpi_virt_tcg);
@@ -1646,6 +1683,7 @@ int main(int argc, char *argv[])
qtest_add_func("acpi/virt/memhp", test_acpi_virt_tcg_memhp);
qtest_add_func("acpi/virt/pxb", test_acpi_virt_tcg_pxb);
qtest_add_func("acpi/virt/oem-fields", test_acpi_oem_fields_virt);
+ qtest_add_func("acpi/virt/viot", test_acpi_virt_viot);
}
}
ret = g_test_run();
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v7 7/8] tests/acpi: add expected blobs for VIOT test on q35 machine
2021-12-10 17:04 [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Jean-Philippe Brucker
` (5 preceding siblings ...)
2021-12-10 17:04 ` [PATCH v7 6/8] tests/acpi: add test case for VIOT Jean-Philippe Brucker
@ 2021-12-10 17:04 ` Jean-Philippe Brucker
2021-12-10 17:04 ` [PATCH v7 8/8] tests/acpi: add expected blob for VIOT test on virt machine Jean-Philippe Brucker
` (2 subsequent siblings)
9 siblings, 0 replies; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-10 17:04 UTC (permalink / raw)
To: mst, imammedo, peter.maydell, shannon.zhaosl
Cc: ani, eric.auger, qemu-arm, qemu-devel, Jean-Philippe Brucker
Add expected blobs of the VIOT and DSDT table for the VIOT test on the
q35 machine.
Since the test instantiates a virtio device and two PCIe expander
bridges, DSDT.viot has more blocks than the base DSDT.
The VIOT table generated for the q35 test is:
[000h 0000 4] Signature : "VIOT" [Virtual I/O Translation Table]
[004h 0004 4] Table Length : 00000070
[008h 0008 1] Revision : 00
[009h 0009 1] Checksum : 3D
[00Ah 0010 6] Oem ID : "BOCHS "
[010h 0016 8] Oem Table ID : "BXPC "
[018h 0024 4] Oem Revision : 00000001
[01Ch 0028 4] Asl Compiler ID : "BXPC"
[020h 0032 4] Asl Compiler Revision : 00000001
[024h 0036 2] Node count : 0003
[026h 0038 2] Node offset : 0030
[028h 0040 8] Reserved : 0000000000000000
[030h 0048 1] Type : 03 [VirtIO-PCI IOMMU]
[031h 0049 1] Reserved : 00
[032h 0050 2] Length : 0010
[034h 0052 2] PCI Segment : 0000
[036h 0054 2] PCI BDF number : 0010
[038h 0056 8] Reserved : 0000000000000000
[040h 0064 1] Type : 01 [PCI Range]
[041h 0065 1] Reserved : 00
[042h 0066 2] Length : 0018
[044h 0068 4] Endpoint start : 00003000
[048h 0072 2] PCI Segment start : 0000
[04Ah 0074 2] PCI Segment end : 0000
[04Ch 0076 2] PCI BDF start : 3000
[04Eh 0078 2] PCI BDF end : 30FF
[050h 0080 2] Output node : 0030
[052h 0082 6] Reserved : 000000000000
[058h 0088 1] Type : 01 [PCI Range]
[059h 0089 1] Reserved : 00
[05Ah 0090 2] Length : 0018
[05Ch 0092 4] Endpoint start : 00001000
[060h 0096 2] PCI Segment start : 0000
[062h 0098 2] PCI Segment end : 0000
[064h 0100 2] PCI BDF start : 1000
[066h 0102 2] PCI BDF end : 10FF
[068h 0104 2] Output node : 0030
[06Ah 0106 6] Reserved : 000000000000
And the DSDT diff is:
@@ -5,13 +5,13 @@
*
* Disassembling to symbolic ASL+ operators
*
- * Disassembly of tests/data/acpi/q35/DSDT, Fri Dec 10 15:03:08 2021
+ * Disassembly of /tmp/aml-H9Y5D1, Fri Dec 10 15:02:27 2021
*
* Original Table Header:
* Signature "DSDT"
- * Length 0x00002061 (8289)
+ * Length 0x000024B6 (9398)
* Revision 0x01 **** 32-bit table (V1), no 64-bit math support
- * Checksum 0xFA
+ * Checksum 0xA7
* OEM ID "BOCHS "
* OEM Table ID "BXPC "
* OEM Revision 0x00000001 (1)
@@ -3114,6 +3114,339 @@
}
}
+ Scope (\_SB)
+ {
+ Device (PC30)
+ {
+ Name (_UID, 0x30) // _UID: Unique ID
+ Name (_BBN, 0x30) // _BBN: BIOS Bus Number
+ Name (_HID, EisaId ("PNP0A08") /* PCI Express Bus */) // _HID: Hardware ID
+ Name (_CID, EisaId ("PNP0A03") /* PCI Bus */) // _CID: Compatible ID
+ Method (_OSC, 4, NotSerialized) // _OSC: Operating System Capabilities
+ {
+ CreateDWordField (Arg3, Zero, CDW1)
+ If ((Arg0 == ToUUID ("33db4d5b-1ff7-401c-9657-7441c03dd766") /* PCI Host Bridge Device */))
+ {
+ CreateDWordField (Arg3, 0x04, CDW2)
+ CreateDWordField (Arg3, 0x08, CDW3)
+ Local0 = CDW3 /* \_SB_.PC30._OSC.CDW3 */
+ Local0 &= 0x1F
+ If ((Arg1 != One))
+ {
+ CDW1 |= 0x08
+ }
+
+ If ((CDW3 != Local0))
+ {
+ CDW1 |= 0x10
+ }
+
+ CDW3 = Local0
+ }
+ Else
+ {
+ CDW1 |= 0x04
+ }
+
+ Return (Arg3)
+ }
+
+ Method (_PRT, 0, NotSerialized) // _PRT: PCI Routing Table
+ {
+ Local0 = Package (0x80){}
+ Local1 = Zero
+ While ((Local1 < 0x80))
+ {
+ Local2 = (Local1 >> 0x02)
+ Local3 = ((Local1 + Local2) & 0x03)
+ If ((Local3 == Zero))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKD,
+ Zero
+ }
+ }
+
+ If ((Local3 == One))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKA,
+ Zero
+ }
+ }
+
+ If ((Local3 == 0x02))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKB,
+ Zero
+ }
+ }
+
+ If ((Local3 == 0x03))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKC,
+ Zero
+ }
+ }
+
+ Local4 [Zero] = ((Local2 << 0x10) | 0xFFFF)
+ Local4 [One] = (Local1 & 0x03)
+ Local0 [Local1] = Local4
+ Local1++
+ }
+
+ Return (Local0)
+ }
+
+ Name (_CRS, ResourceTemplate () // _CRS: Current Resource Settings
+ {
+ WordBusNumber (ResourceProducer, MinFixed, MaxFixed, PosDecode,
+ 0x0000, // Granularity
+ 0x0030, // Range Minimum
+ 0x0030, // Range Maximum
+ 0x0000, // Translation Offset
+ 0x0001, // Length
+ ,, )
+ })
+ }
+ }
+
+ Scope (\_SB)
+ {
+ Device (PC20)
+ {
+ Name (_UID, 0x20) // _UID: Unique ID
+ Name (_BBN, 0x20) // _BBN: BIOS Bus Number
+ Name (_HID, EisaId ("PNP0A08") /* PCI Express Bus */) // _HID: Hardware ID
+ Name (_CID, EisaId ("PNP0A03") /* PCI Bus */) // _CID: Compatible ID
+ Method (_OSC, 4, NotSerialized) // _OSC: Operating System Capabilities
+ {
+ CreateDWordField (Arg3, Zero, CDW1)
+ If ((Arg0 == ToUUID ("33db4d5b-1ff7-401c-9657-7441c03dd766") /* PCI Host Bridge Device */))
+ {
+ CreateDWordField (Arg3, 0x04, CDW2)
+ CreateDWordField (Arg3, 0x08, CDW3)
+ Local0 = CDW3 /* \_SB_.PC20._OSC.CDW3 */
+ Local0 &= 0x1F
+ If ((Arg1 != One))
+ {
+ CDW1 |= 0x08
+ }
+
+ If ((CDW3 != Local0))
+ {
+ CDW1 |= 0x10
+ }
+
+ CDW3 = Local0
+ }
+ Else
+ {
+ CDW1 |= 0x04
+ }
+
+ Return (Arg3)
+ }
+
+ Method (_PRT, 0, NotSerialized) // _PRT: PCI Routing Table
+ {
+ Local0 = Package (0x80){}
+ Local1 = Zero
+ While ((Local1 < 0x80))
+ {
+ Local2 = (Local1 >> 0x02)
+ Local3 = ((Local1 + Local2) & 0x03)
+ If ((Local3 == Zero))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKD,
+ Zero
+ }
+ }
+
+ If ((Local3 == One))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKA,
+ Zero
+ }
+ }
+
+ If ((Local3 == 0x02))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKB,
+ Zero
+ }
+ }
+
+ If ((Local3 == 0x03))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKC,
+ Zero
+ }
+ }
+
+ Local4 [Zero] = ((Local2 << 0x10) | 0xFFFF)
+ Local4 [One] = (Local1 & 0x03)
+ Local0 [Local1] = Local4
+ Local1++
+ }
+
+ Return (Local0)
+ }
+
+ Name (_CRS, ResourceTemplate () // _CRS: Current Resource Settings
+ {
+ WordBusNumber (ResourceProducer, MinFixed, MaxFixed, PosDecode,
+ 0x0000, // Granularity
+ 0x0020, // Range Minimum
+ 0x0020, // Range Maximum
+ 0x0000, // Translation Offset
+ 0x0001, // Length
+ ,, )
+ })
+ }
+ }
+
+ Scope (\_SB)
+ {
+ Device (PC10)
+ {
+ Name (_UID, 0x10) // _UID: Unique ID
+ Name (_BBN, 0x10) // _BBN: BIOS Bus Number
+ Name (_HID, EisaId ("PNP0A08") /* PCI Express Bus */) // _HID: Hardware ID
+ Name (_CID, EisaId ("PNP0A03") /* PCI Bus */) // _CID: Compatible ID
+ Method (_OSC, 4, NotSerialized) // _OSC: Operating System Capabilities
+ {
+ CreateDWordField (Arg3, Zero, CDW1)
+ If ((Arg0 == ToUUID ("33db4d5b-1ff7-401c-9657-7441c03dd766") /* PCI Host Bridge Device */))
+ {
+ CreateDWordField (Arg3, 0x04, CDW2)
+ CreateDWordField (Arg3, 0x08, CDW3)
+ Local0 = CDW3 /* \_SB_.PC10._OSC.CDW3 */
+ Local0 &= 0x1F
+ If ((Arg1 != One))
+ {
+ CDW1 |= 0x08
+ }
+
+ If ((CDW3 != Local0))
+ {
+ CDW1 |= 0x10
+ }
+
+ CDW3 = Local0
+ }
+ Else
+ {
+ CDW1 |= 0x04
+ }
+
+ Return (Arg3)
+ }
+
+ Method (_PRT, 0, NotSerialized) // _PRT: PCI Routing Table
+ {
+ Local0 = Package (0x80){}
+ Local1 = Zero
+ While ((Local1 < 0x80))
+ {
+ Local2 = (Local1 >> 0x02)
+ Local3 = ((Local1 + Local2) & 0x03)
+ If ((Local3 == Zero))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKD,
+ Zero
+ }
+ }
+
+ If ((Local3 == One))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKA,
+ Zero
+ }
+ }
+
+ If ((Local3 == 0x02))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKB,
+ Zero
+ }
+ }
+
+ If ((Local3 == 0x03))
+ {
+ Local4 = Package (0x04)
+ {
+ Zero,
+ Zero,
+ LNKC,
+ Zero
+ }
+ }
+
+ Local4 [Zero] = ((Local2 << 0x10) | 0xFFFF)
+ Local4 [One] = (Local1 & 0x03)
+ Local0 [Local1] = Local4
+ Local1++
+ }
+
+ Return (Local0)
+ }
+
+ Name (_CRS, ResourceTemplate () // _CRS: Current Resource Settings
+ {
+ WordBusNumber (ResourceProducer, MinFixed, MaxFixed, PosDecode,
+ 0x0000, // Granularity
+ 0x0010, // Range Minimum
+ 0x0010, // Range Maximum
+ 0x0000, // Translation Offset
+ 0x0001, // Length
+ ,, )
+ })
+ }
+ }
+
Scope (\_SB.PCI0)
{
Name (_CRS, ResourceTemplate () // _CRS: Current Resource Settings
@@ -3121,9 +3454,9 @@
WordBusNumber (ResourceProducer, MinFixed, MaxFixed, PosDecode,
0x0000, // Granularity
0x0000, // Range Minimum
- 0x00FF, // Range Maximum
+ 0x000F, // Range Maximum
0x0000, // Translation Offset
- 0x0100, // Length
+ 0x0010, // Length
,, )
IO (Decode16,
0x0CF8, // Range Minimum
@@ -3278,6 +3611,26 @@
}
}
+ Device (S10)
+ {
+ Name (_ADR, 0x00020000) // _ADR: Address
+ }
+
+ Device (S18)
+ {
+ Name (_ADR, 0x00030000) // _ADR: Address
+ }
+
+ Device (S20)
+ {
+ Name (_ADR, 0x00040000) // _ADR: Address
+ }
+
+ Device (S28)
+ {
+ Name (_ADR, 0x00050000) // _ADR: Address
+ }
+
Method (PCNT, 0, NotSerialized)
{
}
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
tests/qtest/bios-tables-test-allowed-diff.h | 2 --
tests/data/acpi/q35/DSDT.viot | Bin 0 -> 9398 bytes
tests/data/acpi/q35/VIOT.viot | Bin 0 -> 112 bytes
3 files changed, 2 deletions(-)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index 29b5b1eabc..8367ffe1d4 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1,4 +1,2 @@
/* List of comma-separated changed AML files to ignore */
"tests/data/acpi/virt/VIOT",
-"tests/data/acpi/q35/DSDT.viot",
-"tests/data/acpi/q35/VIOT.viot",
diff --git a/tests/data/acpi/q35/DSDT.viot b/tests/data/acpi/q35/DSDT.viot
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..1c3b4da5cbe81ecab5e1ef50d383b561c5e0f55f 100644
GIT binary patch
literal 9398
zcmeHNO>7&-8J*>iv|O&FB}G~Oi$yp||57BBoWHhc5OS9yDTx$CQgH$r;8Idr*-4Q_
z5(9Az1F`}niVsB-)<KW7p`g9Br(A2Gm-gmc1N78GFS!;)e2V(MnH_0{q<{#yMgn&C
zn|*J-d9yqFhO_H6z19~`FlPL*u<DkZ*}|)JH;X@mF-FI<cPg<fti9tEN*yB^i5czN
zNq&q?!OZ;BE3B7{KWzJ-`Tn~f`9?Qj8~2^N8{Oc8J%57{==w%rS#;nOCp*nTr@iZ1
zb+?i;JLQUJ=O0?8*>S~D)a>NF1~WVB6^~_B#yhJ`H+JU@=6aXs`?Yv)J2h=N?drcS
zeLZ*n<<Bm^n}6`jfBx#u8&(W}1?)}iF9o#mZ~E2+zwdn7yK3AbIzKnxpZ>JRPm3~#
z&ICS{+_OayRW-l=Mtk=~uaS3o8z<_udd|(wqg`&JnVPfCe>BUOO`Su3e>pff_^UW%
z&JE^NO`)=Amg~iqRB1pPscP?(>#ZuY8GHCmlEvD$9g3%4Db~Dfz2SATnddvrR-Oe^
z;s;dJec!hnzi)ri^I6YN9vtkm{^TdUF8h7gX8-<Qe4p)GQ=)AtYx2VcwdLVAEXEjG
z^Mj|UHPqkj-LsWuzQem1>F3atdZn=zv3$#RmZzSHN+6-yyU#8cJb=YDilX&sl}vNm
znkgAR^O<3kj4if>{ly5fwRfMWuC5=lrlvKPX~i#654Cp}R_d*JS$9laZ$ra6)<ns8
zFZy28G%xP(nit&F>LDi%G<tIc=TY=gl$jSD&Uv!Yat~XR46h%rI$!}a%!|xG7u8Zn
zeY8_|n=K>xz_v_W8VX$W-Fg-qFWcT}7MCyz{%%{ia7hZ>Law-k6NOr}VI&_48U=2l
zwqDKFE8eTwwozDdms#e?x?5a|v>&JF;2_v0L~z5n%BYU^52<*cWuD4|GYUm@1+?))
zte^45>Rz)t*<T5V#={r>@t@{%?^i#W{i=HAZ*Dc9y59Va-+#P!jrGs;u38a{fLr`N
zvT@rUu>DljxJ?^&Z?-?vyJn3C>3D=qux{Y*bs5|5n)Qmi$TD^Zdn4GU$ocJS2Hh-<
z`xPI^^+v0nUVdjMos8k`WGl7hA`{03ju%<lrgAHSpd^DRf-*}_#Ly0mB!LSfVgWcQ
z&T$@~G9)JI=hz5m0vkrel+Xy{Oh7pkAu-V!j*W7rY(bO}Q$nMH2`FbGB&N)QaV4<4
zo)~9JXiP9=;}NPl<C@MmXG&;XFlFNrsyfFsonxFSp<}vEgsRSQP3O3#b6nSnP}ON_
zI!#Tdsp~|j>ckUB>FI=~GokB5sOq#dotCE4(sd$KbtW~PNlj-`*NIToiD#j5J#9^=
zt?NXn>YUJYPG~wObe#xQos*i*NloXZt`niEb4t@WrRki~bs|)CI+{*L)9L6s5vn><
zn$DD_Go|Z9sOn5>I@6lYw5}7Os&iV?Ij!lO)^#FOb!If38BJ$K*NIToIiu;E(R9w}
zIuWWmPiZ<&X*y5oIuWWmF_XaEC!a&Jn$B5WCqh-{X-(&8P3LJ{Cqh-{8P3dyPr@^t
zSqL9?X9Uwd3W@23*s~h*tj0X6GZCuHa~kuU#yqDp5vt7d8uPryJg+kms?5hU=3^T3
zF`bD}WnSP+=`t5MQ$FJ_2&Q~+BP6E0f^%BVIW6a$o)e+SX~IDBih-7z6{O~7YTy`&
zLjy&Cv?7QikV#>n0>>@MV8oK`Gmun34-FKdlm-J8SZSaNlnhir4-FI{S|bfqV8e)V
zss<{chX#reE#g=hsKAC%sF6d-Km}BWs!kZFsFpKfpbC@>6rprQGEjt4Ck#|zITHq|
zK*>M_l;<P^MJRQ`Kn0dFVW0|>3{*fllMEE0)CmI>Sk8ojDo`>|0p(0GP=xY&!axO<
zGhv_#lnhirIg<<&q0|Wj6<E%MfhtfkPyyvkGEjt4Ck#|zITHq|K*>M_lrzad5lWpf
zP=V!47^ngz0~JutBm+e#b;3XemNQ|X3X}{~Ksl2P6rt1!0~J`#gn=qhGEf2KOfpb}
zQYQ>lU^x>8szAv=1(Y+%KoLrvFi?TzOc<yFB?A>u&LjgxD0RX>1(q{mpbC@>R6seC
z3>2Z%2?G^a&V+#~P%=;f<xDbAgi<FARA4z12C6{GKn0XD$v_cGoiI>=<xCi;0wn_#
zP|hR+MJRQ`Kn0dFVW0|>3{*fllMEE0)CmI>Sk8ojDo`>|0p(0GP=rz^3{+q_69%e4
z$v_2^Gs!>^N}VuJf#pmXr~)Me6;RG314Srx!axxz28u{EP=u<1B2)}iVZuNaCK;&0
zBm-5LFi?dF167!0pbC==RAItE6($T+VUmF=Ofpb~2?JG_Fi?d_2C6X0Kouqo6p_5T
zFi=FeV!SiSKoR0H$dH(_Z(*Q_WZ%L-5y`$K14StNmJAdjmWs}HV4<vU_xO+1efmLq
zZ;W>N_U)fP6Qy6Nw5mbt9Y(#emWSi66=>tq#xoh#Ue=0qyhxi8ZOUe5y0V7VfPUhp
zwX=;ymc+i5%sg9Ja~lZ&8oAV@mHc>&CHP9v4R(jhtT?un;O4e9#pno)Xkh7OWgK&a
zyj=3Iv0OuoK_;5rOr5f(Kb~ZXDBO+V`OWYo#_C08imwChQxnjdd?wZLDou8aj;$SD
zGDYiA3<$Tu<JnHL(KPOChi#zrR32t83}naR$+ym4P_h?z_5#|cW-nw$XD_sOtE62l
zrD3@*)NVyiklt0&yF9%+klsBey&I<Y2E<!f(E8TuJte)z(|ZHyy<^gQVfx}=`q&B5
z7nSryp1wGczIaUfVwiq$Fn#<4=@*ssi#+|}K>EdF(l3VTOM~ghPLRH&q%ZOGrGfON
zW73zx^yR_y<0nX8R??Sw`tm^f@-gYlNFSp|*<gA{q?Zp5Oe-+l#rmyYmKozi9y=P>
zVReJU*h=ZuVXiS$ohTbw-O#v9>(yZbGE|)?8(H1ZIKvV!jWa0>vy!3eMA^vdhQ>`s
zuMSg{q3T50$m)j1!HixV<}X9liL#N^4c*tL^y)CF8LCc{jjV3yKAqL8!%SzWI#H%q
z=bSrQ&)%JCRttF5g4Zf`6l?y@>PzD7MA^D>wBlcH6r1ucwJ<p0O%rZ?JzIY3-QdmZ
zzs|n>`a5r3e|z)wcUaqS>nqFQ-8x}eCF4u`OWUxqst-@1rSmUs%WmKP5e0dcb?e2N
z;Z|x*!);VwF|Yuhqs^khqOM!@u*jY!WYldISF(V6`BoNd&6Qfk3>X#SuD^7J>p_D=
zBPa51y^_n#=cpOt#Zf$ya$Ae9Mfz56n|<i!a=ELS@)%a{^NIH3SDuN<R~sah1km#P
zU@?*f%<rG=4W1wgfi;C?_n|W@%lm$&8YfvNOJodIg&IcIpIJQRHr<+ej11GQ6)&eF
z2Lam*jIH}#y0>KnY%4JQfOYS$*uU%f#@$U6`N8I3N-lV?5ErFCdv~xDmu2(wexld4
z4v^;aVAT2k6GJ^m*FD(Wqc(Qg^)6a<?}h$zLoj}4;PP!+(O{@!a1y-hoAhF_7!z+6
zslpAmNtYbjHrw-~#SPVk_FUf>-Obg6yV`8o$8_`PyJe_;bY5_EMBfBfWU!Q=*9HsG
z%_Cda{@_Krr!oHVhv9+y+T5qR8zZ2aZ>5r!$*|f$^U%yBUYfR&B!+EYy_PwL!BeUi
zJH^}r3r9Q+B)X@Z)fk=P13w&7x#wBtXTZ)g>WITPg5r&pQc!nmyrmk#S(>>b9xnNr
zx_b#v9Xv-Y><Wb%?S^0Xe&<)bbKl_=Z|3C$tf|F<bYzE*mfHB;uC)`q-?buaBe?l?
zcLTpK*k<49Z32`K?|nSBMFqxTK^_IE-li2fEGdK~(ZdoKBl6ab4a;Hler#`xvEXJG
zb?<E%EZExfX>jcOVhS*0rS~RS1dA#xhkv@Nct@#q?LyeKS<$uFec!bw>{@uu$gZ6a
zyVen1i{1BKd%~`D7|m$;U0a<I*3I7%^N%N%lGYdU_GS!gaR8T$NA@GzFi~z`l7hdl
zarZy6590|88pi(1zq;V(>38zM0sT&<zX;R5$1w3;`_JMG`;&I&0Y23DMx1%@(w(R9
z4M$j;D5J+Gy%fijRQsctzFKf&cv|BAz#YLq3CZJWDdtL4u1u1|mkdcUp7|sxJC+?Y
z_@@s`v3j}Q7*z>6X~cwUxUL8G1KT)_XTp!KAbs;vCp{K3&~_X@+ew=-D}v`2MbFV0
zQsVsL=rXi-pI*G|iiz;VTCutgUs)hDzV1+4?8KcoP3xROf<M%qC6lgVdpFt4<-|uM
z=#rl_b1#YjSIl6Toj2z_hOZcKupkdE(LozC(fN=FY(x|sk)ym|;Rq2E1xJWD%Z!ol
Gu>S+TT-130
literal 0
HcmV?d00001
diff --git a/tests/data/acpi/q35/VIOT.viot b/tests/data/acpi/q35/VIOT.viot
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..9b179266ccbf84f1c250ee646812d17e27987764 100644
GIT binary patch
literal 112
zcmWIZ^baXu00LVle`k+i1*eDrX9XZ&1PX!JAex!M0Hgv8m>C3sGzdcgBZCA3T-xBj
Q0Zb)W9Hva*zW_`e0M!8s0RR91
literal 0
HcmV?d00001
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* [PATCH v7 8/8] tests/acpi: add expected blob for VIOT test on virt machine
2021-12-10 17:04 [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Jean-Philippe Brucker
` (6 preceding siblings ...)
2021-12-10 17:04 ` [PATCH v7 7/8] tests/acpi: add expected blobs for VIOT test on q35 machine Jean-Philippe Brucker
@ 2021-12-10 17:04 ` Jean-Philippe Brucker
2021-12-12 4:49 ` [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Ani Sinha
2021-12-13 17:50 ` Michael S. Tsirkin
9 siblings, 0 replies; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-10 17:04 UTC (permalink / raw)
To: mst, imammedo, peter.maydell, shannon.zhaosl
Cc: ani, eric.auger, qemu-arm, qemu-devel, Jean-Philippe Brucker
The VIOT blob contains the following:
[000h 0000 4] Signature : "VIOT" [Virtual I/O Translation Table]
[004h 0004 4] Table Length : 00000058
[008h 0008 1] Revision : 00
[009h 0009 1] Checksum : 66
[00Ah 0010 6] Oem ID : "BOCHS "
[010h 0016 8] Oem Table ID : "BXPC "
[018h 0024 4] Oem Revision : 00000001
[01Ch 0028 4] Asl Compiler ID : "BXPC"
[020h 0032 4] Asl Compiler Revision : 00000001
[024h 0036 2] Node count : 0002
[026h 0038 2] Node offset : 0030
[028h 0040 8] Reserved : 0000000000000000
[030h 0048 1] Type : 03 [VirtIO-PCI IOMMU]
[031h 0049 1] Reserved : 00
[032h 0050 2] Length : 0010
[034h 0052 2] PCI Segment : 0000
[036h 0054 2] PCI BDF number : 0008
[038h 0056 8] Reserved : 0000000000000000
[040h 0064 1] Type : 01 [PCI Range]
[041h 0065 1] Reserved : 00
[042h 0066 2] Length : 0018
[044h 0068 4] Endpoint start : 00000000
[048h 0072 2] PCI Segment start : 0000
[04Ah 0074 2] PCI Segment end : 0000
[04Ch 0076 2] PCI BDF start : 0000
[04Eh 0078 2] PCI BDF end : 00FF
[050h 0080 2] Output node : 0030
[052h 0082 6] Reserved : 000000000000
Acked-by: Ani Sinha <ani@anisinha.ca>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
tests/qtest/bios-tables-test-allowed-diff.h | 1 -
tests/data/acpi/virt/VIOT | Bin 0 -> 88 bytes
2 files changed, 1 deletion(-)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index 8367ffe1d4..dfb8523c8b 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1,2 +1 @@
/* List of comma-separated changed AML files to ignore */
-"tests/data/acpi/virt/VIOT",
diff --git a/tests/data/acpi/virt/VIOT b/tests/data/acpi/virt/VIOT
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..921f40d88c28ba2171a4d664e119914335309e7d 100644
GIT binary patch
literal 88
zcmWIZ^bd((0D?3pe`k+i1*eDrX9XZ&1PX!JAexE60Hgv8m>C3sGzXN&z`)2L0cSHX
I{D-Rq0Q5fy0RR91
literal 0
HcmV?d00001
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread* Re: [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests)
2021-12-10 17:04 [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Jean-Philippe Brucker
` (7 preceding siblings ...)
2021-12-10 17:04 ` [PATCH v7 8/8] tests/acpi: add expected blob for VIOT test on virt machine Jean-Philippe Brucker
@ 2021-12-12 4:49 ` Ani Sinha
2021-12-13 9:28 ` Jean-Philippe Brucker
2021-12-13 17:50 ` Michael S. Tsirkin
9 siblings, 1 reply; 16+ messages in thread
From: Ani Sinha @ 2021-12-12 4:49 UTC (permalink / raw)
To: Jean-Philippe Brucker
Cc: peter.maydell, eric.auger, mst, qemu-devel, shannon.zhaosl,
qemu-arm, imammedo
On Fri, Dec 10, 2021 at 10:35 PM Jean-Philippe Brucker
<jean-philippe@linaro.org> wrote:
>
> Add ACPI support for virtio-iommu on the virt machine, by instantiating
> a VIOT table. Also add the tests for the ACPI table.
>
> Since last posting [1], I rebased onto v6.2.0-rc4. Note that v6 of this
> series [2] only contained the table and x86 support, which have been
> merged. Everything has now been reviewed and should be good to go.
>
> * Patches 1-2 add the VIOT table for the virt machine
> * Patches 3-4 are minor fixes
> * Patches 5-8 add tests for the VIOT table. They contain the tests for
> q35 as well, which didn't make it last time because they depended on
> another fix that has now been merged.
I believe the entire patchset has been reviewed and you are re-sending
it so that it gets pulled in after the release?
>
> [1] https://lore.kernel.org/qemu-devel/20211020172745.620101-1-jean-philippe@linaro.org/
> [2] https://lore.kernel.org/qemu-devel/20211026182024.2642038-1-jean-philippe@linaro.org/
>
> Jean-Philippe Brucker (8):
> hw/arm/virt-acpi-build: Add VIOT table for virtio-iommu
> hw/arm/virt: Remove device tree restriction for virtio-iommu
> hw/arm/virt: Reject instantiation of multiple IOMMUs
> hw/arm/virt: Use object_property_set instead of qdev_prop_set
> tests/acpi: allow updates of VIOT expected data files
> tests/acpi: add test case for VIOT
> tests/acpi: add expected blobs for VIOT test on q35 machine
> tests/acpi: add expected blob for VIOT test on virt machine
>
> hw/arm/virt-acpi-build.c | 7 ++++++
> hw/arm/virt.c | 20 ++++++++---------
> hw/virtio/virtio-iommu-pci.c | 12 ++---------
> tests/qtest/bios-tables-test.c | 38 +++++++++++++++++++++++++++++++++
> hw/arm/Kconfig | 1 +
> tests/data/acpi/q35/DSDT.viot | Bin 0 -> 9398 bytes
> tests/data/acpi/q35/VIOT.viot | Bin 0 -> 112 bytes
> tests/data/acpi/virt/VIOT | Bin 0 -> 88 bytes
> 8 files changed, 58 insertions(+), 20 deletions(-)
> create mode 100644 tests/data/acpi/q35/DSDT.viot
> create mode 100644 tests/data/acpi/q35/VIOT.viot
> create mode 100644 tests/data/acpi/virt/VIOT
>
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 16+ messages in thread* Re: [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests)
2021-12-12 4:49 ` [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Ani Sinha
@ 2021-12-13 9:28 ` Jean-Philippe Brucker
2021-12-13 10:04 ` Peter Maydell
0 siblings, 1 reply; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-13 9:28 UTC (permalink / raw)
To: Ani Sinha
Cc: peter.maydell, eric.auger, mst, qemu-devel, shannon.zhaosl,
qemu-arm, imammedo
On Sun, Dec 12, 2021 at 10:19:47AM +0530, Ani Sinha wrote:
> On Fri, Dec 10, 2021 at 10:35 PM Jean-Philippe Brucker
> <jean-philippe@linaro.org> wrote:
> >
> > Add ACPI support for virtio-iommu on the virt machine, by instantiating
> > a VIOT table. Also add the tests for the ACPI table.
> >
> > Since last posting [1], I rebased onto v6.2.0-rc4. Note that v6 of this
> > series [2] only contained the table and x86 support, which have been
> > merged. Everything has now been reviewed and should be good to go.
> >
> > * Patches 1-2 add the VIOT table for the virt machine
> > * Patches 3-4 are minor fixes
> > * Patches 5-8 add tests for the VIOT table. They contain the tests for
> > q35 as well, which didn't make it last time because they depended on
> > another fix that has now been merged.
>
> I believe the entire patchset has been reviewed and you are re-sending
> it so that it gets pulled in after the release?
Yes
Thanks,
Jean
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests)
2021-12-13 9:28 ` Jean-Philippe Brucker
@ 2021-12-13 10:04 ` Peter Maydell
2021-12-13 10:56 ` Jean-Philippe Brucker
0 siblings, 1 reply; 16+ messages in thread
From: Peter Maydell @ 2021-12-13 10:04 UTC (permalink / raw)
To: Jean-Philippe Brucker
Cc: eric.auger, mst, qemu-devel, shannon.zhaosl, qemu-arm, Ani Sinha,
imammedo
On Mon, 13 Dec 2021 at 09:28, Jean-Philippe Brucker
<jean-philippe@linaro.org> wrote:
>
> On Sun, Dec 12, 2021 at 10:19:47AM +0530, Ani Sinha wrote:
> > On Fri, Dec 10, 2021 at 10:35 PM Jean-Philippe Brucker
> > <jean-philippe@linaro.org> wrote:
> > >
> > > Add ACPI support for virtio-iommu on the virt machine, by instantiating
> > > a VIOT table. Also add the tests for the ACPI table.
> > >
> > > Since last posting [1], I rebased onto v6.2.0-rc4. Note that v6 of this
> > > series [2] only contained the table and x86 support, which have been
> > > merged. Everything has now been reviewed and should be good to go.
> > >
> > > * Patches 1-2 add the VIOT table for the virt machine
> > > * Patches 3-4 are minor fixes
> > > * Patches 5-8 add tests for the VIOT table. They contain the tests for
> > > q35 as well, which didn't make it last time because they depended on
> > > another fix that has now been merged.
> >
> > I believe the entire patchset has been reviewed and you are re-sending
> > it so that it gets pulled in after the release?
>
> Yes
To be clear, do you mean:
(1) The patchset has been reviewed, and I plan to resend it (ie a v8)
to be pulled in after 6.2 releases
or
(2) The patchset has been reviewed already as v6, and this (v7) was just
a resend to be pulled in after 6.2 releases
?
thanks
-- PMM
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests)
2021-12-13 10:04 ` Peter Maydell
@ 2021-12-13 10:56 ` Jean-Philippe Brucker
2021-12-13 11:07 ` Peter Maydell
0 siblings, 1 reply; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-13 10:56 UTC (permalink / raw)
To: Peter Maydell
Cc: eric.auger, mst, qemu-devel, shannon.zhaosl, qemu-arm, Ani Sinha,
imammedo
On Mon, Dec 13, 2021 at 10:04:57AM +0000, Peter Maydell wrote:
> On Mon, 13 Dec 2021 at 09:28, Jean-Philippe Brucker
> <jean-philippe@linaro.org> wrote:
> >
> > On Sun, Dec 12, 2021 at 10:19:47AM +0530, Ani Sinha wrote:
> > > On Fri, Dec 10, 2021 at 10:35 PM Jean-Philippe Brucker
> > > <jean-philippe@linaro.org> wrote:
> > > >
> > > > Add ACPI support for virtio-iommu on the virt machine, by instantiating
> > > > a VIOT table. Also add the tests for the ACPI table.
> > > >
> > > > Since last posting [1], I rebased onto v6.2.0-rc4. Note that v6 of this
> > > > series [2] only contained the table and x86 support, which have been
> > > > merged. Everything has now been reviewed and should be good to go.
> > > >
> > > > * Patches 1-2 add the VIOT table for the virt machine
> > > > * Patches 3-4 are minor fixes
> > > > * Patches 5-8 add tests for the VIOT table. They contain the tests for
> > > > q35 as well, which didn't make it last time because they depended on
> > > > another fix that has now been merged.
> > >
> > > I believe the entire patchset has been reviewed and you are re-sending
> > > it so that it gets pulled in after the release?
> >
> > Yes
>
> To be clear, do you mean:
> (1) The patchset has been reviewed, and I plan to resend it (ie a v8)
> to be pulled in after 6.2 releases
> or
> (2) The patchset has been reviewed already as v6, and this (v7) was just
> a resend to be pulled in after 6.2 releases
I meant (2), v6 had already been reviewed and I'm resending v7 to be
pulled after 6.2 releases, sorry about the confusion. Just for my
understanding, should I have resent this after the 6.2 release, or is this
OK and I should just state the intention for a patchset more clearly?
Thanks,
Jean
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests)
2021-12-13 10:56 ` Jean-Philippe Brucker
@ 2021-12-13 11:07 ` Peter Maydell
2021-12-13 14:58 ` Jean-Philippe Brucker
0 siblings, 1 reply; 16+ messages in thread
From: Peter Maydell @ 2021-12-13 11:07 UTC (permalink / raw)
To: Jean-Philippe Brucker
Cc: eric.auger, mst, qemu-devel, shannon.zhaosl, qemu-arm, Ani Sinha,
imammedo
On Mon, 13 Dec 2021 at 10:56, Jean-Philippe Brucker
<jean-philippe@linaro.org> wrote:
>
> On Mon, Dec 13, 2021 at 10:04:57AM +0000, Peter Maydell wrote:
> > On Mon, 13 Dec 2021 at 09:28, Jean-Philippe Brucker
> > <jean-philippe@linaro.org> wrote:
> > >
> > > On Sun, Dec 12, 2021 at 10:19:47AM +0530, Ani Sinha wrote:
> > > > On Fri, Dec 10, 2021 at 10:35 PM Jean-Philippe Brucker
> > > > <jean-philippe@linaro.org> wrote:
> > > > >
> > > > > Add ACPI support for virtio-iommu on the virt machine, by instantiating
> > > > > a VIOT table. Also add the tests for the ACPI table.
> > > > >
> > > > > Since last posting [1], I rebased onto v6.2.0-rc4. Note that v6 of this
> > > > > series [2] only contained the table and x86 support, which have been
> > > > > merged. Everything has now been reviewed and should be good to go.
> > > > >
> > > > > * Patches 1-2 add the VIOT table for the virt machine
> > > > > * Patches 3-4 are minor fixes
> > > > > * Patches 5-8 add tests for the VIOT table. They contain the tests for
> > > > > q35 as well, which didn't make it last time because they depended on
> > > > > another fix that has now been merged.
> > > >
> > > > I believe the entire patchset has been reviewed and you are re-sending
> > > > it so that it gets pulled in after the release?
> > >
> > > Yes
> >
> > To be clear, do you mean:
> > (1) The patchset has been reviewed, and I plan to resend it (ie a v8)
> > to be pulled in after 6.2 releases
> > or
> > (2) The patchset has been reviewed already as v6, and this (v7) was just
> > a resend to be pulled in after 6.2 releases
>
> I meant (2), v6 had already been reviewed and I'm resending v7 to be
> pulled after 6.2 releases, sorry about the confusion. Just for my
> understanding, should I have resent this after the 6.2 release, or is this
> OK and I should just state the intention for a patchset more clearly?
Sending for-7.0 patchsets before 6.2 officially releases is fine; I just wanted
to check what you intended. I've queued this patchset to target-arm.next.
thanks
-- PMM
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests)
2021-12-13 11:07 ` Peter Maydell
@ 2021-12-13 14:58 ` Jean-Philippe Brucker
0 siblings, 0 replies; 16+ messages in thread
From: Jean-Philippe Brucker @ 2021-12-13 14:58 UTC (permalink / raw)
To: Peter Maydell
Cc: eric.auger, mst, qemu-devel, shannon.zhaosl, qemu-arm, Ani Sinha,
imammedo
On Mon, Dec 13, 2021 at 11:07:38AM +0000, Peter Maydell wrote:
> On Mon, 13 Dec 2021 at 10:56, Jean-Philippe Brucker
> <jean-philippe@linaro.org> wrote:
> >
> > On Mon, Dec 13, 2021 at 10:04:57AM +0000, Peter Maydell wrote:
> > > On Mon, 13 Dec 2021 at 09:28, Jean-Philippe Brucker
> > > <jean-philippe@linaro.org> wrote:
> > > >
> > > > On Sun, Dec 12, 2021 at 10:19:47AM +0530, Ani Sinha wrote:
> > > > > On Fri, Dec 10, 2021 at 10:35 PM Jean-Philippe Brucker
> > > > > <jean-philippe@linaro.org> wrote:
> > > > > >
> > > > > > Add ACPI support for virtio-iommu on the virt machine, by instantiating
> > > > > > a VIOT table. Also add the tests for the ACPI table.
> > > > > >
> > > > > > Since last posting [1], I rebased onto v6.2.0-rc4. Note that v6 of this
> > > > > > series [2] only contained the table and x86 support, which have been
> > > > > > merged. Everything has now been reviewed and should be good to go.
> > > > > >
> > > > > > * Patches 1-2 add the VIOT table for the virt machine
> > > > > > * Patches 3-4 are minor fixes
> > > > > > * Patches 5-8 add tests for the VIOT table. They contain the tests for
> > > > > > q35 as well, which didn't make it last time because they depended on
> > > > > > another fix that has now been merged.
> > > > >
> > > > > I believe the entire patchset has been reviewed and you are re-sending
> > > > > it so that it gets pulled in after the release?
> > > >
> > > > Yes
> > >
> > > To be clear, do you mean:
> > > (1) The patchset has been reviewed, and I plan to resend it (ie a v8)
> > > to be pulled in after 6.2 releases
> > > or
> > > (2) The patchset has been reviewed already as v6, and this (v7) was just
> > > a resend to be pulled in after 6.2 releases
> >
> > I meant (2), v6 had already been reviewed and I'm resending v7 to be
> > pulled after 6.2 releases, sorry about the confusion. Just for my
> > understanding, should I have resent this after the 6.2 release, or is this
> > OK and I should just state the intention for a patchset more clearly?
>
> Sending for-7.0 patchsets before 6.2 officially releases is fine; I just wanted
> to check what you intended. I've queued this patchset to target-arm.next.
Great, thank you!
Jean
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests)
2021-12-10 17:04 [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Jean-Philippe Brucker
` (8 preceding siblings ...)
2021-12-12 4:49 ` [PATCH v7 0/8] virtio-iommu: Add ACPI support (Arm part + tests) Ani Sinha
@ 2021-12-13 17:50 ` Michael S. Tsirkin
9 siblings, 0 replies; 16+ messages in thread
From: Michael S. Tsirkin @ 2021-12-13 17:50 UTC (permalink / raw)
To: Jean-Philippe Brucker
Cc: peter.maydell, eric.auger, qemu-devel, shannon.zhaosl, qemu-arm,
ani, imammedo
On Fri, Dec 10, 2021 at 05:04:08PM +0000, Jean-Philippe Brucker wrote:
> Add ACPI support for virtio-iommu on the virt machine, by instantiating
> a VIOT table. Also add the tests for the ACPI table.
ACPI parts:
Acked-by: Michael S. Tsirkin <mst@redhat.com>
> Since last posting [1], I rebased onto v6.2.0-rc4. Note that v6 of this
> series [2] only contained the table and x86 support, which have been
> merged. Everything has now been reviewed and should be good to go.
>
> * Patches 1-2 add the VIOT table for the virt machine
> * Patches 3-4 are minor fixes
> * Patches 5-8 add tests for the VIOT table. They contain the tests for
> q35 as well, which didn't make it last time because they depended on
> another fix that has now been merged.
>
> [1] https://lore.kernel.org/qemu-devel/20211020172745.620101-1-jean-philippe@linaro.org/
> [2] https://lore.kernel.org/qemu-devel/20211026182024.2642038-1-jean-philippe@linaro.org/
>
> Jean-Philippe Brucker (8):
> hw/arm/virt-acpi-build: Add VIOT table for virtio-iommu
> hw/arm/virt: Remove device tree restriction for virtio-iommu
> hw/arm/virt: Reject instantiation of multiple IOMMUs
> hw/arm/virt: Use object_property_set instead of qdev_prop_set
> tests/acpi: allow updates of VIOT expected data files
> tests/acpi: add test case for VIOT
> tests/acpi: add expected blobs for VIOT test on q35 machine
> tests/acpi: add expected blob for VIOT test on virt machine
>
> hw/arm/virt-acpi-build.c | 7 ++++++
> hw/arm/virt.c | 20 ++++++++---------
> hw/virtio/virtio-iommu-pci.c | 12 ++---------
> tests/qtest/bios-tables-test.c | 38 +++++++++++++++++++++++++++++++++
> hw/arm/Kconfig | 1 +
> tests/data/acpi/q35/DSDT.viot | Bin 0 -> 9398 bytes
> tests/data/acpi/q35/VIOT.viot | Bin 0 -> 112 bytes
> tests/data/acpi/virt/VIOT | Bin 0 -> 88 bytes
> 8 files changed, 58 insertions(+), 20 deletions(-)
> create mode 100644 tests/data/acpi/q35/DSDT.viot
> create mode 100644 tests/data/acpi/q35/VIOT.viot
> create mode 100644 tests/data/acpi/virt/VIOT
>
> --
> 2.34.1
^ permalink raw reply [flat|nested] 16+ messages in thread