spapr: Do NVDIMM/PC-DIMM device hotplug sanity checks at pre-plug only
commitea042c53f4d9e48e7b3c84f5d0eb70a84aa34413
authorGreg Kurz <groug@kaod.org>
Fri, 20 Nov 2020 23:42:01 +0000 (21 00:42 +0100)
committerDavid Gibson <david@gibson.dropbear.id.au>
Mon, 14 Dec 2020 04:50:55 +0000 (14 15:50 +1100)
treed2a9f052765816b28a1c0dadd2c1f75e7eeb6250
parent9e4dc0a1c2743cb115ebdb32258a78b328c05205
spapr: Do NVDIMM/PC-DIMM device hotplug sanity checks at pre-plug only

Pre-plug of a memory device, be it an NVDIMM or a PC-DIMM, ensures
that the memory slot is available and that addresses don't overlap
with existing memory regions. The corresponding DRCs in the LMB
and PMEM namespaces are thus necessarily attachable at plug time.

Pass &error_abort to spapr_drc_attach() in spapr_add_lmbs() and
spapr_add_nvdimm(). This allows to greatly simplify error handling
on the plug path.

Signed-off-by: Greg Kurz <groug@kaod.org>
Message-Id: <20201120234208.683521-3-groug@kaod.org>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
hw/ppc/spapr.c
hw/ppc/spapr_nvdimm.c
include/hw/ppc/spapr_nvdimm.h