|
 |
76daa3 |
From c10d3ce55f7e0adfceb61559c5b37984ca7010e5 Mon Sep 17 00:00:00 2001
|
|
|
76daa3 |
From: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
76daa3 |
Date: Thu, 18 May 2017 17:57:08 +0200
|
|
|
76daa3 |
Subject: [PATCH 22/27] iommu: Don't crash if machine is not PC_MACHINE
|
|
|
76daa3 |
|
|
|
76daa3 |
RH-Author: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
76daa3 |
Message-id: <20170518175708.22623-1-ehabkost@redhat.com>
|
|
|
76daa3 |
Patchwork-id: 75335
|
|
|
76daa3 |
O-Subject: [RHV-7.4 qemu-kvm-rhev PATCH] iommu: Don't crash if machine is not PC_MACHINE
|
|
|
76daa3 |
Bugzilla: 1451483
|
|
|
76daa3 |
RH-Acked-by: Peter Xu <peterx@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Thomas Huth <thuth@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Marcel Apfelbaum <marcel@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
Currently it's possible to crash QEMU using "-device *-iommu" and
|
|
|
76daa3 |
"-machine none":
|
|
|
76daa3 |
|
|
|
76daa3 |
$ qemu-system-x86_64 -machine none -device amd-iommu
|
|
|
76daa3 |
qemu/hw/i386/amd_iommu.c:1140:amdvi_realize: Object 0x55627dafbc90 is not an instance of type generic-pc-machine
|
|
|
76daa3 |
Aborted (core dumped)
|
|
|
76daa3 |
$ qemu-system-x86_64 -machine none -device intel-iommu
|
|
|
76daa3 |
qemu/hw/i386/intel_iommu.c:2972:vtd_realize: Object 0x56292ec0bc90 is not an instance of type generic-pc-machine
|
|
|
76daa3 |
Aborted (core dumped)
|
|
|
76daa3 |
|
|
|
76daa3 |
Fix amd-iommu and intel-iommu to ensure the current machine is really a
|
|
|
76daa3 |
TYPE_PC_MACHINE instance at their realize methods.
|
|
|
76daa3 |
|
|
|
76daa3 |
Resulting error messages:
|
|
|
76daa3 |
|
|
|
76daa3 |
$ qemu-system-x86_64 -machine none -device amd-iommu
|
|
|
76daa3 |
qemu-system-x86_64: -device amd-iommu: Machine-type 'none' not supported by amd-iommu
|
|
|
76daa3 |
$ qemu-system-x86_64 -machine none -device intel-iommu
|
|
|
76daa3 |
qemu-system-x86_64: -device intel-iommu: Machine-type 'none' not supported by intel-iommu
|
|
|
76daa3 |
|
|
|
76daa3 |
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
76daa3 |
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
76daa3 |
(cherry picked from commit ef0e8fc768a561dd13a86420b3268f6f3d5d0621)
|
|
|
76daa3 |
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
|
|
|
76daa3 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
76daa3 |
---
|
|
|
76daa3 |
hw/i386/amd_iommu.c | 15 ++++++++++++++-
|
|
|
76daa3 |
hw/i386/intel_iommu.c | 14 ++++++++++++--
|
|
|
76daa3 |
2 files changed, 26 insertions(+), 3 deletions(-)
|
|
|
76daa3 |
|
|
|
76daa3 |
diff --git a/hw/i386/amd_iommu.c b/hw/i386/amd_iommu.c
|
|
|
76daa3 |
index f86a40a..516ebae 100644
|
|
|
76daa3 |
--- a/hw/i386/amd_iommu.c
|
|
|
76daa3 |
+++ b/hw/i386/amd_iommu.c
|
|
|
76daa3 |
@@ -21,6 +21,7 @@
|
|
|
76daa3 |
*/
|
|
|
76daa3 |
#include "qemu/osdep.h"
|
|
|
76daa3 |
#include "hw/i386/amd_iommu.h"
|
|
|
76daa3 |
+#include "qapi/error.h"
|
|
|
76daa3 |
#include "qemu/error-report.h"
|
|
|
76daa3 |
#include "trace.h"
|
|
|
76daa3 |
|
|
|
76daa3 |
@@ -1137,7 +1138,19 @@ static void amdvi_realize(DeviceState *dev, Error **err)
|
|
|
76daa3 |
int ret = 0;
|
|
|
76daa3 |
AMDVIState *s = AMD_IOMMU_DEVICE(dev);
|
|
|
76daa3 |
X86IOMMUState *x86_iommu = X86_IOMMU_DEVICE(dev);
|
|
|
76daa3 |
- PCIBus *bus = PC_MACHINE(qdev_get_machine())->bus;
|
|
|
76daa3 |
+ MachineState *ms = MACHINE(qdev_get_machine());
|
|
|
76daa3 |
+ MachineClass *mc = MACHINE_GET_CLASS(ms);
|
|
|
76daa3 |
+ PCMachineState *pcms =
|
|
|
76daa3 |
+ PC_MACHINE(object_dynamic_cast(OBJECT(ms), TYPE_PC_MACHINE));
|
|
|
76daa3 |
+ PCIBus *bus;
|
|
|
76daa3 |
+
|
|
|
76daa3 |
+ if (!pcms) {
|
|
|
76daa3 |
+ error_setg(err, "Machine-type '%s' not supported by amd-iommu",
|
|
|
76daa3 |
+ mc->name);
|
|
|
76daa3 |
+ return;
|
|
|
76daa3 |
+ }
|
|
|
76daa3 |
+
|
|
|
76daa3 |
+ bus = pcms->bus;
|
|
|
76daa3 |
s->iotlb = g_hash_table_new_full(amdvi_uint64_hash,
|
|
|
76daa3 |
amdvi_uint64_equal, g_free, g_free);
|
|
|
76daa3 |
|
|
|
76daa3 |
diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
|
|
|
76daa3 |
index 02f047c..a12b176 100644
|
|
|
76daa3 |
--- a/hw/i386/intel_iommu.c
|
|
|
76daa3 |
+++ b/hw/i386/intel_iommu.c
|
|
|
76daa3 |
@@ -2969,11 +2969,21 @@ static bool vtd_decide_config(IntelIOMMUState *s, Error **errp)
|
|
|
76daa3 |
|
|
|
76daa3 |
static void vtd_realize(DeviceState *dev, Error **errp)
|
|
|
76daa3 |
{
|
|
|
76daa3 |
- PCMachineState *pcms = PC_MACHINE(qdev_get_machine());
|
|
|
76daa3 |
- PCIBus *bus = pcms->bus;
|
|
|
76daa3 |
+ MachineState *ms = MACHINE(qdev_get_machine());
|
|
|
76daa3 |
+ MachineClass *mc = MACHINE_GET_CLASS(ms);
|
|
|
76daa3 |
+ PCMachineState *pcms =
|
|
|
76daa3 |
+ PC_MACHINE(object_dynamic_cast(OBJECT(ms), TYPE_PC_MACHINE));
|
|
|
76daa3 |
+ PCIBus *bus;
|
|
|
76daa3 |
IntelIOMMUState *s = INTEL_IOMMU_DEVICE(dev);
|
|
|
76daa3 |
X86IOMMUState *x86_iommu = X86_IOMMU_DEVICE(dev);
|
|
|
76daa3 |
|
|
|
76daa3 |
+ if (!pcms) {
|
|
|
76daa3 |
+ error_setg(errp, "Machine-type '%s' not supported by intel-iommu",
|
|
|
76daa3 |
+ mc->name);
|
|
|
76daa3 |
+ return;
|
|
|
76daa3 |
+ }
|
|
|
76daa3 |
+
|
|
|
76daa3 |
+ bus = pcms->bus;
|
|
|
76daa3 |
VTD_DPRINTF(GENERAL, "");
|
|
|
76daa3 |
x86_iommu->type = TYPE_INTEL;
|
|
|
76daa3 |
|
|
|
76daa3 |
--
|
|
|
76daa3 |
1.8.3.1
|
|
|
76daa3 |
|