linux-kernel - Re: [PATCH 22/40] mips,kgdb: kdb low level trap catch and stack trace

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4B4F548B.7090405@caviumnetworks.com>
Date:	Thu, 14 Jan 2010 09:29:47 -0800
From:	David Daney <ddaney@...iumnetworks.com>
To:	Jason Wessel <jason.wessel@...driver.com>
CC:	linux-kernel@...r.kernel.org, kgdb-bugreport@...ts.sourceforge.net,
	mingo@...e.hu, Ralf Baechle <ralf@...ux-mips.org>
Subject: Re: [PATCH 22/40] mips,kgdb: kdb low level trap catch and stack trace

Jason Wessel wrote:
> The only way the debugger can handle a trap in inside rcu_lock,
> notify_die, or atomic_notifier_call_chain without a recursive fault is
> to have a low level "first opportunity handler" do_trap_or_bp() handler.
> 
> Generally this will be something the vast majority of folks will not
> need, but for those who need it, it is added as a kernel .config
> option called KGDB_LOW_LEVEL_TRAP.
> 
> Also added was a die notification for oops such that kdb can catch an
> oops for analysis.
> 
> There appeared to be no obvious way to pass the struct pt_regs from
> the original exception back to the stack back tracer, so a special
> case was added to show_stack() for when kdb is active because you
> generally desire to generally look at the back trace of the original
> exception.
> 
> CC: Ralf Baechle <ralf@...ux-mips.org>
> Signed-off-by: Jason Wessel <jason.wessel@...driver.com>
> ---
>  arch/mips/include/asm/kgdb.h |    2 ++
>  arch/mips/kernel/kgdb.c      |   22 +++++++++++++++++++++-
>  arch/mips/kernel/traps.c     |   14 ++++++++++++++
>  lib/Kconfig.kgdb             |    2 +-
[...]
> +#include "../../../kernel/debug/kdb/kdb_private.h"
>  

There must be a better way of referring to this include file.  Perhaps 
it should be moved to a more central place that is already in the 
include path so we can refer to it with #include <.../file.h>

David Daney
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/