yeahuh / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone
76daa3
From c0af45c7da067bc9679b0b491971b026e5722876 Mon Sep 17 00:00:00 2001
76daa3
From: David Gibson <dgibson@redhat.com>
76daa3
Date: Thu, 27 Apr 2017 02:15:58 +0200
76daa3
Subject: [PATCH 23/23] spapr: Workaround for broken radix guests
76daa3
76daa3
RH-Author: David Gibson <dgibson@redhat.com>
76daa3
Message-id: <20170427021558.4884-8-dgibson@redhat.com>
76daa3
Patchwork-id: 74920
76daa3
O-Subject: [Pegas-1.0 qemu-kvm-rhev PATCH 7/7] spapr: Workaround for broken radix guests
76daa3
Bugzilla: 1368786
76daa3
RH-Acked-by: Thomas Huth <thuth@redhat.com>
76daa3
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
76daa3
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
76daa3
76daa3
From: Sam Bobroff <sam.bobroff@au1.ibm.com>
76daa3
76daa3
For a little while around 4.9, Linux kernels that saw the radix bit in
76daa3
ibm,pa-features would attempt to set up the MMU as if they were a
76daa3
hypervisor, even if they were a guest, which would cause them to
76daa3
crash.
76daa3
76daa3
Work around this by detecting pre-ISA 3.0 guests by their lack of that
76daa3
bit in option vector 1, and then removing the radix bit from
76daa3
ibm,pa-features. Note: This now requires regeneration of that node
76daa3
after CAS negotiation.
76daa3
76daa3
Signed-off-by: Sam Bobroff <sam.bobroff@au1.ibm.com>
76daa3
[dwg: Fix style nits]
76daa3
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
76daa3
76daa3
(cherry picked from commit e957f6a9b92439a222ecd4ff1c8cdc9700710c72)
76daa3
76daa3
Siged-off-by: David Gibson <dgibson@redhat.com>
76daa3
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
76daa3
---
76daa3
 hw/ppc/spapr.c              | 15 +++++++++++++--
76daa3
 hw/ppc/spapr_hcall.c        |  6 ++++--
76daa3
 include/hw/ppc/spapr.h      |  1 +
76daa3
 include/hw/ppc/spapr_ovec.h |  3 +++
76daa3
 4 files changed, 21 insertions(+), 4 deletions(-)
76daa3
76daa3
diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
76daa3
index 5a5196a..cc3ccc1 100644
76daa3
--- a/hw/ppc/spapr.c
76daa3
+++ b/hw/ppc/spapr.c
76daa3
@@ -228,7 +228,8 @@ static int spapr_fixup_cpu_numa_dt(void *fdt, int offset, CPUState *cs)
76daa3
 }
76daa3
 
76daa3
 /* Populate the "ibm,pa-features" property */
76daa3
-static void spapr_populate_pa_features(CPUPPCState *env, void *fdt, int offset)
76daa3
+static void spapr_populate_pa_features(CPUPPCState *env, void *fdt, int offset,
76daa3
+                                      bool legacy_guest)
76daa3
 {
76daa3
     uint8_t pa_features_206[] = { 6, 0,
76daa3
         0xf6, 0x1f, 0xc7, 0x00, 0x80, 0xc0 };
76daa3
@@ -295,6 +296,12 @@ static void spapr_populate_pa_features(CPUPPCState *env, void *fdt, int offset)
76daa3
     if (kvmppc_has_cap_htm() && pa_size > 24) {
76daa3
         pa_features[24] |= 0x80;    /* Transactional memory support */
76daa3
     }
76daa3
+    if (legacy_guest && pa_size > 40) {
76daa3
+        /* Workaround for broken kernels that attempt (guest) radix
76daa3
+         * mode when they can't handle it, if they see the radix bit set
76daa3
+         * in pa-features. So hide it from them. */
76daa3
+        pa_features[40 + 2] &= ~0x80; /* Radix MMU */
76daa3
+    }
76daa3
 
76daa3
     _FDT((fdt_setprop(fdt, offset, "ibm,pa-features", pa_features, pa_size)));
76daa3
 }
76daa3
@@ -309,6 +316,7 @@ static int spapr_fixup_cpu_dt(void *fdt, sPAPRMachineState *spapr)
76daa3
 
76daa3
     CPU_FOREACH(cs) {
76daa3
         PowerPCCPU *cpu = POWERPC_CPU(cs);
76daa3
+        CPUPPCState *env = &cpu->env;
76daa3
         DeviceClass *dc = DEVICE_GET_CLASS(cs);
76daa3
         int index = ppc_get_vcpu_dt_id(cpu);
76daa3
         int compat_smt = MIN(smp_threads, ppc_compat_max_threads(cpu));
76daa3
@@ -350,6 +358,9 @@ static int spapr_fixup_cpu_dt(void *fdt, sPAPRMachineState *spapr)
76daa3
         if (ret < 0) {
76daa3
             return ret;
76daa3
         }
76daa3
+
76daa3
+        spapr_populate_pa_features(env, fdt, offset,
76daa3
+                                         spapr->cas_legacy_guest_workaround);
76daa3
     }
