From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756056AbdC2LVg (ORCPT ); Wed, 29 Mar 2017 07:21:36 -0400 Received: from foss.arm.com ([217.140.101.70]:60318 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932289AbdC2LUx (ORCPT ); Wed, 29 Mar 2017 07:20:53 -0400 Date: Wed, 29 Mar 2017 12:20:31 +0100 From: Mark Rutland To: Doug Berger Cc: robh+dt@kernel.org, catalin.marinas@arm.com, will.deacon@arm.com, computersforpeace@gmail.com, gregory.0xf0@gmail.com, f.fainelli@gmail.com, bcm-kernel-feedback-list@broadcom.com, wangkefeng.wang@huawei.com, james.morse@arm.com, mingo@kernel.org, sandeepa.s.prabhu@gmail.com, shijie.huang@arm.com, linus.walleij@linaro.org, treding@nvidia.com, jonathanh@nvidia.com, olof@lixom.net, mirza.krak@gmail.com, suzuki.poulose@arm.com, bgolaszewski@baylibre.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 6/8] bus: brcmstb_gisb: Add ARM64 support Message-ID: <20170329112030.GD23442@leverpostej> References: <20170328213431.10904-1-opendmb@gmail.com> <20170328213431.10904-7-opendmb@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170328213431.10904-7-opendmb@gmail.com> 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 Hi, On Tue, Mar 28, 2017 at 02:34:29PM -0700, Doug Berger wrote: > From: Florian Fainelli > > Hook to the ARM64 data abort exception #16: synchronous external > abort, which is how the GISB errors will be funneled back to the > ARM64 CPU in case of problems I believe that you can use a die notifier for this, and that you don't need to hook the low-level architectural fault here. I note that the code doesn't even look at the faulting address, and likely doesn't have the information to do anything with it, so it make no sense to me for this code to hook the low-level fault. Further, as an aside, from digging into how we handle unexpected faults it appears that we don't always treat some faults (e.g. TLB conflict, faults on PTW) sufficiently fatally when they occur at EL0, so we likely need to do so rework there. Thanks, Mark. > > Signed-off-by: Florian Fainelli > --- > drivers/bus/Kconfig | 2 +- > drivers/bus/brcmstb_gisb.c | 15 ++++++++++++--- > 2 files changed, 13 insertions(+), 4 deletions(-) > > diff --git a/drivers/bus/Kconfig b/drivers/bus/Kconfig > index 0a52da439abf..d2a5f1184022 100644 > --- a/drivers/bus/Kconfig > +++ b/drivers/bus/Kconfig > @@ -57,7 +57,7 @@ config ARM_CCN > > config BRCMSTB_GISB_ARB > bool "Broadcom STB GISB bus arbiter" > - depends on ARM || MIPS > + depends on ARM || ARM64 || MIPS > default ARCH_BRCMSTB || BMIPS_GENERIC > help > Driver for the Broadcom Set Top Box System-on-a-chip internal bus > diff --git a/drivers/bus/brcmstb_gisb.c b/drivers/bus/brcmstb_gisb.c > index edf79432f899..500b6bb5c739 100644 > --- a/drivers/bus/brcmstb_gisb.c > +++ b/drivers/bus/brcmstb_gisb.c > @@ -30,6 +30,11 @@ > #include > #endif > > +#ifdef CONFIG_ARM64 > +#include > +#include > +#endif > + > #ifdef CONFIG_MIPS > #include > #endif > @@ -225,7 +230,7 @@ static int brcmstb_gisb_arb_decode_addr(struct brcmstb_gisb_arb_device *gdev, > return 0; > } > > -#ifdef CONFIG_ARM > +#if defined(CONFIG_ARM) || defined(CONFIG_ARM64) > static int brcmstb_bus_error_handler(unsigned long addr, unsigned int fsr, > struct pt_regs *regs) > { > @@ -235,7 +240,7 @@ static int brcmstb_bus_error_handler(unsigned long addr, unsigned int fsr, > list_for_each_entry(gdev, &brcmstb_gisb_arb_device_list, next) > brcmstb_gisb_arb_decode_addr(gdev, "bus error"); > > -#if !defined(CONFIG_ARM_LPAE) > +#if defined(CONFIG_ARM) && !defined(CONFIG_ARM_LPAE) > /* > * If it was an imprecise abort, then we need to correct the > * return address to be _after_ the instruction. > @@ -247,7 +252,7 @@ static int brcmstb_bus_error_handler(unsigned long addr, unsigned int fsr, > /* Always report unhandled exception */ > return 1; > } > -#endif > +#endif /* CONFIG_ARM || CONFIG_ARM64 */ > > #ifdef CONFIG_MIPS > static int brcmstb_bus_error_handler(struct pt_regs *regs, int is_fixup) > @@ -395,6 +400,10 @@ static int __init brcmstb_gisb_arb_probe(struct platform_device *pdev) > "imprecise external abort"); > #endif > #endif /* CONFIG_ARM */ > +#ifdef CONFIG_ARM64 > + hook_fault_code(16, brcmstb_bus_error_handler, SIGBUS, 0, > + "synchronous external abort"); > +#endif > #ifdef CONFIG_MIPS > board_be_handler = brcmstb_bus_error_handler; > #endif > -- > 2.12.0 >