902636
From 113078b23a4747b07eb363719d7cbc0af403dd2a Mon Sep 17 00:00:00 2001
902636
From: Miroslav Rezanina <mrezanin@redhat.com>
902636
Date: Fri, 11 Jan 2019 09:54:45 +0100
902636
Subject: Machine type related general changes
902636
902636
This patch is first part of original "Add RHEL machine types" patch we
902636
split to allow easier review. It contains changes not related to any
902636
architecture.
902636
902636
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
902636
902636
Rebase changes (4.0.0):
902636
- Remove e1000 device duplication changes to reflect upstream solution
902636
- Rewrite machine compat properties to upstream solution
902636
902636
Rebase changes (4.1.0):
902636
- Removed optional flag for machine compat properties (upstream)
902636
- Remove c3e002cb chunk from hw/net/e1000.c
902636
- Reorder compat structures
902636
- Use one format for compat scructures
902636
- Added compat for virtio-balloon-pci.any_layout for rhel71
902636
902636
Merged patches (4.0.0):
902636
- d4c0957 compat: Generic HW_COMPAT_RHEL7_6
902636
- cbac773 virtio: Make disable-legacy/disable-modern compat properties optional
902636
902636
Merged patches (4.1.0):
902636
- 479ad30 redhat: fix cut'n'paste garbage in hw_compat comments
902636
- f19738e compat: Generic hw_compat_rhel_8_0
902636
902636
Merged patches (4.2.0):
902636
- 9f2bfaa machine types: Update hw_compat_rhel_8_0 from hw_compat_4_0
902636
- ca4a5e8 virtio: Make disable-legacy/disable-modern compat properties optional
902636
- compat: Generic hw_compat_rhel_8_1 (patch 93040/92956)
902636
902636
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
902636
---
902636
 hw/acpi/ich9.c          |  16 ++++
902636
 hw/acpi/piix4.c         |   5 +-
902636
 hw/char/serial.c        |  16 ++++
902636
 hw/core/machine.c       | 170 ++++++++++++++++++++++++++++++++++++++++
902636
 hw/display/vga-isa.c    |   2 +-
902636
 hw/net/e1000e.c         |  21 +++++
902636
 hw/net/rtl8139.c        |   4 +-
902636
 hw/rtc/mc146818rtc.c    |   6 ++
902636
 hw/smbios/smbios.c      |   1 +
902636
 hw/timer/i8254_common.c |   2 +-
902636
 hw/usb/hcd-uhci.c       |   4 +-
902636
 hw/usb/hcd-xhci.c       |  20 +++++
902636
 hw/usb/hcd-xhci.h       |   2 +
902636
 include/hw/acpi/ich9.h  |   3 +
902636
 include/hw/boards.h     |  24 ++++++
902636
 include/hw/usb.h        |   4 +
902636
 migration/migration.c   |   2 +
902636
 migration/migration.h   |   5 ++
902636
 18 files changed, 301 insertions(+), 6 deletions(-)
902636
902636
diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c
902636
index ab203ad448..7ec26884e8 100644
902636
--- a/hw/acpi/ich9.c
902636
+++ b/hw/acpi/ich9.c
902636
@@ -444,6 +444,18 @@ static void ich9_pm_set_enable_tco(Object *obj, bool value, Error **errp)
902636
     s->pm.enable_tco = value;
902636
 }
902636
 
902636
+static bool ich9_pm_get_force_rev1_fadt(Object *obj, Error **errp)
902636
+{
902636
+    ICH9LPCState *s = ICH9_LPC_DEVICE(obj);
902636
+    return s->pm.force_rev1_fadt;
902636
+}
902636
+
902636
+static void ich9_pm_set_force_rev1_fadt(Object *obj, bool value, Error **errp)
902636
+{
902636
+    ICH9LPCState *s = ICH9_LPC_DEVICE(obj);
902636
+    s->pm.force_rev1_fadt = value;
902636
+}
902636
+
902636
 void ich9_pm_add_properties(Object *obj, ICH9LPCPMRegs *pm, Error **errp)
902636
 {
902636
     static const uint32_t gpe0_len = ICH9_PMIO_GPE0_LEN;
902636
@@ -468,6 +480,10 @@ void ich9_pm_add_properties(Object *obj, ICH9LPCPMRegs *pm, Error **errp)
902636
                              ich9_pm_get_cpu_hotplug_legacy,
902636
                              ich9_pm_set_cpu_hotplug_legacy,
902636
                              NULL);