76daa3
     return ret;
76daa3
 }
76daa3
@@ -547,7 +558,7 @@ static void spapr_populate_cpu_dt(CPUState *cs, void *fdt, int offset,
76daa3
                           page_sizes_prop, page_sizes_prop_size)));
76daa3
     }
76daa3
 
76daa3
-    spapr_populate_pa_features(env, fdt, offset);
76daa3
+    spapr_populate_pa_features(env, fdt, offset, false);
76daa3
 
76daa3
     _FDT((fdt_setprop_cell(fdt, offset, "ibm,chip-id",
76daa3
                            cs->cpu_index / vcpus_per_socket)));
76daa3
diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c
76daa3
index cbd1f29..9f18f75 100644
76daa3
--- a/hw/ppc/spapr_hcall.c
76daa3
+++ b/hw/ppc/spapr_hcall.c
76daa3
@@ -1062,7 +1062,7 @@ static target_ulong h_client_architecture_support(PowerPCCPU *cpu,
76daa3
     uint32_t max_compat = cpu->max_compat;
76daa3
     uint32_t best_compat = 0;
76daa3
     int i;
76daa3
-    sPAPROptionVector *ov5_guest, *ov5_cas_old, *ov5_updates;
76daa3
+    sPAPROptionVector *ov1_guest, *ov5_guest, *ov5_cas_old, *ov5_updates;
76daa3
     bool guest_radix;
76daa3
 
76daa3
     /*
76daa3
@@ -1114,6 +1114,7 @@ static target_ulong h_client_architecture_support(PowerPCCPU *cpu,
76daa3
     /* For the future use: here @ov_table points to the first option vector */
76daa3
     ov_table = list;
76daa3
 
76daa3
+    ov1_guest = spapr_ovec_parse_vector(ov_table, 1);
76daa3
     ov5_guest = spapr_ovec_parse_vector(ov_table, 5);
76daa3
     if (spapr_ovec_test(ov5_guest, OV5_MMU_BOTH)) {
76daa3
         error_report("guest requested hash and radix MMU, which is invalid.");
76daa3
@@ -1155,7 +1156,8 @@ static target_ulong h_client_architecture_support(PowerPCCPU *cpu,
76daa3
             exit(EXIT_FAILURE);
76daa3
         }
76daa3
     }
76daa3
-
76daa3
+    spapr->cas_legacy_guest_workaround = !spapr_ovec_test(ov1_guest,
76daa3
+                                                          OV1_PPC_3_00);
76daa3
     if (!spapr->cas_reboot) {
76daa3
         spapr->cas_reboot =
76daa3
             (spapr_h_cas_compose_response(spapr, args[1], args[2],
76daa3
diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h
76daa3
index 247c969..a6ef8cb 100644
76daa3
--- a/include/hw/ppc/spapr.h
76daa3
+++ b/include/hw/ppc/spapr.h
76daa3
@@ -77,6 +77,7 @@ struct sPAPRMachineState {
76daa3
     sPAPROptionVector *ov5;         /* QEMU-supported option vectors */
76daa3
     sPAPROptionVector *ov5_cas;     /* negotiated (via CAS) option vectors */
76daa3
     bool cas_reboot;
76daa3
+    bool cas_legacy_guest_workaround;
76daa3
 
76daa3
     Notifier epow_notifier;
76daa3
     QTAILQ_HEAD(, sPAPREventLogEntry) pending_events;
76daa3
diff --git a/include/hw/ppc/spapr_ovec.h b/include/hw/ppc/spapr_ovec.h
76daa3
index f7f2abe..f088833 100644
76daa3
--- a/include/hw/ppc/spapr_ovec.h
76daa3
+++ b/include/hw/ppc/spapr_ovec.h
76daa3
@@ -43,6 +43,9 @@ typedef struct sPAPROptionVector sPAPROptionVector;
76daa3
 
76daa3
 #define OV_BIT(byte, bit) ((byte - 1) * BITS_PER_BYTE + bit)
76daa3
 
76daa3
+/* option vector 1 */
76daa3
+#define OV1_PPC_3_00            OV_BIT(3, 0) /* guest supports PowerPC 3.00? */
76daa3
+
76daa3
 /* option vector 5 */
76daa3
 #define OV5_DRCONF_MEMORY       OV_BIT(2, 2)
76daa3
 #define OV5_FORM1_AFFINITY      OV_BIT(5, 0)
76daa3
-- 
76daa3
1.8.3.1
76daa3