This second version contains fixes based on the v1 review made by Greg. changes from v1: - patch 1: * added Greg's R-b - patch 2: * fixed 'order' typo * changed exit(1) to exit(EXIT_FAILURE) * added Greg's R-b - patch 3: * clarified in the function comment the rounding of distance values 30, 60 and 120 * fixed a bug where distances 30, 60 and 120 wasn't being PAPRified (use <= instead of just <) - patch 4: * made the changes multi-line and more explicit, allowing to easily see the differences between current and legacy arrays - patch 5: * turned spapr_numa_get_NUMA_level() to lowercase * added a switch clause for distance = 10 * removed MachineState parameter of spapr_numa_define_associativity_domains() * clarified the reason why the associativity domains need to be initialized with 'node_id' in the comment * we're now handling distances > 120 appropriately (no NUMA match case)
v1 link: https://lists.gnu.org/archive/html/qemu-devel/2020-09/msg08549.html Daniel Henrique Barboza (6): spapr: add spapr_machine_using_legacy_numa() helper spapr_numa: forbid asymmetrical NUMA setups spapr_numa: translate regular NUMA distance to PAPR distance spapr_numa: change reference-points and maxdomain settings spapr_numa: consider user input when defining associativity specs/ppc-spapr-numa: update with new NUMA support docs/specs/ppc-spapr-numa.rst | 213 +++++++++++++++++++++++++++++++++ hw/ppc/spapr.c | 12 ++ hw/ppc/spapr_numa.c | 217 ++++++++++++++++++++++++++++++++-- include/hw/ppc/spapr.h | 2 + 4 files changed, 436 insertions(+), 8 deletions(-) -- 2.26.2