Hi Santosh,
On Mon, Mar 18, 2013 at 06:51:30AM +0000, Santosh Shilimkar wrote:
> On Friday 15 March 2013 10:30 AM, Will Deacon wrote:
> > Furthermore, I was under the impression that hw_breakpoint did actually
> > work on panda, which implies that a cold boot *does* manage to reset the
> > registers (can you please confirm this by looking in your dmesg during
> > boot?). In that case, it seems as though a PM cycle is powering down a
> > bunch of debug logic that was powered up during boot, and then we trip over
> > because we can't access the register bank.
> >
> Actually it seems to be without PM. Thanks to analysis from Lokesh, the issue
> can be seen even with just suspend or cpu hotplug. So cold boot as such is
> fine.
Right, so what you're saying is that monitor-mode hardware debug only works
until the first pm/suspend/hotplug operation, then it's dead in the water?
> > The proper solution to this problem requires us to establish exactly what is
> > turning off the debug registers, and then having an OMAP PM notifier to
> > enable it again. Assuming this has always been the case, I expect hardware
> > debug across PM fails silently with older kernels.
> >
> This has been always the case it seems with CPU power cycle.
> After the CPU is power cycled, 'DBGAUTHSTATUS' reads '0xaa' rather
> than '0xaf' which means 'DBGEN = 0' and hence code fails to enable
> monitor mode. This happens on both secure and GP devices and it can not
> be patched since the secure code is ROM'ed. We didn't notice so far
> because hw_breakpoint support was not default enabled on OMAP till the
> multi-platform build.
That really sucks :( Does this affect all OMAP-based boards?
> >> I was also wondering whether we should just warn once rather
> >> than continuous warnings in the notifier. Patch is end of the
> >> email.
> >
> > Could do, but I'd like to see a fix for the real issue before we simply hide
> > the warnings :)
> >
> Agree here too. As evident above, the feature won't work on OMAP4
> devices with PM and hence some solution is needed.
>
> What you think of below ?
Comments inline...
>
> From d74b4264f6a5967b0f7ada96aad77ab0ac30dbed Mon Sep 17 00:00:00 2001
> From: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
> Date: Mon, 18 Mar 2013 11:59:04 +0530
> Subject: [PATCH] ARM: hw_breakpoints: Check for CPU debug availability before
> enabling it
>
> CPU debug features like hardware break, watchpoints can be used only when
> the debug mode is enabled and available for non-secure mode.
>
> Hence check 'DBGAUTHSTATUS.DBGEN' before proceeding to enable the
> features.
>
> Thanks to Will for pointers and Lokesh for the analysis of the issue on
> OMAP4 where after a CPU power cycle, debug mode gets disabled.
>
> Cc: Will Deacon <Will.Deacon@xxxxxxx>
>
> Tested-by: Lokesh Vutla <lokeshvutla@xxxxxx>
> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
> ---
> arch/arm/kernel/hw_breakpoint.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/arch/arm/kernel/hw_breakpoint.c b/arch/arm/kernel/hw_breakpoint.c
> index 96093b7..683a7cf 100644
> --- a/arch/arm/kernel/hw_breakpoint.c
> +++ b/arch/arm/kernel/hw_breakpoint.c
> @@ -930,6 +930,14 @@ static void reset_ctrl_regs(void *unused)
> int i, raw_num_brps, err = 0, cpu = smp_processor_id();
> u32 val;
>
> + /* Check if we have access to CPU debug features */
> + ARM_DBG_READ(c7, c14, 6, val);
> + if ((val & 0x1) == 0) {
> + pr_warn_once("CPU %d debug is unavailable\n", cpu);
> + cpumask_or(&debug_err_mask, &debug_err_mask, cpumask_of(cpu));
> + return;
> + }
There are a few of problems here:
1. That is only checking for non-secure access, which precludes
running Linux in secure mode.
2. DBGAUTHSTATUS accesses are UNPREDICTABLE when the double-lock is
set for v7.1 processors.
3. DBGAUTHSTATUS doesn't exist in ARMv6.
4. CPUs without the security extensions have DBGAUTHSTATUS.NSE == 0
5. Accessing DBGAUTHSTATUS requires DBGEN to be driven high.
Assuming that your pr_warn_once is emitted, this implies that
DBGEN is driven high from cold boot, yet the NSE bit is low,
implying that DBGEN is also low. That's contradictory, so I have
no idea what's going on...
Apart from that, it's fine!
Will
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
