diff options
| author | Greg Kurz <groug@kaod.org> | 2019-10-24 16:27:33 +0200 |
|---|---|---|
| committer | Laurent Vivier <lvivier@redhat.com> | 2019-11-18 11:50:25 +0100 |
| commit | 0a83b47055246d3942084f03fc54731c4fb9b731 (patch) | |
| tree | 15a0f96fe9cac621b126b962c1f8c5ea7089e225 /hw/intc/xive.c | |
| parent | 35886de140b7ff781b775d2da5e7475e8a8cb4c6 (diff) | |
| download | qemu-0a83b47055246d3942084f03fc54731c4fb9b731.zip | |
ppc: Skip partially initialized vCPUs in 'info pic'
CPU_FOREACH() can race with vCPU hotplug/unplug on sPAPR machines, ie.
we may try to print out info about a vCPU with a NULL presenter pointer.
Check that in order to prevent QEMU from crashing.
Signed-off-by: Greg Kurz <groug@kaod.org>
Message-Id: <157192725327.3146912.12047076483178652551.stgit@bahia.lan>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Diffstat (limited to 'hw/intc/xive.c')
| -rw-r--r-- | hw/intc/xive.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/hw/intc/xive.c b/hw/intc/xive.c index 952a461d53..75dce82fb2 100644 --- a/hw/intc/xive.c +++ b/hw/intc/xive.c @@ -523,9 +523,18 @@ static const char * const xive_tctx_ring_names[] = { void xive_tctx_pic_print_info(XiveTCTX *tctx, Monitor *mon) { - int cpu_index = tctx->cs ? tctx->cs->cpu_index : -1; + int cpu_index; int i; + /* Skip partially initialized vCPUs. This can happen on sPAPR when vCPUs + * are hot plugged or unplugged. + */ + if (!tctx) { + return; + } + + cpu_index = tctx->cs ? tctx->cs->cpu_index : -1; + if (kvm_irqchip_in_kernel()) { Error *local_err = NULL; |