Re: [Qemu-devel] [PATCH] xics_kvm: cache already enabled vCPU ids

2017-05-17 Thread David Gibson
On Wed, May 17, 2017 at 04:38:20PM +0200, Greg Kurz wrote: > Since commit a45863bda90d ("xics_kvm: Don't enable KVM_CAP_IRQ_XICS if > already enabled"), we were able to re-hotplug a vCPU that had been hot- > unplugged ealier, thanks to a boolean flag in ICPState that we set when > enabling KVM_CAP_

Re: [Qemu-devel] [PATCH] xics_kvm: cache already enabled vCPU ids

2017-05-17 Thread Cédric Le Goater
On 05/17/2017 04:38 PM, Greg Kurz wrote: > Since commit a45863bda90d ("xics_kvm: Don't enable KVM_CAP_IRQ_XICS if > already enabled"), we were able to re-hotplug a vCPU that had been hot- > unplugged ealier, thanks to a boolean flag in ICPState that we set when > enabling KVM_CAP_IRQ_XICS. > > Thi

Re: [Qemu-devel] [PATCH] xics_kvm: cache already enabled vCPU ids

2017-05-17 Thread Laurent Vivier
On 17/05/2017 17:18, Laurent Vivier wrote: > On 17/05/2017 16:38, Greg Kurz wrote: >> Since commit a45863bda90d ("xics_kvm: Don't enable KVM_CAP_IRQ_XICS if >> already enabled"), we were able to re-hotplug a vCPU that had been hot- >> unplugged ealier, thanks to a boolean flag in ICPState that we s

Re: [Qemu-devel] [PATCH] xics_kvm: cache already enabled vCPU ids

2017-05-17 Thread Laurent Vivier
On 17/05/2017 16:38, Greg Kurz wrote: > Since commit a45863bda90d ("xics_kvm: Don't enable KVM_CAP_IRQ_XICS if > already enabled"), we were able to re-hotplug a vCPU that had been hot- > unplugged ealier, thanks to a boolean flag in ICPState that we set when > enabling KVM_CAP_IRQ_XICS. > > This c

[Qemu-devel] [PATCH] xics_kvm: cache already enabled vCPU ids

2017-05-17 Thread Greg Kurz
Since commit a45863bda90d ("xics_kvm: Don't enable KVM_CAP_IRQ_XICS if already enabled"), we were able to re-hotplug a vCPU that had been hot- unplugged ealier, thanks to a boolean flag in ICPState that we set when enabling KVM_CAP_IRQ_XICS. This could work because the lifecycle of all ICPState ob