[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[qemu-xen staging] ppc/pnv: Make PSI device types not user creatable



commit 14de3d4ac58c0f63e9f5560c7e01e62e39467edd
Author:     Greg Kurz <groug@xxxxxxxx>
AuthorDate: Tue Jul 7 18:35:57 2020 +0200
Commit:     David Gibson <david@xxxxxxxxxxxxxxxxxxxxx>
CommitDate: Mon Jul 20 09:21:39 2020 +1000

    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 2f35254aa0 ("pnv/psi: Correct
    the pnv-psi* devices not to be sysbus devices").
    
    Fixes: 54f59d786c ("ppc/pnv: Add cut down PSI bridge model and hookup 
external interrupt")
    Reported-by: Thomas Huth <thuth@xxxxxxxxxx>
    Signed-off-by: Greg Kurz <groug@xxxxxxxx>
    Message-Id: <159413975752.169116.5808968580649255382.stgit@xxxxxxxxx>
    Signed-off-by: David Gibson <david@xxxxxxxxxxxxxxxxxxxxx>
---
 hw/ppc/pnv_psi.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/hw/ppc/pnv_psi.c b/hw/ppc/pnv_psi.c
index 5bdeec700e..6a479cac53 100644
--- a/hw/ppc/pnv_psi.c
+++ b/hw/ppc/pnv_psi.c
@@ -929,6 +929,7 @@ static void pnv_psi_class_init(ObjectClass *klass, void 
*data)
     dc->desc = "PowerNV PSI Controller";
     device_class_set_props(dc, pnv_psi_properties);
     dc->reset = pnv_psi_reset;
+    dc->user_creatable = false;
 }
 
 static const TypeInfo pnv_psi_info = {
--
generated by git-patchbot for /home/xen/git/qemu-xen.git#staging



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.