On 29.11.2013, at 03:01, “tiejun.chen” <tiejun.c...@windriver.com> wrote:
> Alex, > > I suppose Scott already elaborate anything you want to know. Thanks, applied with this minor patch on top: diff --git a/arch/powerpc/kvm/bookehv_interrupts.S b/arch/powerpc/kvm/bookehv_interrupts.S index 191c32b..be3de1d 100644 --- a/arch/powerpc/kvm/bookehv_interrupts.S +++ b/arch/powerpc/kvm/bookehv_interrupts.S @@ -470,7 +470,7 @@ _GLOBAL(kvmppc_resume_host) #ifdef CONFIG_64BIT /* * We enter with interrupts disabled in hardware, but - * we need to call RECONCILE_IRQ_STATE anyway to ensure + * we need to call RECONCILE_IRQ_STATE to ensure * that the software state is kept in sync. */ RECONCILE_IRQ_STATE(r3,r5) I've also taken the liberty of making the patch description be more verbose: KVM: PPC: Book3E HV: call RECONCILE_IRQ_STATE to sync the software state Rather than calling hard_irq_disable() when we're back in C code we can just call RECONCILE_IRQ_STATE to soft disable IRQs while we're already in hard disabled state. This should be functionally equivalent to the code before, but cleaner and faster. Signed-off-by: Tiejun Chen <tiejun.c...@windriver.com> [agraf: fix comment, commit message] Signed-off-by: Alexander Graf <ag...@suse.de> Thanks, applied to kvm-ppc-queue. Alex -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html