Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-18 Thread Murali Karicheri
Russell, On 08/18/2015 04:13 AM, Russell King - ARM Linux wrote: On Mon, Aug 17, 2015 at 08:09:17PM -0700, santosh.shilim...@oracle.com wrote: From the logs this seems to be mostly clock related issue for some peripheral. If the bootloader clock enable all hack still exists, may be you can try

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-18 Thread Murali Karicheri
On 08/18/2015 04:28 AM, Jisheng Zhang wrote: On Tue, 18 Aug 2015 09:13:34 +0100 Russell King - ARM Linux wrote: On Mon, Aug 17, 2015 at 08:09:17PM -0700, santosh.shilim...@oracle.com wrote: From the logs this seems to be mostly clock related issue for some peripheral. If the bootloader clock

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-18 Thread Afzal Mohammed
Hi Murali, On Tue, Aug 18, 2015 at 10:28:20AM +0200, Lucas Stach wrote: > Am Dienstag, den 18.08.2015, 09:13 +0100 schrieb Russell King - ARM > Linux: > > It seems to be pointing towards something in the boot loader... > > > > Normally, uboot will hook itself into the vectors to report errors, b

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-18 Thread Jisheng Zhang
On Tue, 18 Aug 2015 09:13:34 +0100 Russell King - ARM Linux wrote: > On Mon, Aug 17, 2015 at 08:09:17PM -0700, santosh.shilim...@oracle.com wrote: > > From the logs this seems to be mostly clock related issue for some > > peripheral. If the bootloader clock enable all hack still exists, > > may b

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-18 Thread Lucas Stach
Am Dienstag, den 18.08.2015, 09:13 +0100 schrieb Russell King - ARM Linux: > On Mon, Aug 17, 2015 at 08:09:17PM -0700, santosh.shilim...@oracle.com wrote: > > From the logs this seems to be mostly clock related issue for some > > peripheral. If the bootloader clock enable all hack still exists, > >

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-18 Thread Russell King - ARM Linux
On Mon, Aug 17, 2015 at 08:09:17PM -0700, santosh.shilim...@oracle.com wrote: > From the logs this seems to be mostly clock related issue for some > peripheral. If the bootloader clock enable all hack still exists, > may be you can try that out. > > Another way to debug this is to start disabling

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-17 Thread santosh.shilim...@oracle.com
Murali, On 8/17/15 3:12 PM, Murali Karicheri wrote: On 08/14/2015 05:56 PM, Russell King - ARM Linux wrote: On Fri, Aug 14, 2015 at 05:53:00PM -0400, Murali Karicheri wrote: We have spend some time already to debug the root cause. Do you have idea on how this was hunted down on OMAP that we ca

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-17 Thread Russell King - ARM Linux
On Mon, Aug 17, 2015 at 06:12:52PM -0400, Murali Karicheri wrote: > Unfortunately, this patch causes boot to stop very early just after > local_abt_enable() is called in early_trap_init(). Before and After applying > the patch, here is what the boot log looks like. Do you see any issue with > the p

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-17 Thread Murali Karicheri
On 08/14/2015 05:56 PM, Russell King - ARM Linux wrote: On Fri, Aug 14, 2015 at 05:53:00PM -0400, Murali Karicheri wrote: We have spend some time already to debug the root cause. Do you have idea on how this was hunted down on OMAP that we can learn from? The bad address is NULL and it seems to

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-17 Thread Murali Karicheri
On 08/14/2015 10:11 AM, Lucas Stach wrote: Am Freitag, den 14.08.2015, 10:04 -0400 schrieb Murali Karicheri: On 08/11/2015 03:13 PM, Murali Karicheri wrote: Currently on some devices, an asynchronous external abort exception happens during boot up when exception handlers are enabled in kernel b

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-17 Thread Murali Karicheri
On 08/14/2015 05:56 PM, Russell King - ARM Linux wrote: On Fri, Aug 14, 2015 at 05:55:09PM -0400, Murali Karicheri wrote: On 08/14/2015 10:20 AM, Lucas Stach wrote: Hi Russell, Am Freitag, den 14.08.2015, 15:09 +0100 schrieb Russell King - ARM Linux: [...] What causes the abort? We should

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-14 Thread santosh.shilim...@oracle.com
On 8/14/15 2:53 PM, Murali Karicheri wrote: On 08/14/2015 11:14 AM, santosh shilimkar wrote: On 8/14/2015 7:09 AM, Russell King - ARM Linux wrote: On Fri, Aug 14, 2015 at 10:04:41AM -0400, Murali Karicheri wrote: On 08/11/2015 03:13 PM, Murali Karicheri wrote: Currently on some devices, an as

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-14 Thread Russell King - ARM Linux
On Fri, Aug 14, 2015 at 05:55:09PM -0400, Murali Karicheri wrote: > On 08/14/2015 10:20 AM, Lucas Stach wrote: > >Hi Russell, > > > >Am Freitag, den 14.08.2015, 15:09 +0100 schrieb Russell King - ARM > >Linux: > > > >[...] > > > >> > >>What causes the abort? We shouldn't be adding hacks like this

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-14 Thread Russell King - ARM Linux
On Fri, Aug 14, 2015 at 05:53:00PM -0400, Murali Karicheri wrote: > We have spend some time already to debug the root cause. Do you have idea on > how this was hunted down on OMAP that we can learn from? The bad address is > NULL and it seems to happen very rarely and is not easily reproducible. >

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-14 Thread Murali Karicheri
On 08/14/2015 10:20 AM, Lucas Stach wrote: Hi Russell, Am Freitag, den 14.08.2015, 15:09 +0100 schrieb Russell King - ARM Linux: [...] What causes the abort? We shouldn't be adding hacks like this to the kernel without having the full picture... some of the issues with tracking down such

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-14 Thread Murali Karicheri
On 08/14/2015 11:14 AM, santosh shilimkar wrote: On 8/14/2015 7:09 AM, Russell King - ARM Linux wrote: On Fri, Aug 14, 2015 at 10:04:41AM -0400, Murali Karicheri wrote: On 08/11/2015 03:13 PM, Murali Karicheri wrote: Currently on some devices, an asynchronous external abort exception happens d

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-14 Thread santosh shilimkar
On 8/14/2015 7:09 AM, Russell King - ARM Linux wrote: On Fri, Aug 14, 2015 at 10:04:41AM -0400, Murali Karicheri wrote: On 08/11/2015 03:13 PM, Murali Karicheri wrote: Currently on some devices, an asynchronous external abort exception happens during boot up when exception handlers are enabled

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-14 Thread Lucas Stach
Hi Russell, Am Freitag, den 14.08.2015, 15:09 +0100 schrieb Russell King - ARM Linux: [...] > > What causes the abort? We shouldn't be adding hacks like this to the > kernel without having the full picture... > some of the issues with tracking down such imprecise external aborts are due to t

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-14 Thread Lucas Stach
Am Freitag, den 14.08.2015, 10:04 -0400 schrieb Murali Karicheri: > On 08/11/2015 03:13 PM, Murali Karicheri wrote: > > Currently on some devices, an asynchronous external abort exception > > happens during boot up when exception handlers are enabled in kernel > > before switching to user space. Th

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-14 Thread Russell King - ARM Linux
On Fri, Aug 14, 2015 at 10:04:41AM -0400, Murali Karicheri wrote: > On 08/11/2015 03:13 PM, Murali Karicheri wrote: > >Currently on some devices, an asynchronous external abort exception > >happens during boot up when exception handlers are enabled in kernel > >before switching to user space. This

Re: [PATCH] ARM: keystone: add a work around to handle asynchronous external abort

2015-08-14 Thread Murali Karicheri
On 08/11/2015 03:13 PM, Murali Karicheri wrote: Currently on some devices, an asynchronous external abort exception happens during boot up when exception handlers are enabled in kernel before switching to user space. This patch adds a workaround to handle this once during boot. Many customers are