22c213
From 136eae41007e2e5b0d693cc656f3ec36cbabf16f Mon Sep 17 00:00:00 2001
22c213
From: Miroslav Rezanina <mrezanin@redhat.com>
22c213
Date: Fri, 19 Oct 2018 13:27:13 +0200
22c213
Subject: Add ppc64 machine types
22c213
22c213
Adding changes to add RHEL machine types for ppc64 architecture.
22c213
22c213
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
22c213
22c213
Rebase changes (4.0.0):
22c213
- remove instance options and use upstream solution
22c213
- Use upstream compat handling
22c213
- Replace SPAPR_PCI_2_7_MMIO_WIN_SIZE with value (changed upstream)
22c213
- re-add handling of instance_options (removed upstream)
22c213
- Use p8 as default for rhel machine types (p9 default upstream)
22c213
- sPAPRMachineClass renamed to SpaprMachineClass (upstream)
22c213
22c213
Rebase changes (4.1.0):
22c213
- Update format for compat structures
22c213
22c213
Merged patches (4.0.0):
22c213
- 467d59a redhat: define pseries-rhel8.0.0 machine type
22c213
22c213
Merged patches (4.1.0):
22c213
- f21757edc target/ppc/spapr: Enable mitigations by default for pseries-4.0 machine type
22c213
- 2511c63 redhat: sync pseries-rhel7.6.0 with rhel-av-8.0.1
22c213
- 89f01da redhat: define pseries-rhel8.1.0 machine type
22c213
22c213
Merged patches (4.2.0):
22c213
- bcba728 redhat: update pseries-rhel8.1.0 machine type
22c213
- redhat: update pseries-rhel-7.6.0 machine type (patch 93039)
22c213
- redhat: define pseries-rhel8.2.0 machine type (patch 93041)
22c213
22c213
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
22c213
---
22c213
 hw/ppc/spapr.c          | 278 ++++++++++++++++++++++++++++++++++++++++
22c213
 hw/ppc/spapr_cpu_core.c |  13 ++
22c213
 include/hw/ppc/spapr.h  |   1 +
22c213
 target/ppc/compat.c     |  13 +-
22c213
 target/ppc/cpu.h        |   1 +
22c213
 5 files changed, 305 insertions(+), 1 deletion(-)
22c213
22c213
diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
22c213
index e076f6023c..8749c72066 100644
22c213
--- a/hw/ppc/spapr.c
22c213
+++ b/hw/ppc/spapr.c
22c213
@@ -4447,6 +4447,7 @@ static void spapr_machine_class_init(ObjectClass *oc, void *data)
22c213
     smc->linux_pci_probe = true;
22c213
     smc->smp_threads_vsmt = true;
22c213
     smc->nr_xirqs = SPAPR_NR_XIRQS;
22c213
+    smc->has_power9_support = true;
22c213
 }
22c213
 
