Skip to contentSkip to conversation

piix4: disable bridge acpi hotplug for 1.7 and older

PATCH In review Michael S. Tsirkin 2 participants 2 messages Jan 20, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 piix4: disable bridge acpi hotplug for 1.7 and older

Patch was written before 1.7 was out, forgot to update.

---

I moved for_anthony tag to include this patch
as well, no reason to make people suffer until
next pull request.

 hw/i386/pc_piix.c    |  4 ++++
 include/hw/i386/pc.h | 13 ++++++++-----
 2 files changed, 12 insertions(+), 5 deletions(-)
hw/i386/pc_piix.c +4 -0
index 2766414..a327d71 100644
@@ -393,6 +393,10 @@ static QEMUMachine pc_i440fx_machine_v1_7 = {
     PC_I440FX_1_7_MACHINE_OPTIONS,
     .name = "pc-i440fx-1.7",
     .init = pc_init_pci_1_7,
+    .compat_props = (GlobalProperty[]) {
+        PC_COMPAT_1_7,
+        { /* end of list */ }
+    },
 };
 
 #define PC_I440FX_1_6_MACHINE_OPTIONS PC_I440FX_MACHINE_OPTIONS
include/hw/i386/pc.h +8 -5
index 71653e8..fd9f369 100644
@@ -240,7 +240,15 @@ uint16_t pvpanic_port(void);
 
 int e820_add_entry(uint64_t, uint64_t, uint32_t);
 
+#define PC_COMPAT_1_7 \
+        {\
+            .driver   = "PIIX4_PM",\
+            .property = "acpi-pci-hotplug-with-bridge-support",\
+            .value    = "off",\
+        }
+
 #define PC_Q35_COMPAT_1_7 \
+        PC_COMPAT_1_7, \
         {\
             .driver   = "hpet",\
             .property = HPET_INTCAP,\
@@ -261,11 +269,6 @@ int e820_add_entry(uint64_t, uint64_t, uint32_t);
 
 #define PC_COMPAT_1_6 \
         {\
-            .driver   = "PIIX4_PM",\
-            .property = "acpi-pci-hotplug-with-bridge-support",\
-            .value    = "off",\
-        }, \
-        {\
             .driver   = "e1000",\
             .property = "mitigation",\
             .value    = "off",\
Signature · 1 line
MST
Reviewed-by on v1 1/1
#1 lore raw
Quoted 8 lines · On Mon, 20 Jan 2014 18:35:17 +0200 "Michael S. Tsirkin" <[email protected]> wrote:

Patch was written before 1.7 was out, forgot to update.

---

I moved for_anthony tag to include this patch
as well, no reason to make people suffer until
next pull request.

Quoted 52 lines
 hw/i386/pc_piix.c    |  4 ++++
 include/hw/i386/pc.h | 13 ++++++++-----
 2 files changed, 12 insertions(+), 5 deletions(-)
hw/i386/pc_piix.c +4 -0
index 2766414..a327d71 100644
@@ -393,6 +393,10 @@ static QEMUMachine pc_i440fx_machine_v1_7 = {
     PC_I440FX_1_7_MACHINE_OPTIONS,
     .name = "pc-i440fx-1.7",
     .init = pc_init_pci_1_7,
+    .compat_props = (GlobalProperty[]) {
+        PC_COMPAT_1_7,
+        { /* end of list */ }
+    },
 };
 
 #define PC_I440FX_1_6_MACHINE_OPTIONS PC_I440FX_MACHINE_OPTIONS
include/hw/i386/pc.h +8 -5
index 71653e8..fd9f369 100644
@@ -240,7 +240,15 @@ uint16_t pvpanic_port(void);
 
 int e820_add_entry(uint64_t, uint64_t, uint32_t);
 
+#define PC_COMPAT_1_7 \
+        {\
+            .driver   = "PIIX4_PM",\
+            .property = "acpi-pci-hotplug-with-bridge-support",\
+            .value    = "off",\
+        }
+
 #define PC_Q35_COMPAT_1_7 \
+        PC_COMPAT_1_7, \
         {\
             .driver   = "hpet",\
             .property = HPET_INTCAP,\
@@ -261,11 +269,6 @@ int e820_add_entry(uint64_t, uint64_t, uint32_t);
 
 #define PC_COMPAT_1_6 \
         {\
-            .driver   = "PIIX4_PM",\
-            .property = "acpi-pci-hotplug-with-bridge-support",\
-            .value    = "off",\
-        }, \
-        {\
             .driver   = "e1000",\
             .property = "mitigation",\
             .value    = "off",\