From: Cornelia Huck <cohuck@redhat.com>
To: Greg Kurz <groug@kaod.org>
Cc: qemu-devel@nongnu.org, qemu-ppc@nongnu.org,
David Gibson <david@gibson.dropbear.id.au>,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>,
Igor Mammedov <imammedo@redhat.com>,
Markus Armbruster <armbru@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 0/2] monitor: increase the refcount of the current CPU
Date: Fri, 13 Oct 2017 10:46:22 +0200 [thread overview]
Message-ID: <20171013104622.2f0b4749.cohuck@redhat.com> (raw)
In-Reply-To: <150788370618.25736.8030708425923435364.stgit@bahia>
On Fri, 13 Oct 2017 10:35:06 +0200
Greg Kurz <groug@kaod.org> wrote:
> If a CPU selected with the "cpu" command is hot-unplugged then "info cpus"
> causes QEMU to exit:
>
> (qemu) device_del cpu1
> (qemu) info cpus
> qemu:qemu_cpu_kick_thread: No such process
>
> I could verify that this happens with x86 and ppc, but I guess s390x is
> also impacted.
Not really, as s390x does not support cpu unplug.
>
> This series tries to fix the issue by using object_ref() and object_unref()
> in the monitor code. For this to work on ppc, some preliminary work is
> needed to let QOM handle the CPU object lifecycle.
>
> Please comment.
>
> --
> Greg
>
> ---
>
> Greg Kurz (2):
> spapr_cpu_core: instantiate CPUs separately
> monitor: add proper reference counting of the current CPU
>
>
> hw/ppc/spapr.c | 10 +++-------
> hw/ppc/spapr_cpu_core.c | 29 +++++++++--------------------
> include/hw/ppc/spapr_cpu_core.h | 2 +-
> monitor.c | 12 ++++++++++++
> 4 files changed, 25 insertions(+), 28 deletions(-)
>
next prev parent reply other threads:[~2017-10-13 8:46 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-13 8:35 [Qemu-devel] [PATCH 0/2] monitor: increase the refcount of the current CPU Greg Kurz
2017-10-13 8:35 ` [Qemu-devel] [PATCH 1/2] spapr_cpu_core: instantiate CPUs separately Greg Kurz
2017-10-13 9:21 ` Igor Mammedov
2017-10-13 9:43 ` Greg Kurz
2017-10-13 8:35 ` [Qemu-devel] [PATCH 2/2] monitor: add proper reference counting of the current CPU Greg Kurz
2017-10-13 9:24 ` Igor Mammedov
2017-10-13 10:05 ` Greg Kurz
2017-10-16 8:07 ` Igor Mammedov
2017-10-16 17:16 ` Greg Kurz
2017-10-13 8:46 ` Cornelia Huck [this message]
2017-10-13 9:14 ` [Qemu-devel] [PATCH 0/2] monitor: increase the refcount " Greg Kurz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171013104622.2f0b4749.cohuck@redhat.com \
--to=cohuck@redhat.com \
--cc=armbru@redhat.com \
--cc=david@gibson.dropbear.id.au \
--cc=dgilbert@redhat.com \
--cc=groug@kaod.org \
--cc=imammedo@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.