902636
+    object_property_add_bool(obj, "__com.redhat_force-rev1-fadt",
902636
+                             ich9_pm_get_force_rev1_fadt,
902636
+                             ich9_pm_set_force_rev1_fadt,
902636
+                             NULL);
902636
     object_property_add(obj, ACPI_PM_PROP_S3_DISABLED, "uint8",
902636
                         ich9_pm_get_disable_s3,
902636
                         ich9_pm_set_disable_s3,
902636
diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
902636
index 93aec2dd2c..3a26193cbe 100644
902636
--- a/hw/acpi/piix4.c
902636
+++ b/hw/acpi/piix4.c
902636
@@ -274,6 +274,7 @@ static const VMStateDescription vmstate_acpi = {
902636
     .name = "piix4_pm",
902636
     .version_id = 3,
902636
     .minimum_version_id = 3,
902636
+    .minimum_version_id = 2,
902636
     .post_load = vmstate_acpi_post_load,
902636
     .fields = (VMStateField[]) {
902636
         VMSTATE_PCI_DEVICE(parent_obj, PIIX4PMState),
902636
@@ -627,8 +628,8 @@ static void piix4_send_gpe(AcpiDeviceIf *adev, AcpiEventStatusBits ev)
902636
 
902636
 static Property piix4_pm_properties[] = {
902636
     DEFINE_PROP_UINT32("smb_io_base", PIIX4PMState, smb_io_base, 0),
902636
-    DEFINE_PROP_UINT8(ACPI_PM_PROP_S3_DISABLED, PIIX4PMState, disable_s3, 0),
902636
-    DEFINE_PROP_UINT8(ACPI_PM_PROP_S4_DISABLED, PIIX4PMState, disable_s4, 0),
902636
+    DEFINE_PROP_UINT8(ACPI_PM_PROP_S3_DISABLED, PIIX4PMState, disable_s3, 1),
902636
+    DEFINE_PROP_UINT8(ACPI_PM_PROP_S4_DISABLED, PIIX4PMState, disable_s4, 1),
902636
     DEFINE_PROP_UINT8(ACPI_PM_PROP_S4_VAL, PIIX4PMState, s4_val, 2),
902636
     DEFINE_PROP_BOOL("acpi-pci-hotplug-with-bridge-support", PIIX4PMState,
902636
                      use_acpi_pci_hotplug, true),
902636
diff --git a/hw/char/serial.c b/hw/char/serial.c
902636
index b4aa250950..0012f0e44d 100644
902636
--- a/hw/char/serial.c
902636
+++ b/hw/char/serial.c
902636
@@ -34,6 +34,7 @@
902636
 #include "sysemu/runstate.h"
902636
 #include "qemu/error-report.h"
902636
 #include "trace.h"
902636
+#include "migration/migration.h"
902636
 
902636
 //#define DEBUG_SERIAL
902636
 
902636
@@ -703,6 +704,9 @@ static int serial_post_load(void *opaque, int version_id)
902636
 static bool serial_thr_ipending_needed(void *opaque)
902636
 {
902636
     SerialState *s = opaque;
902636
+    if (migrate_pre_2_2) {
902636
+        return false;
902636
+    }
902636
 
902636
     if (s->ier & UART_IER_THRI) {
902636
         bool expected_value = ((s->iir & UART_IIR_ID) == UART_IIR_THRI);
902636
@@ -784,6 +788,10 @@ static const VMStateDescription vmstate_serial_xmit_fifo = {
902636
 static bool serial_fifo_timeout_timer_needed(void *opaque)
902636
 {
902636
     SerialState *s = (SerialState *)opaque;
902636
+    if (migrate_pre_2_2) {
902636
+        return false;
902636
+    }
902636
+
902636
     return timer_pending(s->fifo_timeout_timer);
902636
 }
902636
 
902636
@@ -801,6 +809,10 @@ static const VMStateDescription vmstate_serial_fifo_timeout_timer = {
902636
 static bool serial_timeout_ipending_needed(void *opaque)
902636
 {
902636
     SerialState *s = (SerialState *)opaque;
902636
+    if (migrate_pre_2_2) {
902636
+        return false;
902636
+    }
902636
+
902636
     return s->timeout_ipending != 0;
902636
 }
902636
 
902636
@@ -818,6 +830,10 @@ static const VMStateDescription vmstate_serial_timeout_ipending = {
902636
 static bool serial_poll_needed(void *opaque)
902636
 {
902636
     SerialState *s = (SerialState *)opaque;
902636
+    if (migrate_pre_2_2) {
902636
+        return false;
902636
+    }
902636
+
902636
     return s->poll_msl >= 0;
902636
 }
902636
 
902636
diff --git a/hw/core/machine.c b/hw/core/machine.c
902636
index 1689ad3bf8..e0e0eec8bf 100644
902636
--- a/hw/core/machine.c
902636
+++ b/hw/core/machine.c
902636
@@ -27,6 +27,176 @@
902636
 #include "hw/pci/pci.h"
902636
 #include "hw/mem/nvdimm.h"
902636
 
902636
+/*
902636
+ * The same as hw_compat_4_1
902636
+ */
902636
+GlobalProperty hw_compat_rhel_8_1[] = {
902636
+    /* hw_compat_rhel_8_1 from hw_compat_4_1 */
902636
+    { "virtio-pci", "x-pcie-flr-init", "off" },
902636
+};
902636
+const size_t hw_compat_rhel_8_1_len = G_N_ELEMENTS(hw_compat_rhel_8_1);
902636
+
902636
+/* The same as hw_compat_3_1
902636
+ * format of array has been changed by:
902636
+ *     6c36bddf5340 ("machine: Use shorter format for GlobalProperty arrays")
902636
+ */
902636
+GlobalProperty hw_compat_rhel_8_0[] = {
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "pcie-root-port", "x-speed", "2_5" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "pcie-root-port", "x-width", "1" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "memory-backend-file", "x-use-canonical-path-for-ramblock-id", "true" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "memory-backend-memfd", "x-use-canonical-path-for-ramblock-id", "true" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "tpm-crb", "ppi", "false" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "tpm-tis", "ppi", "false" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "usb-kbd", "serial", "42" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "usb-mouse", "serial", "42" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "usb-tablet", "serial", "42" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "virtio-blk-device", "discard", "false" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 */
902636
+    { "virtio-blk-device", "write-zeroes", "false" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_4_0 */
902636
+    { "VGA",            "edid", "false" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_4_0 */
902636
+    { "secondary-vga",  "edid", "false" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_4_0 */
902636
+    { "bochs-display",  "edid", "false" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_4_0 */
902636
+    { "virtio-vga",     "edid", "false" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_4_0 */
902636
+    { "virtio-gpu-pci", "edid", "false" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_4_0 */
902636
+    { "virtio-device", "use-started", "false" },
902636
+    /* hw_compat_rhel_8_0 from hw_compat_3_1 - that was added in 4.1 */
902636
+    { "pcie-root-port-base", "disable-acs", "true" },
902636
+};
902636
+const size_t hw_compat_rhel_8_0_len = G_N_ELEMENTS(hw_compat_rhel_8_0);
902636
+
902636
+/* The same as hw_compat_3_0 + hw_compat_2_12
902636
+ * except that
902636
+ *   there's nothing in 3_0
902636
+ *   migration.decompress-error-check=off was in 7.5 from bz 1584139
902636
+ */
902636
+GlobalProperty hw_compat_rhel_7_6[] = {
902636
+    /* hw_compat_rhel_7_6 from hw_compat_2_12 */
902636
+    { "hda-audio", "use-timer", "false" },
902636
+    /* hw_compat_rhel_7_6 from hw_compat_2_12 */
902636
+    { "cirrus-vga", "global-vmstate", "true" },
902636
+    /* hw_compat_rhel_7_6 from hw_compat_2_12 */
902636
+    { "VGA", "global-vmstate", "true" },
902636
+    /* hw_compat_rhel_7_6 from hw_compat_2_12 */
902636
+    { "vmware-svga", "global-vmstate", "true" },
902636
+    /* hw_compat_rhel_7_6 from hw_compat_2_12 */
902636
+    { "qxl-vga", "global-vmstate",  "true" },
902636
+};
902636
+const size_t hw_compat_rhel_7_6_len = G_N_ELEMENTS(hw_compat_rhel_7_6);
902636
+
902636
+/* The same as hw_compat_2_11 + hw_compat_2_10 */
902636
+GlobalProperty hw_compat_rhel_7_5[] = {
902636
+    /* hw_compat_rhel_7_5 from hw_compat_2_11 */
902636
+    { "hpet", "hpet-offset-saved", "false" },
902636
+    /* hw_compat_rhel_7_5 from hw_compat_2_11 */
902636
+    { "virtio-blk-pci", "vectors", "2" },
902636
+    /* hw_compat_rhel_7_5 from hw_compat_2_11 */
902636
+    { "vhost-user-blk-pci", "vectors", "2" },
902636
+    /* hw_compat_rhel_7_5 from hw_compat_2_11
902636
+       bz 1608778 modified for our naming */
902636
+    { "e1000-82540em", "migrate_tso_props", "off" },
902636
+    /* hw_compat_rhel_7_5 from hw_compat_2_10 */
902636
+    { "virtio-mouse-device", "wheel-axis", "false" },
902636
+    /* hw_compat_rhel_7_5 from hw_compat_2_10 */
902636
+    { "virtio-tablet-device", "wheel-axis", "false" },
902636
+    { "cirrus-vga", "vgamem_mb", "16" },
902636
+    { "migration", "decompress-error-check", "off" },
902636
+};
902636
+const size_t hw_compat_rhel_7_5_len = G_N_ELEMENTS(hw_compat_rhel_7_5);
902636
+
902636
+/* Mostly like hw_compat_2_9 except
902636
+ * x-mtu-bypass-backend, x-migrate-msix has already been
902636
+ * backported to RHEL7.4. shpc was already on in 7.4.
902636
+ */
902636
+GlobalProperty hw_compat_rhel_7_4[] = {
902636
+    { "intel-iommu", "pt", "off" },
902636
+};
902636
+
902636
+const size_t hw_compat_rhel_7_4_len = G_N_ELEMENTS(hw_compat_rhel_7_4);
902636
+/* Mostly like HW_COMPAT_2_6 + HW_COMPAT_2_7 + HW_COMPAT_2_8 except
902636
+ * disable-modern, disable-legacy, page-per-vq have already been
902636
+ * backported to RHEL7.3
902636
+ */
902636
+GlobalProperty hw_compat_rhel_7_3[] = {
902636
+    { "virtio-mmio", "format_transport_address", "off" },
902636
+    { "virtio-serial-device", "emergency-write", "off" },
902636
+    { "ioapic", "version", "0x11" },
902636
+    { "intel-iommu", "x-buggy-eim", "true" },
902636
+    { "virtio-pci", "x-ignore-backend-features", "on" },
902636
+    { "fw_cfg_mem", "x-file-slots", stringify(0x10) },
902636
+    { "fw_cfg_io", "x-file-slots", stringify(0x10) },
902636
+    { "pflash_cfi01", "old-multiple-chip-handling", "on" },
902636
+    { TYPE_PCI_DEVICE, "x-pcie-extcap-init", "off" },
902636
+    { "virtio-pci", "x-pcie-deverr-init", "off" },
902636
+    { "virtio-pci", "x-pcie-lnkctl-init", "off" },
902636
+    { "virtio-pci", "x-pcie-pm-init", "off" },
902636
+    { "virtio-net-device", "x-mtu-bypass-backend", "off" },
902636
+    { "e1000e", "__redhat_e1000e_7_3_intr_state", "on" },
902636
+};
902636
+const size_t hw_compat_rhel_7_3_len = G_N_ELEMENTS(hw_compat_rhel_7_3);
902636
+
902636
+/* Mostly like hw_compat_2_4 + 2_3 but:
902636
+ *  we don't need "any_layout" as it has been backported to 7.2
902636
+ */
902636
+GlobalProperty hw_compat_rhel_7_2[] = {
902636
+        { "virtio-blk-device", "scsi", "true" },
902636
+        { "e1000-82540em", "extra_mac_registers", "off" },
902636
+        { "virtio-pci", "x-disable-pcie", "on" },
902636
+        { "virtio-pci", "migrate-extra", "off" },
902636
+        { "fw_cfg_mem", "dma_enabled", "off" },
902636
+        { "fw_cfg_io", "dma_enabled", "off" },
902636
+        { "isa-fdc", "fallback", "144" },
902636
+        /* Optional because not all virtio-pci devices support legacy mode */
902636
+        { "virtio-pci", "disable-modern", "on", .optional = true },
902636
+        { "virtio-pci", "disable-legacy", "off", .optional = true },
902636
+        { TYPE_PCI_DEVICE, "x-pcie-lnksta-dllla", "off" },
902636
+        { "virtio-pci", "page-per-vq", "on" },
902636
+        /* hw_compat_rhel_7_2 - introduced with 2.10.0 */
902636
+        { "migration", "send-section-footer", "off" },
902636
+        /* hw_compat_rhel_7_2 - introduced with 2.10.0 */
902636
+        { "migration", "store-global-state", "off",
902636
+        },
902636
+};
902636
+const size_t hw_compat_rhel_7_2_len = G_N_ELEMENTS(hw_compat_rhel_7_2);
902636
+
902636
+/* Mostly like hw_compat_2_1 but:
902636
+ *    we don't need virtio-scsi-pci since 7.0 already had that on
902636
+ *
902636
+ * RH: Note, qemu-extended-regs should have been enabled in the 7.1
902636
+ * machine type, but was accidentally turned off in 7.2 onwards.
902636
+ */
902636
+GlobalProperty hw_compat_rhel_7_1[] = {
902636
+    { "intel-hda-generic", "old_msi_addr", "on" },
902636
+    { "VGA", "qemu-extended-regs", "off" },
902636
+    { "secondary-vga", "qemu-extended-regs", "off" },
902636
+    { "usb-mouse", "usb_version", stringify(1) },
902636
+    { "usb-kbd", "usb_version", stringify(1) },
902636
+    { "virtio-pci",  "virtio-pci-bus-master-bug-migration", "on" },
902636
+    { "virtio-blk-pci", "any_layout", "off" },
902636
+    { "virtio-balloon-pci", "any_layout", "off" },
902636
+    { "virtio-serial-pci", "any_layout", "off" },
902636
+    { "virtio-9p-pci", "any_layout", "off" },
902636
+    { "virtio-rng-pci", "any_layout", "off" },
902636
+    /* HW_COMPAT_RHEL7_1 - introduced with 2.10.0 */
902636
+    { "migration", "send-configuration", "off" },
902636
+};
902636
+const size_t hw_compat_rhel_7_1_len = G_N_ELEMENTS(hw_compat_rhel_7_1);
902636
+
902636
 GlobalProperty hw_compat_4_1[] = {
902636
     { "virtio-pci", "x-pcie-flr-init", "off" },
902636
 };
902636
diff --git a/hw/display/vga-isa.c b/hw/display/vga-isa.c
902636
index 873e5e9706..d1a2efe47e 100644
902636
--- a/hw/display/vga-isa.c
902636
+++ b/hw/display/vga-isa.c
902636
@@ -82,7 +82,7 @@ static void vga_isa_realizefn(DeviceState *dev, Error **errp)
902636
 }
902636
 
902636
 static Property vga_isa_properties[] = {
902636
-    DEFINE_PROP_UINT32("vgamem_mb", ISAVGAState, state.vram_size_mb, 8),
902636
+    DEFINE_PROP_UINT32("vgamem_mb", ISAVGAState, state.vram_size_mb, 16),
902636
     DEFINE_PROP_END_OF_LIST(),
902636
 };
902636
 
902636
diff --git a/hw/net/e1000e.c b/hw/net/e1000e.c
902636
index b69fd7d8ad..d8be50a1ce 100644
902636
--- a/hw/net/e1000e.c
902636
+++ b/hw/net/e1000e.c
902636
@@ -79,6 +79,11 @@ typedef struct E1000EState {
902636
 
902636
     E1000ECore core;
902636
 
902636
+    /* 7.3 had the intr_state field that was in the original e1000e code
902636
+     * but that was removed prior to 2.7's release
902636
+     */
902636
+    bool redhat_7_3_intr_state_enable;
902636
+    uint32_t redhat_7_3_intr_state;
902636
 } E1000EState;
902636
 
902636
 #define E1000E_MMIO_IDX     0
902636
@@ -94,6 +99,10 @@ typedef struct E1000EState {
902636
 #define E1000E_MSIX_TABLE   (0x0000)
902636
 #define E1000E_MSIX_PBA     (0x2000)
902636
 
902636
+/* Values as in RHEL 7.3 build and original upstream */
902636
+#define RH_E1000E_USE_MSI     BIT(0)
902636
+#define RH_E1000E_USE_MSIX    BIT(1)
902636
+
902636
 static uint64_t
902636
 e1000e_mmio_read(void *opaque, hwaddr addr, unsigned size)
902636
 {
902636
@@ -305,6 +314,8 @@ e1000e_init_msix(E1000EState *s)
902636
     } else {
902636
         if (!e1000e_use_msix_vectors(s, E1000E_MSIX_VEC_NUM)) {
902636
             msix_uninit(d, &s->msix, &s->msix);
902636
+        } else {
902636
+            s->redhat_7_3_intr_state |= RH_E1000E_USE_MSIX;
902636
         }
902636
     }
902636
 }
902636
@@ -476,6 +487,8 @@ static void e1000e_pci_realize(PCIDevice *pci_dev, Error **errp)
902636
     ret = msi_init(PCI_DEVICE(s), 0xD0, 1, true, false, NULL);
902636
     if (ret) {
902636
         trace_e1000e_msi_init_fail(ret);
902636
+    } else {
902636
+        s->redhat_7_3_intr_state |= RH_E1000E_USE_MSI;
902636
     }
902636
 
902636
     if (e1000e_add_pm_capability(pci_dev, e1000e_pmrb_offset,
902636
@@ -599,6 +612,11 @@ static const VMStateDescription e1000e_vmstate_intr_timer = {
902636
     VMSTATE_STRUCT_ARRAY(_f, _s, _num, 0,                           \
902636
                          e1000e_vmstate_intr_timer, E1000IntrDelayTimer)
902636
 
902636
+static bool rhel_7_3_check(void *opaque, int version_id)
902636
+{
902636
+    return ((E1000EState *)opaque)->redhat_7_3_intr_state_enable;
902636
+}
902636
+
902636
 static const VMStateDescription e1000e_vmstate = {
902636
     .name = "e1000e",
902636
     .version_id = 1,
902636
@@ -610,6 +628,7 @@ static const VMStateDescription e1000e_vmstate = {
902636
         VMSTATE_MSIX(parent_obj, E1000EState),
902636
 
902636
         VMSTATE_UINT32(ioaddr, E1000EState),
902636
+        VMSTATE_UINT32_TEST(redhat_7_3_intr_state, E1000EState, rhel_7_3_check),
902636
         VMSTATE_UINT32(core.rxbuf_min_shift, E1000EState),
902636
         VMSTATE_UINT8(core.rx_desc_len, E1000EState),
902636
         VMSTATE_UINT32_ARRAY(core.rxbuf_sizes, E1000EState,
902636
@@ -658,6 +677,8 @@ static PropertyInfo e1000e_prop_disable_vnet,
902636
 
902636
 static Property e1000e_properties[] = {
902636
     DEFINE_NIC_PROPERTIES(E1000EState, conf),
902636
+    DEFINE_PROP_BOOL("__redhat_e1000e_7_3_intr_state", E1000EState,
902636
+                        redhat_7_3_intr_state_enable, false),
902636
     DEFINE_PROP_SIGNED("disable_vnet_hdr", E1000EState, disable_vnet, false,
902636
                         e1000e_prop_disable_vnet, bool),
902636
     DEFINE_PROP_SIGNED("subsys_ven", E1000EState, subsys_ven,
902636
diff --git a/hw/net/rtl8139.c b/hw/net/rtl8139.c
902636
index 88a97d756d..21d80e96cf 100644
902636
--- a/hw/net/rtl8139.c
902636
+++ b/hw/net/rtl8139.c
902636
@@ -3177,7 +3177,7 @@ static int rtl8139_pre_save(void *opaque)
902636
 
902636
 static const VMStateDescription vmstate_rtl8139 = {
902636
     .name = "rtl8139",
902636
-    .version_id = 5,
902636
+    .version_id = 4,
902636
     .minimum_version_id = 3,
902636
     .post_load = rtl8139_post_load,
902636
     .pre_save  = rtl8139_pre_save,
902636
@@ -3258,7 +3258,9 @@ static const VMStateDescription vmstate_rtl8139 = {
902636
         VMSTATE_UINT32(tally_counters.TxMCol, RTL8139State),
902636
         VMSTATE_UINT64(tally_counters.RxOkPhy, RTL8139State),
902636
         VMSTATE_UINT64(tally_counters.RxOkBrd, RTL8139State),
902636
+#if 0 /* Disabled for Red Hat Enterprise Linux bz 1420195 */
902636
         VMSTATE_UINT32_V(tally_counters.RxOkMul, RTL8139State, 5),
902636
+#endif
902636
         VMSTATE_UINT16(tally_counters.TxAbt, RTL8139State),
902636
         VMSTATE_UINT16(tally_counters.TxUndrn, RTL8139State),
902636
 
902636
diff --git a/hw/rtc/mc146818rtc.c b/hw/rtc/mc146818rtc.c
902636
index 74ae74bc5c..73820517df 100644
902636
--- a/hw/rtc/mc146818rtc.c
902636
+++ b/hw/rtc/mc146818rtc.c
902636
@@ -42,6 +42,7 @@
902636
 #include "qapi/visitor.h"
902636
 #include "exec/address-spaces.h"
902636
 #include "hw/rtc/mc146818rtc_regs.h"
902636
+#include "migration/migration.h"
902636
 
902636
 #ifdef TARGET_I386
902636
 #include "qapi/qapi-commands-misc-target.h"
902636
@@ -820,6 +821,11 @@ static int rtc_post_load(void *opaque, int version_id)
902636
 static bool rtc_irq_reinject_on_ack_count_needed(void *opaque)
902636
 {
902636
     RTCState *s = (RTCState *)opaque;
902636
+
902636
+    if (migrate_pre_2_2) {
902636
+        return false;
902636
+    }
902636
+
902636
     return s->irq_reinject_on_ack_count != 0;
902636
 }
902636
 
902636
diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c
902636
index 11d476c4a2..e6e9355384 100644
902636
--- a/hw/smbios/smbios.c
902636
+++ b/hw/smbios/smbios.c
902636
@@ -777,6 +777,7 @@ void smbios_set_defaults(const char *manufacturer, const char *product,
902636
     SMBIOS_SET_DEFAULT(type1.manufacturer, manufacturer);
902636
     SMBIOS_SET_DEFAULT(type1.product, product);
902636
     SMBIOS_SET_DEFAULT(type1.version, version);
902636
+    SMBIOS_SET_DEFAULT(type1.family, "Red Hat Enterprise Linux");
902636
     SMBIOS_SET_DEFAULT(type2.manufacturer, manufacturer);
902636
     SMBIOS_SET_DEFAULT(type2.product, product);
902636
     SMBIOS_SET_DEFAULT(type2.version, version);
902636
diff --git a/hw/timer/i8254_common.c b/hw/timer/i8254_common.c
902636
index 050875b497..32935da46c 100644
902636
--- a/hw/timer/i8254_common.c
902636
+++ b/hw/timer/i8254_common.c
902636
@@ -231,7 +231,7 @@ static const VMStateDescription vmstate_pit_common = {
902636
     .pre_save = pit_dispatch_pre_save,
902636
     .post_load = pit_dispatch_post_load,
902636
     .fields = (VMStateField[]) {
902636
-        VMSTATE_UINT32_V(channels[0].irq_disabled, PITCommonState, 3),
902636
+        VMSTATE_UINT32(channels[0].irq_disabled, PITCommonState), /* qemu-kvm's v2 had 'flags' here */
902636
         VMSTATE_STRUCT_ARRAY(channels, PITCommonState, 3, 2,
902636
                              vmstate_pit_channel, PITChannelState),
902636
         VMSTATE_INT64(channels[0].next_transition_time,
902636
diff --git a/hw/usb/hcd-uhci.c b/hw/usb/hcd-uhci.c
902636
index 23507ad3b5..9fd87a7ad9 100644
902636
--- a/hw/usb/hcd-uhci.c
902636
+++ b/hw/usb/hcd-uhci.c
902636
@@ -1219,12 +1219,14 @@ static void usb_uhci_common_realize(PCIDevice *dev, Error **errp)
902636
     UHCIState *s = UHCI(dev);
902636
     uint8_t *pci_conf = s->dev.config;
902636
     int i;
902636
+    int irq_pin;
902636
 
902636
     pci_conf[PCI_CLASS_PROG] = 0x00;
902636
     /* TODO: reset value should be 0. */
902636
     pci_conf[USB_SBRN] = USB_RELEASE_1; // release number
902636
 
902636
-    pci_config_set_interrupt_pin(pci_conf, u->info.irq_pin + 1);
902636
+    irq_pin = u->info.irq_pin;
902636
+    pci_config_set_interrupt_pin(pci_conf, irq_pin + 1);
902636
 
902636
     if (s->masterbus) {
902636
         USBPort *ports[NB_PORTS];
902636
diff --git a/hw/usb/hcd-xhci.c b/hw/usb/hcd-xhci.c
902636
index 80988bb305..8fed2eedd6 100644
902636
--- a/hw/usb/hcd-xhci.c
902636
+++ b/hw/usb/hcd-xhci.c
902636
@@ -3590,9 +3590,27 @@ static const VMStateDescription vmstate_xhci_slot = {
902636
     }
902636
 };
902636
 
902636
+static int xhci_event_pre_save(void *opaque)
902636
+{
902636
+    XHCIEvent *s = opaque;
902636
+
902636
+    s->cve_2014_5263_a = ((uint8_t *)&s->type)[0];
902636
+    s->cve_2014_5263_b = ((uint8_t *)&s->type)[1];
902636
+
902636
+    return 0;
902636
+}
902636
+
902636
+bool migrate_cve_2014_5263_xhci_fields;
902636
+
902636
+static bool xhci_event_cve_2014_5263(void *opaque, int version_id)
902636
+{
902636
+    return migrate_cve_2014_5263_xhci_fields;
902636
+}
902636
+
902636
 static const VMStateDescription vmstate_xhci_event = {
902636
     .name = "xhci-event",
902636
     .version_id = 1,
902636
+    .pre_save = xhci_event_pre_save,
902636
     .fields = (VMStateField[]) {
902636
         VMSTATE_UINT32(type,   XHCIEvent),
902636
         VMSTATE_UINT32(ccode,  XHCIEvent),
902636
@@ -3601,6 +3619,8 @@ static const VMStateDescription vmstate_xhci_event = {
902636
         VMSTATE_UINT32(flags,  XHCIEvent),
902636
         VMSTATE_UINT8(slotid,  XHCIEvent),
902636
         VMSTATE_UINT8(epid,    XHCIEvent),
902636
+        VMSTATE_UINT8_TEST(cve_2014_5263_a, XHCIEvent, xhci_event_cve_2014_5263),
902636
+        VMSTATE_UINT8_TEST(cve_2014_5263_b, XHCIEvent, xhci_event_cve_2014_5263),
902636
         VMSTATE_END_OF_LIST()
902636
     }
902636
 };
902636
diff --git a/hw/usb/hcd-xhci.h b/hw/usb/hcd-xhci.h
902636
index 2fad4df2a7..f554b671e3 100644
902636
--- a/hw/usb/hcd-xhci.h
902636
+++ b/hw/usb/hcd-xhci.h
902636
@@ -157,6 +157,8 @@ typedef struct XHCIEvent {
902636
     uint32_t flags;
902636
     uint8_t slotid;
902636
     uint8_t epid;
902636
+    uint8_t cve_2014_5263_a;
902636
+    uint8_t cve_2014_5263_b;
902636
 } XHCIEvent;
902636
 
902636
 typedef struct XHCIInterrupter {
902636
diff --git a/include/hw/acpi/ich9.h b/include/hw/acpi/ich9.h
902636
index 41568d1837..1a23ccc412 100644
902636
--- a/include/hw/acpi/ich9.h
902636
+++ b/include/hw/acpi/ich9.h
902636
@@ -61,6 +61,9 @@ typedef struct ICH9LPCPMRegs {
902636
     uint8_t smm_enabled;
902636
     bool enable_tco;
902636
     TCOIORegs tco_regs;
902636
+
902636
+    /* RH addition, see bz 1489800 */
902636
+    bool force_rev1_fadt;
902636
 } ICH9LPCPMRegs;
902636
 
902636
 #define ACPI_PM_PROP_TCO_ENABLED "enable_tco"
902636
diff --git a/include/hw/boards.h b/include/hw/boards.h
902636
index de45087f34..6f85a0e032 100644
902636
--- a/include/hw/boards.h
902636
+++ b/include/hw/boards.h
902636
@@ -377,4 +377,28 @@ extern const size_t hw_compat_2_2_len;
902636
 extern GlobalProperty hw_compat_2_1[];
902636
 extern const size_t hw_compat_2_1_len;
902636
 
902636
+extern GlobalProperty hw_compat_rhel_8_1[];
902636
+extern const size_t hw_compat_rhel_8_1_len;
902636
+
902636
+extern GlobalProperty hw_compat_rhel_8_0[];
902636
+extern const size_t hw_compat_rhel_8_0_len;
902636
+
902636
+extern GlobalProperty hw_compat_rhel_7_6[];
902636
+extern const size_t hw_compat_rhel_7_6_len;
902636
+
902636
+extern GlobalProperty hw_compat_rhel_7_5[];
902636
+extern const size_t hw_compat_rhel_7_5_len;
902636
+
902636
+extern GlobalProperty hw_compat_rhel_7_4[];
902636
+extern const size_t hw_compat_rhel_7_4_len;
902636
+
902636
+extern GlobalProperty hw_compat_rhel_7_3[];
902636
+extern const size_t hw_compat_rhel_7_3_len;
902636
+
902636
+extern GlobalProperty hw_compat_rhel_7_2[];
902636
+extern const size_t hw_compat_rhel_7_2_len;
902636
+
902636
+extern GlobalProperty hw_compat_rhel_7_1[];
902636
+extern const size_t hw_compat_rhel_7_1_len;
902636
+
902636
 #endif
902636
diff --git a/include/hw/usb.h b/include/hw/usb.h
902636
index c24d968a19..b353438ea0 100644
902636
--- a/include/hw/usb.h
902636
+++ b/include/hw/usb.h
902636
@@ -605,4 +605,8 @@ int usb_get_quirks(uint16_t vendor_id, uint16_t product_id,
902636
                    uint8_t interface_class, uint8_t interface_subclass,
902636
                    uint8_t interface_protocol);
902636
 
902636
+
902636
+/* hcd-xhci.c -- rhel7.0.0 machine type compatibility */
902636
+extern bool migrate_cve_2014_5263_xhci_fields;
902636
+
902636
 #endif
902636
diff --git a/migration/migration.c b/migration/migration.c
902636
index 354ad072fa..30c53c623b 100644
902636
--- a/migration/migration.c
902636
+++ b/migration/migration.c
902636
@@ -121,6 +121,8 @@ enum mig_rp_message_type {
902636
     MIG_RP_MSG_MAX
902636
 };
902636
 
902636
+bool migrate_pre_2_2;
902636
+
902636
 /* When we add fault tolerance, we could have several
902636
    migrations at once.  For now we don't need to add
902636
    dynamic creation of migration */
902636
diff --git a/migration/migration.h b/migration/migration.h
902636
index 79b3dda146..0b1b0d4df5 100644
902636
--- a/migration/migration.h
902636
+++ b/migration/migration.h
902636
@@ -335,6 +335,11 @@ void init_dirty_bitmap_incoming_migration(void);
902636
 void migrate_add_address(SocketAddress *address);
902636
 
902636
 int foreach_not_ignored_block(RAMBlockIterFunc func, void *opaque);
902636
+/*
902636
+ * Disables a load of subsections that were added in 2.2/rh7.2 for backwards
902636
+ * migration compatibility.
902636
+ */
902636
+extern bool migrate_pre_2_2;
902636
 
902636
 #define qemu_ram_foreach_block \
902636
   #warning "Use foreach_not_ignored_block in migration code"
902636
-- 
902636
2.21.0
902636