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

Reply via email to