Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752586AbaBJOnQ (ORCPT ); Mon, 10 Feb 2014 09:43:16 -0500 Received: from fw-tnat.cambridge.arm.com ([217.140.96.21]:51390 "EHLO cam-smtp0.cambridge.arm.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752098AbaBJOnI (ORCPT ); Mon, 10 Feb 2014 09:43:08 -0500 Date: Mon, 10 Feb 2014 14:42:28 +0000 From: Dave Martin To: Russell King - ARM Linux Cc: Will Deacon , Jonathan Austin , "nico@linaro.org" , Marc Zyngier , Catalin Marinas , "sboyd@codeaurora.org" , "linux-kernel@vger.kernel.org" , "vgupta@synopsys.com" , "ben.dooks@codethink.co.uk" , "u.kleine-koenig@pengutronix.de" , Fabrice Gasnier , "linux-arm-kernel@lists.infradead.org" , "maxime.coquelin@st.com" Subject: Re: [RFC PATCH] ARM: Add imprecise abort enable/disable macro Message-ID: <20140210144228.GC2794@e103592.cambridge.arm.com> References: <1391789955-26927-1-git-send-email-fabrice.gasnier@st.com> <1391789955-26927-2-git-send-email-fabrice.gasnier@st.com> <20140207170903.GT5976@mudshark.cambridge.arm.com> <52F892C8.80508@st.com> <20140210111710.GC17766@mudshark.cambridge.arm.com> <20140210135659.GU26684@n2100.arm.linux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140210135659.GU26684@n2100.arm.linux.org.uk> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 10, 2014 at 01:56:59PM +0000, Russell King - ARM Linux wrote: > On Mon, Feb 10, 2014 at 11:17:10AM +0000, Will Deacon wrote: > > On Mon, Feb 10, 2014 at 08:50:16AM +0000, Fabrice Gasnier wrote: > > > On 02/07/2014 06:09 PM, Will Deacon wrote: > > > > On Fri, Feb 07, 2014 at 04:19:15PM +0000, Fabrice GASNIER wrote: > > > >> diff --git a/arch/arm/kernel/traps.c b/arch/arm/kernel/traps.c > > > >> index 4636d56..ef15709 100644 > > > >> --- a/arch/arm/kernel/traps.c > > > >> +++ b/arch/arm/kernel/traps.c > > > >> @@ -900,6 +900,10 @@ void __init early_trap_init(void *vectors_base) > > > >> > > > >> flush_icache_range(vectors, vectors + PAGE_SIZE * 2); > > > >> modify_domain(DOMAIN_USER, DOMAIN_CLIENT); > > > >> + > > > >> + /* Enable imprecise aborts */ > > > >> + local_abt_enable(); > > > > Surely we want to enable this as early as possible? Now, putting this into > > > > head.S is ugly, as it duplicating it across all the proc*.S files, so why > > > > not setup_arch? > > > Sorry, I'm not sure to understand your last comment. > > > At least, I need it enabled before probing drivers (PCIe bus) > > > I've added imprecise abort enable code in traps.c, following Russel > > > King's advice, please see: > > > http://archive.arm.linux.org.uk/lurker/message/20140131.170827.d752a1cc.en.html > > > As abort bit is local to a cpu, i've also added it in smp.c, but this > > > may not be the right place ? > > > > > > Please elaborate, > > > > I was just suggesting that we move your local_abt_enable() call to > > setup_arch, since that's called before early_trap_init on the primary CPU. > > Why would we want to enable aborts before we've setup the vectors page > to handle an abort? That's akin to enabling interrupts and hoping there > isn't one pending... Should we require CPSR.A to me masked in Booting, for all CPUs that have it? By definition, we cannot dispatch those exceptions for a while, until some vectors have been set up; so it makes sense in the same way that requiring CPSR.[IF] to be set makes sense. The kernel should do its best to cope anyway, and immediately mask CPSR.A on entry if it's a v6 or later kernel. safe_svcmode_maskall was originally intended to do that, but it got watered down to cope with the Zaurus issues, so it now looks like most code paths don't mask CPSR.A there. I'm happy to take another look at it. Cheers ---Dave -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/