22c213
 static const TypeInfo spapr_machine_info = {
22c213
@@ -4491,6 +4492,7 @@ static const TypeInfo spapr_machine_info = {
22c213
     }                                                                \
22c213
     type_init(spapr_machine_register_##suffix)
22c213
 
22c213
+#if 0 /* Disabled for Red Hat Enterprise Linux */
22c213
 /*
22c213
  * pseries-4.2
22c213
  */
22c213
@@ -4520,6 +4522,7 @@ static void spapr_machine_4_1_class_options(MachineClass *mc)
22c213
 }
22c213
 
22c213
 DEFINE_SPAPR_MACHINE(4_1, "4.1", false);
22c213
+#endif
22c213
 
22c213
 /*
22c213
  * pseries-4.0
22c213
@@ -4536,6 +4539,7 @@ static void phb_placement_4_0(SpaprMachineState *spapr, uint32_t index,
22c213
     *nv2atsd = 0;
22c213
 }
22c213
 
22c213
+#if 0 /* Disabled for Red Hat Enterprise Linux */
22c213
 static void spapr_machine_4_0_class_options(MachineClass *mc)
22c213
 {
22c213
     SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
@@ -4695,6 +4699,7 @@ DEFINE_SPAPR_MACHINE(2_8, "2.8", false);
22c213
 /*
22c213
  * pseries-2.7
22c213
  */
22c213
+#endif
22c213
 
22c213
 static void phb_placement_2_7(SpaprMachineState *spapr, uint32_t index,
22c213
                               uint64_t *buid, hwaddr *pio,
22c213
@@ -4749,6 +4754,7 @@ static void phb_placement_2_7(SpaprMachineState *spapr, uint32_t index,
22c213
     *nv2atsd = 0;
22c213
 }
22c213
 
22c213
+#if 0 /* Disabled for Red Hat Enterprise Linux */
22c213
 static void spapr_machine_2_7_class_options(MachineClass *mc)
22c213
 {
22c213
     SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
@@ -4863,6 +4869,278 @@ static void spapr_machine_2_1_class_options(MachineClass *mc)
22c213
     compat_props_add(mc->compat_props, hw_compat_2_1, hw_compat_2_1_len);
22c213
 }
22c213
 DEFINE_SPAPR_MACHINE(2_1, "2.1", false);
22c213
+#endif
22c213
+
22c213
+/*
22c213
+ * pseries-rhel8.2.0
22c213
+ */
22c213
+
22c213
+static void spapr_machine_rhel820_class_options(MachineClass *mc)
22c213
+{
22c213
+    /* Defaults for the latest behaviour inherited from the base class */
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel820, "rhel8.2.0", true);
22c213
+
22c213
+/*
22c213
+ * pseries-rhel8.1.0
22c213
+ * like pseries-4.1
22c213
+ */
22c213
+
22c213
+static void spapr_machine_rhel810_class_options(MachineClass *mc)
22c213
+{
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
+    static GlobalProperty compat[] = {
22c213
+        /* Only allow 4kiB and 64kiB IOMMU pagesizes */
22c213
+        { TYPE_SPAPR_PCI_HOST_BRIDGE, "pgsz", "0x11000" },
22c213
+    };
22c213
+
22c213
+    spapr_machine_rhel820_class_options(mc);
22c213
+
22c213
+    /* from pseries-4.1 */
22c213
+    smc->linux_pci_probe = false;
22c213
+    smc->smp_threads_vsmt = false;
22c213
+    compat_props_add(mc->compat_props, hw_compat_rhel_8_1,
22c213
+                     hw_compat_rhel_8_1_len);
22c213
+    compat_props_add(mc->compat_props, compat, G_N_ELEMENTS(compat));
22c213
+
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel810, "rhel8.1.0", false);
22c213
+
22c213
+/*
22c213
+ * pseries-rhel8.0.0
22c213
+ * like pseries-3.1 and pseries-4.0
22c213
+ * except SPAPR_CAP_CFPC, SPAPR_CAP_SBBC and SPAPR_CAP_IBS
22c213
+ * that have been backported to pseries-rhel8.0.0
22c213
+ */
22c213
+
22c213
+static void spapr_machine_rhel800_class_options(MachineClass *mc)
22c213
+{
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
+
22c213
+    spapr_machine_rhel810_class_options(mc);
22c213
+    compat_props_add(mc->compat_props, hw_compat_rhel_8_0,
22c213
+                     hw_compat_rhel_8_0_len);
22c213
+
22c213
+    /* pseries-4.0 */
22c213
+    smc->phb_placement = phb_placement_4_0;
22c213
+    smc->irq = &spapr_irq_xics;
22c213
+    smc->pre_4_1_migration = true;
22c213
+
22c213
+    /* pseries-3.1 */
22c213
+    mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8_v2.0");
22c213
+    smc->update_dt_enabled = false;
22c213
+    smc->dr_phb_enabled = false;
22c213
+    smc->broken_host_serial_model = true;
22c213
+    smc->default_caps.caps[SPAPR_CAP_LARGE_DECREMENTER] = SPAPR_CAP_OFF;
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel800, "rhel8.0.0", false);
22c213
+
22c213
+/*
22c213
+ * pseries-rhel7.6.0
22c213
+ * like spapr_compat_2_12 and spapr_compat_3_0
22c213
+ * spapr_compat_0 is empty
22c213
+ */
22c213
+GlobalProperty spapr_compat_rhel7_6[] = {
22c213
+    { TYPE_POWERPC_CPU, "pre-3.0-migration", "on" },
22c213
+    { TYPE_SPAPR_CPU_CORE, "pre-3.0-migration", "on" },
22c213
+};
22c213
+const size_t spapr_compat_rhel7_6_len = G_N_ELEMENTS(spapr_compat_rhel7_6);
22c213
+
22c213
+
22c213
+static void spapr_machine_rhel760_class_options(MachineClass *mc)
22c213
+{
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
+
22c213
+    spapr_machine_rhel800_class_options(mc);
22c213
+    compat_props_add(mc->compat_props, hw_compat_rhel_7_6, hw_compat_rhel_7_6_len);
22c213
+    compat_props_add(mc->compat_props, spapr_compat_rhel7_6, spapr_compat_rhel7_6_len);
22c213
+
22c213
+    /* from spapr_machine_3_0_class_options() */
22c213
+    smc->legacy_irq_allocation = true;
22c213
+    smc->nr_xirqs = 0x400;
22c213
+    smc->irq = &spapr_irq_xics_legacy;
22c213
+
22c213
+    /* from spapr_machine_2_12_class_options() */
22c213
+    /* We depend on kvm_enabled() to choose a default value for the
22c213
+     * hpt-max-page-size capability. Of course we can't do it here
22c213
+     * because this is too early and the HW accelerator isn't initialzed
22c213
+     * yet. Postpone this to machine init (see default_caps_with_cpu()).
22c213
+     */
22c213
+    smc->default_caps.caps[SPAPR_CAP_HPT_MAXPAGESIZE] = 0;
22c213
+
22c213
+    /* SPAPR_CAP_WORKAROUND enabled in pseries-rhel800 by
22c213
+     * f21757edc554
22c213
+     * "Enable mitigations by default for pseries-4.0 machine type")
22c213
+     */
22c213
+    smc->default_caps.caps[SPAPR_CAP_CFPC] = SPAPR_CAP_BROKEN;
22c213
+    smc->default_caps.caps[SPAPR_CAP_SBBC] = SPAPR_CAP_BROKEN;
22c213
+    smc->default_caps.caps[SPAPR_CAP_IBS] = SPAPR_CAP_BROKEN;
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel760, "rhel7.6.0", false);
22c213
+
22c213
+/*
22c213
+ * pseries-rhel7.6.0-sxxm
22c213
+ *
22c213
+ * pseries-rhel7.6.0 with speculative execution exploit mitigations enabled by default
22c213
+ */
22c213
+
22c213
+static void spapr_machine_rhel760sxxm_class_options(MachineClass *mc)
22c213
+{
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
+
22c213
+    spapr_machine_rhel760_class_options(mc);
22c213
+    smc->default_caps.caps[SPAPR_CAP_CFPC] = SPAPR_CAP_WORKAROUND;
22c213
+    smc->default_caps.caps[SPAPR_CAP_SBBC] = SPAPR_CAP_WORKAROUND;
22c213
+    smc->default_caps.caps[SPAPR_CAP_IBS] = SPAPR_CAP_FIXED_CCD;
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel760sxxm, "rhel7.6.0-sxxm", false);
22c213
+
22c213
+static void spapr_machine_rhel750_class_options(MachineClass *mc)
22c213
+{
22c213
+    spapr_machine_rhel760_class_options(mc);
22c213
+    compat_props_add(mc->compat_props, hw_compat_rhel_7_5, hw_compat_rhel_7_5_len);
22c213
+
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel750, "rhel7.5.0", false);
22c213
+
22c213
+/*
22c213
+ * pseries-rhel7.5.0-sxxm
22c213
+ *
22c213
+ * pseries-rhel7.5.0 with speculative execution exploit mitigations enabled by default
22c213
+ */
22c213
+
22c213
+static void spapr_machine_rhel750sxxm_class_options(MachineClass *mc)
22c213
+{
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
+
22c213
+    spapr_machine_rhel750_class_options(mc);
22c213
+    smc->default_caps.caps[SPAPR_CAP_CFPC] = SPAPR_CAP_WORKAROUND;
22c213
+    smc->default_caps.caps[SPAPR_CAP_SBBC] = SPAPR_CAP_WORKAROUND;
22c213
+    smc->default_caps.caps[SPAPR_CAP_IBS] = SPAPR_CAP_FIXED_CCD;
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel750sxxm, "rhel7.5.0-sxxm", false);
22c213
+
22c213
+/*
22c213
+ * pseries-rhel7.4.0
22c213
+ * like spapr_compat_2_9
22c213
+ */
22c213
+GlobalProperty spapr_compat_rhel7_4[] = {
22c213
+    { TYPE_POWERPC_CPU, "pre-2.10-migration", "on" },
22c213
+};
22c213
+const size_t spapr_compat_rhel7_4_len = G_N_ELEMENTS(spapr_compat_rhel7_4);
22c213
+
22c213
+static void spapr_machine_rhel740_class_options(MachineClass *mc)
22c213
+{
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
+
22c213
+    spapr_machine_rhel750_class_options(mc);
22c213
+    compat_props_add(mc->compat_props, hw_compat_rhel_7_4, hw_compat_rhel_7_4_len);
22c213
+    compat_props_add(mc->compat_props, spapr_compat_rhel7_4, spapr_compat_rhel7_4_len);
22c213
+    mc->numa_auto_assign_ram = numa_legacy_auto_assign_ram;
22c213
+    smc->has_power9_support = false;
22c213
+    smc->pre_2_10_has_unused_icps = true;
22c213
+    smc->resize_hpt_default = SPAPR_RESIZE_HPT_DISABLED;
22c213
+    smc->default_caps.caps[SPAPR_CAP_HTM] = SPAPR_CAP_ON;
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel740, "rhel7.4.0", false);
22c213
+
22c213
+/*
22c213
+ * pseries-rhel7.4.0-sxxm
22c213
+ *
22c213
+ * pseries-rhel7.4.0 with speculative execution exploit mitigations enabled by default
22c213
+ */
22c213
+
22c213
+static void spapr_machine_rhel740sxxm_class_options(MachineClass *mc)
22c213
+{
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
+
22c213
+    spapr_machine_rhel740_class_options(mc);
22c213
+    smc->default_caps.caps[SPAPR_CAP_CFPC] = SPAPR_CAP_WORKAROUND;
22c213
+    smc->default_caps.caps[SPAPR_CAP_SBBC] = SPAPR_CAP_WORKAROUND;
22c213
+    smc->default_caps.caps[SPAPR_CAP_IBS] = SPAPR_CAP_FIXED_CCD;
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel740sxxm, "rhel7.4.0-sxxm", false);
22c213
+
22c213
+/*
22c213
+ * pseries-rhel7.3.0
22c213
+ * like spapr_compat_2_6/_2_7/_2_8 but "ddw" has been backported to RHEL7_3
22c213
+ */
22c213
+GlobalProperty spapr_compat_rhel7_3[] = {
22c213
+    { TYPE_SPAPR_PCI_HOST_BRIDGE, "mem_win_size", "0xf80000000" },
22c213
+    { TYPE_SPAPR_PCI_HOST_BRIDGE, "mem64_win_size", "0" },
22c213
+    { TYPE_POWERPC_CPU, "pre-2.8-migration", "on" },
22c213
+    { TYPE_SPAPR_PCI_HOST_BRIDGE, "pre-2.8-migration", "on" },
22c213
+    { TYPE_SPAPR_PCI_HOST_BRIDGE, "pcie-extended-configuration-space", "off" },
22c213
+};
22c213
+const size_t spapr_compat_rhel7_3_len = G_N_ELEMENTS(spapr_compat_rhel7_3);
22c213
+
22c213
+static void spapr_machine_rhel730_class_options(MachineClass *mc)
22c213
+{
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
+
22c213
+    spapr_machine_rhel740_class_options(mc);
22c213
+    mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power7_v2.3");
22c213
+    mc->default_machine_opts = "modern-hotplug-events=off";
22c213
+    compat_props_add(mc->compat_props, hw_compat_rhel_7_3, hw_compat_rhel_7_3_len);
22c213
+    compat_props_add(mc->compat_props, spapr_compat_rhel7_3, spapr_compat_rhel7_3_len);
22c213
+
22c213
+    smc->phb_placement = phb_placement_2_7;
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel730, "rhel7.3.0", false);
22c213
+
22c213
+/*
22c213
+ * pseries-rhel7.3.0-sxxm
22c213
+ *
22c213
+ * pseries-rhel7.3.0 with speculative execution exploit mitigations enabled by default
22c213
+ */
22c213
+
22c213
+static void spapr_machine_rhel730sxxm_class_options(MachineClass *mc)
22c213
+{
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
+
22c213
+    spapr_machine_rhel730_class_options(mc);
22c213
+    smc->default_caps.caps[SPAPR_CAP_CFPC] = SPAPR_CAP_WORKAROUND;
22c213
+    smc->default_caps.caps[SPAPR_CAP_SBBC] = SPAPR_CAP_WORKAROUND;
22c213
+    smc->default_caps.caps[SPAPR_CAP_IBS] = SPAPR_CAP_FIXED_CCD;
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel730sxxm, "rhel7.3.0-sxxm", false);
22c213
+
22c213
+/*
22c213
+ * pseries-rhel7.2.0
22c213
+ */
22c213
+/* Should be like spapr_compat_2_5 + 2_4 + 2_3, but "dynamic-reconfiguration"
22c213
+ * has been backported to RHEL7_2 so we don't need it here.
22c213
+ */
22c213
+
22c213
+GlobalProperty spapr_compat_rhel7_2[] = {
22c213
+    { "spapr-vlan", "use-rx-buffer-pools", "off" },
22c213
+    { TYPE_SPAPR_PCI_HOST_BRIDGE, "ddw", "off" },
22c213
+};
22c213
+const size_t spapr_compat_rhel7_2_len = G_N_ELEMENTS(spapr_compat_rhel7_2);
22c213
+
22c213
+static void spapr_machine_rhel720_class_options(MachineClass *mc)
22c213
+{
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_CLASS(mc);
22c213
+
22c213
+    spapr_machine_rhel730_class_options(mc);
22c213
+    smc->use_ohci_by_default = true;
22c213
+    mc->has_hotpluggable_cpus = NULL;
22c213
+    compat_props_add(mc->compat_props, hw_compat_rhel_7_2, hw_compat_rhel_7_2_len);
22c213
+    compat_props_add(mc->compat_props, spapr_compat_rhel7_2, spapr_compat_rhel7_2_len);
22c213
+}
22c213
+
22c213
+DEFINE_SPAPR_MACHINE(rhel720, "rhel7.2.0", false);
22c213
 
22c213
 static void spapr_machine_register_types(void)
22c213
 {
22c213
diff --git a/hw/ppc/spapr_cpu_core.c b/hw/ppc/spapr_cpu_core.c
22c213
index 301cd7b4e4..ba5a8fb82b 100644
22c213
--- a/hw/ppc/spapr_cpu_core.c
22c213
+++ b/hw/ppc/spapr_cpu_core.c
22c213
@@ -24,6 +24,7 @@
22c213
 #include "sysemu/reset.h"
22c213
 #include "sysemu/hw_accel.h"
22c213
 #include "qemu/error-report.h"
22c213
+#include "cpu-models.h"
22c213
 
22c213
 static void spapr_reset_vcpu(PowerPCCPU *cpu)
22c213
 {
22c213
@@ -242,6 +243,7 @@ static void spapr_realize_vcpu(PowerPCCPU *cpu, SpaprMachineState *spapr,
22c213
     CPUPPCState *env = &cpu->env;
22c213
     CPUState *cs = CPU(cpu);
22c213
     Error *local_err = NULL;
22c213
+    SpaprMachineClass *smc = SPAPR_MACHINE_GET_CLASS(spapr);
22c213
 
22c213
     object_property_set_bool(OBJECT(cpu), true, "realized", &local_err);
22c213
     if (local_err) {
22c213
@@ -254,6 +256,17 @@ static void spapr_realize_vcpu(PowerPCCPU *cpu, SpaprMachineState *spapr,
22c213
     cpu_ppc_set_vhyp(cpu, PPC_VIRTUAL_HYPERVISOR(spapr));
22c213
     kvmppc_set_papr(cpu);
22c213
 
22c213
+    if (!smc->has_power9_support &&
22c213
+        (((spapr->max_compat_pvr &&
22c213
+           ppc_compat_cmp(spapr->max_compat_pvr,
22c213
+                          CPU_POWERPC_LOGICAL_3_00) >= 0)) ||
22c213
+          (!spapr->max_compat_pvr &&
22c213
+           ppc_check_compat(cpu, CPU_POWERPC_LOGICAL_3_00, 0, 0)))) {
22c213
+        error_set(errp, ERROR_CLASS_DEVICE_NOT_FOUND,
22c213
+                  "POWER9 CPU is not supported by this machine class");
22c213
+        return;
22c213
+    }
22c213
+
22c213
     if (spapr_irq_cpu_intc_create(spapr, cpu, &local_err) < 0) {
22c213
         goto error_intc_create;
22c213
     }
22c213
diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h
22c213
index d5ab5ea7b2..aa89cc4a95 100644
22c213
--- a/include/hw/ppc/spapr.h
22c213
+++ b/include/hw/ppc/spapr.h
22c213
@@ -125,6 +125,7 @@ struct SpaprMachineClass {
22c213
     bool linux_pci_probe;
22c213
     bool smp_threads_vsmt; /* set VSMT to smp_threads by default */
22c213
 
22c213
+    bool has_power9_support;
22c213
     void (*phb_placement)(SpaprMachineState *spapr, uint32_t index,
22c213
                           uint64_t *buid, hwaddr *pio, 
22c213
                           hwaddr *mmio32, hwaddr *mmio64,
22c213
diff --git a/target/ppc/compat.c b/target/ppc/compat.c
22c213
index 7de4bf3122..3e2e35342d 100644
22c213
--- a/target/ppc/compat.c
22c213
+++ b/target/ppc/compat.c
22c213
@@ -105,8 +105,19 @@ static const CompatInfo *compat_by_pvr(uint32_t pvr)
22c213
     return NULL;
22c213
 }
22c213
 
22c213
+long ppc_compat_cmp(uint32_t pvr1, uint32_t pvr2)
22c213
+{
22c213
+    const CompatInfo *compat1 = compat_by_pvr(pvr1);
22c213
+    const CompatInfo *compat2 = compat_by_pvr(pvr2);
22c213
+
22c213
+    g_assert(compat1);
22c213
+    g_assert(compat2);
22c213
+
22c213
+    return compat1 - compat2;
22c213
+}
22c213
+
22c213
 static bool pcc_compat(PowerPCCPUClass *pcc, uint32_t compat_pvr,
22c213
-                       uint32_t min_compat_pvr, uint32_t max_compat_pvr)
22c213
+                      uint32_t min_compat_pvr, uint32_t max_compat_pvr)
22c213
 {
22c213
     const CompatInfo *compat = compat_by_pvr(compat_pvr);
22c213
     const CompatInfo *min = compat_by_pvr(min_compat_pvr);
22c213
diff --git a/target/ppc/cpu.h b/target/ppc/cpu.h
22c213
index e3e82327b7..5c53801cfd 100644
22c213
--- a/target/ppc/cpu.h
22c213
+++ b/target/ppc/cpu.h
22c213
@@ -1367,6 +1367,7 @@ static inline int cpu_mmu_index(CPUPPCState *env, bool ifetch)
22c213
 
22c213
 /* Compatibility modes */
22c213
 #if defined(TARGET_PPC64)
22c213
+long ppc_compat_cmp(uint32_t pvr1, uint32_t pvr2);
22c213
 bool ppc_check_compat(PowerPCCPU *cpu, uint32_t compat_pvr,
22c213
                       uint32_t min_compat_pvr, uint32_t max_compat_pvr);
22c213
 bool ppc_type_check_compat(const char *cputype, uint32_t compat_pvr,
22c213
-- 
22c213
2.21.0
22c213