Re: [Qemu-devel] [PATCH] spapr: support CPU hotplug

2013-08-27 Thread Alexander Graf
On 27.08.2013, at 11:03, Alexey Kardashevskiy wrote: > On 08/27/2013 06:54 PM, Alexander Graf wrote: >> >> On 27.08.2013, at 09:41, Alexey Kardashevskiy wrote: >> >>> On 08/27/2013 05:02 PM, Paolo Bonzini wrote: Il 27/08/2013 08:37, Alexey Kardashevskiy ha scritto: >>> So this is here

Re: [Qemu-devel] [PATCH] spapr: support CPU hotplug

2013-08-27 Thread Alexey Kardashevskiy
On 08/27/2013 06:54 PM, Alexander Graf wrote: > > On 27.08.2013, at 09:41, Alexey Kardashevskiy wrote: > >> On 08/27/2013 05:02 PM, Paolo Bonzini wrote: >>> Il 27/08/2013 08:37, Alexey Kardashevskiy ha scritto: >> So this is here to make sure we don't accidentally get out of halted >> st

Re: [Qemu-devel] [PATCH] spapr: support CPU hotplug

2013-08-27 Thread Alexander Graf
On 27.08.2013, at 09:41, Alexey Kardashevskiy wrote: > On 08/27/2013 05:02 PM, Paolo Bonzini wrote: >> Il 27/08/2013 08:37, Alexey Kardashevskiy ha scritto: > So this is here to make sure we don't accidentally get out of halted > state by an interrupt on that vcpu. Could you please someh

Re: [Qemu-devel] [PATCH] spapr: support CPU hotplug

2013-08-27 Thread Alexey Kardashevskiy
On 08/27/2013 05:02 PM, Paolo Bonzini wrote: > Il 27/08/2013 08:37, Alexey Kardashevskiy ha scritto: So this is here to make sure we don't accidentally get out of halted state by an interrupt on that vcpu. Could you please somehow make that part obvious? Either by adding a comment

Re: [Qemu-devel] [PATCH] spapr: support CPU hotplug

2013-08-27 Thread Paolo Bonzini
Il 27/08/2013 08:37, Alexey Kardashevskiy ha scritto: >>> So this is here to make sure we don't accidentally get out of halted state >>> by an interrupt on that vcpu. Could you please somehow make that part >>> obvious? Either by adding a comment or by only explicitly masking DEC and >>> EE and

Re: [Qemu-devel] [PATCH] spapr: support CPU hotplug

2013-08-26 Thread Alexey Kardashevskiy
On 08/27/2013 01:29 AM, David Gibson wrote: > On Mon, Aug 26, 2013 at 02:24:49PM +0200, Alexander Graf wrote: >> >> On 23.08.2013, at 13:30, Alexey Kardashevskiy wrote: >> >>> PAPR+ requires two RTAS calls to be supported by the hypervisor in >>> order to allow hotplugging VCPUs from the guest. The

Re: [Qemu-devel] [PATCH] spapr: support CPU hotplug

2013-08-26 Thread David Gibson
On Mon, Aug 26, 2013 at 02:24:49PM +0200, Alexander Graf wrote: > > On 23.08.2013, at 13:30, Alexey Kardashevskiy wrote: > > > PAPR+ requires two RTAS calls to be supported by the hypervisor in > > order to allow hotplugging VCPUs from the guest. The "start-cpu" RTAS > > call was already there bu

Re: [Qemu-devel] [PATCH] spapr: support CPU hotplug

2013-08-26 Thread Alexander Graf
On 23.08.2013, at 13:30, Alexey Kardashevskiy wrote: > PAPR+ requires two RTAS calls to be supported by the hypervisor in > order to allow hotplugging VCPUs from the guest. The "start-cpu" RTAS > call was already there but "stop-self" was not. > > This adds the "stop-self" RTAS call. > > Signed

[Qemu-devel] [PATCH] spapr: support CPU hotplug

2013-08-23 Thread Alexey Kardashevskiy
PAPR+ requires two RTAS calls to be supported by the hypervisor in order to allow hotplugging VCPUs from the guest. The "start-cpu" RTAS call was already there but "stop-self" was not. This adds the "stop-self" RTAS call. Signed-off-by: Alexey Kardashevskiy --- hw/ppc/spapr_rtas.c | 14