From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Graf Date: Fri, 11 Nov 2011 15:35:30 +0000 Subject: Re: [PATCH] KVM: PPC: booke: check for signals in kvmppc_vcpu_run Message-Id: <4EBD40C2.4090406@suse.de> List-Id: References: <20111108221159.GA9513@schlenkerla.am.freescale.net> <4EBD2C8D.5020005@suse.de> <20111111153206.GB14488@schlenkerla.am.freescale.net> In-Reply-To: <20111111153206.GB14488@schlenkerla.am.freescale.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Scott Wood Cc: kvm@vger.kernel.org, kvm-ppc@vger.kernel.org On 11/11/2011 04:32 PM, Scott Wood wrote: > On Fri, Nov 11, 2011 at 03:09:17PM +0100, Alexander Graf wrote: >> On 11/08/2011 11:11 PM, Scott Wood wrote: >>> Currently we check prior to returning from a lightweight exit, >>> but not prior to initial entry. >>> >>> book3s already does a similar test. >>> >>> Signed-off-by: Scott Wood >>> --- >>> arch/powerpc/kvm/booke.c | 10 +++++++++- >>> 1 files changed, 9 insertions(+), 1 deletions(-) >>> >>> diff --git a/arch/powerpc/kvm/booke.c b/arch/powerpc/kvm/booke.c >>> index b642200..9c78589 100644 >>> --- a/arch/powerpc/kvm/booke.c >>> +++ b/arch/powerpc/kvm/booke.c >>> @@ -322,11 +322,19 @@ int kvmppc_vcpu_run(struct kvm_run *kvm_run, struct kvm_vcpu *vcpu) >>> } >>> >>> local_irq_disable(); >>> + >>> + if (signal_pending(current)) { >> Any reason you're doing this after irq_disable()? > If we get a signal after the check, we want to be sure that we don't > receive the reschedule IPI until after we're in the guest, so that it > will cause another signal check. Makes sense. So the current book3s implementation is wrong? Alex