diff options
| author | Peter Xu | 2016-10-31 08:34:39 +0100 |
|---|---|---|
| committer | Michael S. Tsirkin | 2016-11-15 16:20:36 +0100 |
| commit | 1b39bc1cf67eee07518ee05ce9306eaa53d868e4 (patch) | |
| tree | 448840ed044f2144648a3233e44ce8162ea59154 /hw/i386/acpi-build.c | |
| parent | intel_iommu: fix incorrect assert (diff) | |
| download | qemu-1b39bc1cf67eee07518ee05ce9306eaa53d868e4.tar.gz qemu-1b39bc1cf67eee07518ee05ce9306eaa53d868e4.tar.xz qemu-1b39bc1cf67eee07518ee05ce9306eaa53d868e4.zip | |
acpi: fix DMAR device scope for IOAPIC
We should not use cpu_to_le16() here, instead each of device/function
value is stored in a 8 byte field.
Signed-off-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'hw/i386/acpi-build.c')
| -rw-r--r-- | hw/i386/acpi-build.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index c02f408ab2..a15585717a 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi-build.c @@ -2605,7 +2605,8 @@ build_dmar_q35(GArray *table_data, BIOSLinker *linker) scope->length = ioapic_scope_size; scope->enumeration_id = ACPI_BUILD_IOAPIC_ID; scope->bus = Q35_PSEUDO_BUS_PLATFORM; - scope->path[0] = cpu_to_le16(Q35_PSEUDO_DEVFN_IOAPIC); + scope->path[0].device = PCI_SLOT(Q35_PSEUDO_DEVFN_IOAPIC); + scope->path[0].function = PCI_FUNC(Q35_PSEUDO_DEVFN_IOAPIC); build_header(linker, table_data, (void *)(table_data->data + dmar_start), "DMAR", table_data->len - dmar_start, 1, NULL, NULL); |