Skip to content

Commit 14de3d4

Browse files
gkurzdgibson
authored andcommitted
ppc/pnv: Make PSI device types not user creatable
QEMU aborts with -device pnv-psi-POWER8: $ qemu-system-ppc64 -device pnv-psi-POWER8 qemu-system-ppc64: hw/intc/xics.c:605: ics_realize: Assertion `ics->xics' failed. Aborted (core dumped) The Processor Service Interface Controller is an internal device. It should only be instantiated by the chip, which takes care of configuring the link required by the ICS object in the case of POWER8. It doesn't make sense for a user to specify it on the command line. Note that the PSI model for POWER8 was added 3 yrs ago but the devices weren't available on the command line because of a bug that was fixed by recent commit 2f35254 ("pnv/psi: Correct the pnv-psi* devices not to be sysbus devices"). Fixes: 54f59d7 ("ppc/pnv: Add cut down PSI bridge model and hookup external interrupt") Reported-by: Thomas Huth <[email protected]> Signed-off-by: Greg Kurz <[email protected]> Message-Id: <[email protected]> Signed-off-by: David Gibson <[email protected]>
1 parent 9fc8711 commit 14de3d4

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

hw/ppc/pnv_psi.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -929,6 +929,7 @@ static void pnv_psi_class_init(ObjectClass *klass, void *data)
929929
dc->desc = "PowerNV PSI Controller";
930930
device_class_set_props(dc, pnv_psi_properties);
931931
dc->reset = pnv_psi_reset;
932+
dc->user_creatable = false;
932933
}
933934

934935
static const TypeInfo pnv_psi_info = {

0 commit comments

Comments
 (0)