[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 2/6] spapr_numa: forbid asymmetrical NUMA setups
From: |
David Gibson |
Subject: |
Re: [PATCH v2 2/6] spapr_numa: forbid asymmetrical NUMA setups |
Date: |
Fri, 25 Sep 2020 12:36:48 +1000 |
On Thu, Sep 24, 2020 at 04:50:54PM -0300, Daniel Henrique Barboza wrote:
> The pSeries machine does not support asymmetrical NUMA
> configurations. This doesn't make much of a different
> since we're not using user input for pSeries NUMA setup,
> but this will change in the next patches.
>
> To avoid breaking existing setups, gate this change by
> checking for legacy NUMA support.
>
> Reviewed-by: Greg Kurz <groug@kaod.org>
> Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
> ---
> hw/ppc/spapr_numa.c | 34 ++++++++++++++++++++++++++++++++++
> 1 file changed, 34 insertions(+)
>
> diff --git a/hw/ppc/spapr_numa.c b/hw/ppc/spapr_numa.c
> index 64fe567f5d..fe395e80a3 100644
> --- a/hw/ppc/spapr_numa.c
> +++ b/hw/ppc/spapr_numa.c
> @@ -19,6 +19,24 @@
> /* Moved from hw/ppc/spapr_pci_nvlink2.c */
> #define SPAPR_GPU_NUMA_ID (cpu_to_be32(1))
>
> +static bool spapr_numa_is_symmetrical(MachineState *ms)
> +{
> + int src, dst;
> + int nb_numa_nodes = ms->numa_state->num_nodes;
> + NodeInfo *numa_info = ms->numa_state->nodes;
> +
> + for (src = 0; src < nb_numa_nodes; src++) {
> + for (dst = src; dst < nb_numa_nodes; dst++) {
> + if (numa_info[src].distance[dst] !=
> + numa_info[dst].distance[src]) {
> + return false;
> + }
> + }
> + }
> +
> + return true;
> +}
> +
> void spapr_numa_associativity_init(SpaprMachineState *spapr,
> MachineState *machine)
> {
> @@ -61,6 +79,22 @@ void spapr_numa_associativity_init(SpaprMachineState
> *spapr,
>
> spapr->numa_assoc_array[i][MAX_DISTANCE_REF_POINTS] = cpu_to_be32(i);
> }
> +
> + /*
> + * Legacy NUMA guests (pseries-5.1 and older, or guests with only
> + * 1 NUMA node) will not benefit from anything we're going to do
> + * after this point.
> + */
> + if (spapr_machine_using_legacy_numa(spapr)) {
> + return;
> + }
> +
> + if (!spapr_numa_is_symmetrical(machine)) {
> + error_report("Asymmetrical NUMA topologies aren't supported "
> + "in the pSeries machine");
> + exit(EXIT_FAILURE);
> + }
> +
> }
>
> void spapr_numa_write_associativity_dt(SpaprMachineState *spapr, void *fdt,
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
[PATCH v2 3/6] spapr_numa: translate regular NUMA distance to PAPR distance, Daniel Henrique Barboza, 2020/09/24
[PATCH v2 6/6] specs/ppc-spapr-numa: update with new NUMA support, Daniel Henrique Barboza, 2020/09/24