|
 |
76daa3 |
From 66281117075279667d06ba78709be47e0ac75b2c Mon Sep 17 00:00:00 2001
|
|
|
76daa3 |
From: Laurent Vivier <lvivier@redhat.com>
|
|
|
76daa3 |
Date: Tue, 4 Jul 2017 16:32:56 +0200
|
|
|
76daa3 |
Subject: [PATCH 1/4] spapr: Consolidate HPT freeing code into a routine
|
|
|
76daa3 |
|
|
|
76daa3 |
RH-Author: Laurent Vivier <lvivier@redhat.com>
|
|
|
76daa3 |
Message-id: <20170704163258.9846-2-lvivier@redhat.com>
|
|
|
76daa3 |
Patchwork-id: 75706
|
|
|
76daa3 |
O-Subject: [Pegas-1.0 qemu-kvm PATCH 1/3] spapr: Consolidate HPT freeing code into a routine
|
|
|
76daa3 |
Bugzilla: 1456287
|
|
|
76daa3 |
RH-Acked-by: David Gibson <dgibson@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Thomas Huth <thuth@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
From: Bharata B Rao <bharata@linux.vnet.ibm.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
Consolidate the code that frees HPT into a separate routine
|
|
|
76daa3 |
spapr_free_hpt() as the same chunk of code is called from two places.
|
|
|
76daa3 |
|
|
|
76daa3 |
Signed-off-by: Bharata B Rao <bharata@linux.vnet.ibm.com>
|
|
|
76daa3 |
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
|
|
|
76daa3 |
(cherry picked from commit 06ec79e865a4a496e762a83126d00d0ed39205f5)
|
|
|
76daa3 |
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
|
|
|
76daa3 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
76daa3 |
---
|
|
|
76daa3 |
hw/ppc/spapr.c | 13 +++++++++----
|
|
|
76daa3 |
hw/ppc/spapr_hcall.c | 5 +----
|
|
|
76daa3 |
include/hw/ppc/spapr.h | 1 +
|
|
|
76daa3 |
3 files changed, 11 insertions(+), 8 deletions(-)
|
|
|
76daa3 |
|
|
|
76daa3 |
diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
|
|
|
76daa3 |
index aadb562..2bee3d9 100644
|
|
|
76daa3 |
--- a/hw/ppc/spapr.c
|
|
|
76daa3 |
+++ b/hw/ppc/spapr.c
|
|
|
76daa3 |
@@ -1243,16 +1243,21 @@ static int spapr_hpt_shift_for_ramsize(uint64_t ramsize)
|
|
|
76daa3 |
return shift;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
+void spapr_free_hpt(sPAPRMachineState *spapr)
|
|
|
76daa3 |
+{
|
|
|
76daa3 |
+ g_free(spapr->htab);
|
|
|
76daa3 |
+ spapr->htab = NULL;
|
|
|
76daa3 |
+ spapr->htab_shift = 0;
|
|
|
76daa3 |
+ close_htab_fd(spapr);
|
|
|
76daa3 |
+}
|
|
|
76daa3 |
+
|
|
|
76daa3 |
static void spapr_reallocate_hpt(sPAPRMachineState *spapr, int shift,
|
|
|
76daa3 |
Error **errp)
|
|
|
76daa3 |
{
|
|
|
76daa3 |
long rc;
|
|
|
76daa3 |
|
|
|
76daa3 |
/* Clean up any HPT info from a previous boot */
|
|
|
76daa3 |
- g_free(spapr->htab);
|
|
|
76daa3 |
- spapr->htab = NULL;
|
|
|
76daa3 |
- spapr->htab_shift = 0;
|
|
|
76daa3 |
- close_htab_fd(spapr);
|
|
|
76daa3 |
+ spapr_free_hpt(spapr);
|
|
|
76daa3 |
|
|
|
76daa3 |
rc = kvmppc_reset_htab(shift);
|
|
|
76daa3 |
if (rc < 0) {
|
|
|
76daa3 |
diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c
|
|
|
76daa3 |
index 2735fe9..79f3e0c 100644
|
|
|
76daa3 |
--- a/hw/ppc/spapr_hcall.c
|
|
|
76daa3 |
+++ b/hw/ppc/spapr_hcall.c
|
|
|
76daa3 |
@@ -913,10 +913,7 @@ static void spapr_check_setup_free_hpt(sPAPRMachineState *spapr,
|
|
|
76daa3 |
/* We assume RADIX, so this catches all the "Do Nothing" cases */
|
|
|
76daa3 |
} else if (!(patbe_old & PATBE1_GR)) {
|
|
|
76daa3 |
/* HASH->RADIX : Free HPT */
|
|
|
76daa3 |
- g_free(spapr->htab);
|
|
|
76daa3 |
- spapr->htab = NULL;
|
|
|
76daa3 |
- spapr->htab_shift = 0;
|
|
|
76daa3 |
- close_htab_fd(spapr);
|
|
|
76daa3 |
+ spapr_free_hpt(spapr);
|
|
|
76daa3 |
} else if (!(patbe_new & PATBE1_GR)) {
|
|
|
76daa3 |
/* RADIX->HASH || NOTHING->HASH : Allocate HPT */
|
|
|
76daa3 |
spapr_setup_hpt_and_vrma(spapr);
|
|
|
76daa3 |
diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h
|
|
|
76daa3 |
index a6ef8cb..901f27b 100644
|
|
|
76daa3 |
--- a/include/hw/ppc/spapr.h
|
|
|
76daa3 |
+++ b/include/hw/ppc/spapr.h
|
|
|
76daa3 |
@@ -599,6 +599,7 @@ int spapr_h_cas_compose_response(sPAPRMachineState *sm,
|
|
|
76daa3 |
sPAPROptionVector *ov5_updates);
|
|
|
76daa3 |
void close_htab_fd(sPAPRMachineState *spapr);
|
|
|
76daa3 |
void spapr_setup_hpt_and_vrma(sPAPRMachineState *spapr);
|
|
|
76daa3 |
+void spapr_free_hpt(sPAPRMachineState *spapr);
|
|
|
76daa3 |
sPAPRTCETable *spapr_tce_new_table(DeviceState *owner, uint32_t liobn);
|
|
|
76daa3 |
void spapr_tce_table_enable(sPAPRTCETable *tcet,
|
|
|
76daa3 |
uint32_t page_shift, uint64_t bus_offset,
|
|
|
76daa3 |
--
|
|
|
76daa3 |
1.8.3.1
|
|
|
76daa3 |
|