From mboxrd@z Thu Jan  1 00:00:00 1970
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: Tue, 07 May 2013 20:23:24 +0000
Subject: Re: [PATCH] kvm/ppc: interrupt disabling fixes
Message-Id: <1367958204.25488.14.camel@pasglop>
List-Id: <kvm-ppc.vger.kernel.org>
References: <1367953139.3398.28@snotra>
In-Reply-To: <1367953139.3398.28@snotra>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: Scott Wood <scottwood@freescale.com>
Cc: "tiejun.chen" <tiejun.chen@windriver.com>, Alexander Graf <agraf@suse.de>, kvm-ppc@vger.kernel.org, kvm@vger.kernel.org, Mihai Caraman <mihai.caraman@freescale.com>, Paul Mackerras <paulus@au1.ibm.com>

On Tue, 2013-05-07 at 13:58 -0500, Scott Wood wrote:
> This will have to wait until book3s_hv disables interrupts as well.  If  
> this does eventually happen, then the  
> local_irq_enable()/kvmppc_lazy_ee_enable() should also probably go into  
> kvmppc_prepare_to_enter() -- though that could cause problems in  
> book3s_pr, if it's depending on the kvmppc_lazy_ee_enable() happening  
> as late as it does.

Is book3s calling prepare_to_enter at all ? It has its own things with
no interrupt disabling which afaik is racy vs. checking for signals & resched...

(CC'ing Paul).

BTW. Linus just pulled my tree which contains my changed to hard_irq_disable()
to do the trace_hardirqs_off() when neeed.

Cheers,
Ben.