[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v5 18/18] spapr: Fold spapr_node0_size() into its only caller
From: |
David Gibson |
Subject: |
[PATCH v5 18/18] spapr: Fold spapr_node0_size() into its only caller |
Date: |
Thu, 20 Feb 2020 14:23:16 +1100 |
The Real Mode Area (RMA) needs to fit within the NUMA node owning memory
at address 0. That's usually node 0, but can be a later one if there are
some nodes which have no memory (only CPUs).
This is currently handled by the spapr_node0_size() helper. It has only
one caller, so there's not a lot of point splitting it out. It's also
extremely easy to misread the code as clamping to the size of the smallest
node rather than the first node with any memory.
So, fold it into the caller, and add some commentary to make it a bit
clearer exactly what it's doing.
Signed-off-by: David Gibson <address@hidden>
---
hw/ppc/spapr.c | 37 +++++++++++++++++++++----------------
1 file changed, 21 insertions(+), 16 deletions(-)
diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
index f0354b699d..9ba645c9cb 100644
--- a/hw/ppc/spapr.c
+++ b/hw/ppc/spapr.c
@@ -296,20 +296,6 @@ static void spapr_populate_pa_features(SpaprMachineState
*spapr,
_FDT((fdt_setprop(fdt, offset, "ibm,pa-features", pa_features, pa_size)));
}
-static hwaddr spapr_node0_size(MachineState *machine)
-{
- if (machine->numa_state->num_nodes) {
- int i;
- for (i = 0; i < machine->numa_state->num_nodes; ++i) {
- if (machine->numa_state->nodes[i].node_mem) {
- return MIN(pow2floor(machine->numa_state->nodes[i].node_mem),
- machine->ram_size);
- }
- }
- }
- return machine->ram_size;
-}
-
static void add_str(GString *s, const gchar *s1)
{
g_string_append_len(s, s1, strlen(s1) + 1);
@@ -2652,10 +2638,24 @@ static hwaddr spapr_rma_size(SpaprMachineState *spapr,
Error **errp)
{
MachineState *machine = MACHINE(spapr);
hwaddr rma_size = machine->ram_size;
- hwaddr node0_size = spapr_node0_size(machine);
/* RMA has to fit in the first NUMA node */
- rma_size = MIN(rma_size, node0_size);
+ if (machine->numa_state->num_nodes) {
+ /*
+ * It's possible for there to be some zero-memory nodes first
+ * in the list. We need the RMA to fit inside the memory of
+ * the first node which actually has some memory.
+ */
+ int i;
+
+ for (i = 0; i < machine->numa_state->num_nodes; ++i) {
+ if (machine->numa_state->nodes[i].node_mem != 0) {
+ rma_size = MIN(rma_size,
+ machine->numa_state->nodes[i].node_mem);
+ break;
+ }
+ }
+ }
/*
* VRMA access is via a special 1TiB SLB mapping, so the RMA can
@@ -2672,6 +2672,11 @@ static hwaddr spapr_rma_size(SpaprMachineState *spapr,
Error **errp)
spapr->rma_size = MIN(spapr->rma_size, smc->rma_limit);
}
+ /*
+ * RMA size must be a power of 2
+ */
+ rma_size = pow2floor(rma_size);
+
if (rma_size < (MIN_RMA_SLOF * MiB)) {
error_setg(errp,
"pSeries SLOF firmware requires >= %ldMiB guest RMA (Real Mode Area)",
--
2.24.1
- [PATCH v5 09/18] target/ppc: Correct RMLS table, (continued)
- [PATCH v5 09/18] target/ppc: Correct RMLS table, David Gibson, 2020/02/19
- [PATCH v5 02/18] ppc: Remove stub of PPC970 HID4 implementation, David Gibson, 2020/02/19
- [PATCH v5 14/18] spapr,ppc: Simplify signature of kvmppc_rma_size(), David Gibson, 2020/02/19
- [PATCH v5 06/18] target/ppc: Remove RMOR register from POWER9 & POWER10, David Gibson, 2020/02/19
- [PATCH v5 15/18] spapr: Don't attempt to clamp RMA to VRMA constraint, David Gibson, 2020/02/19
- [PATCH v5 13/18] spapr: Don't use weird units for MIN_RMA_SLOF, David Gibson, 2020/02/19
- [PATCH v5 11/18] target/ppc: Streamline construction of VRMA SLB entry, David Gibson, 2020/02/19
- [PATCH v5 16/18] spapr: Don't clamp RMA to 16GiB on new machine types, David Gibson, 2020/02/19
- [PATCH v5 03/18] target/ppc: Correct handling of real mode accesses with vhyp on hash MMU, David Gibson, 2020/02/19
- [PATCH v5 17/18] spapr: Clean up RMA size calculation, David Gibson, 2020/02/19
- [PATCH v5 18/18] spapr: Fold spapr_node0_size() into its only caller,
David Gibson <=