From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Graf Date: Fri, 11 Nov 2011 14:09:17 +0000 Subject: Re: [PATCH] KVM: PPC: booke: check for signals in kvmppc_vcpu_run Message-Id: <4EBD2C8D.5020005@suse.de> List-Id: References: <20111108221159.GA9513@schlenkerla.am.freescale.net> In-Reply-To: <20111108221159.GA9513@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/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()? Alex