linux-kernel - Re: [RFC PATCH] KGDB: various refactorings

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: <479CD5BA.80009@web.de>
Date:	Sun, 27 Jan 2008 20:04:26 +0100
From:	Jan Kiszka <jan.kiszka@....de>
To:	Jason Wessel <jason.wessel@...driver.com>
CC:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	kgdb-bugreport@...ts.sourceforge.net, Ingo Molnar <mingo@...e.hu>
Subject: Re: [RFC PATCH] KGDB: various refactorings

Jan Kiszka wrote:
> Jason,
> 
> this is a RFC about another round of refactoring for the KGDB version
> in x86-git. It's a single patch so far, but I can break out individual
> hunks on request.
> 
> Things I changed:
>  - Reduce EXPORT_SYMBOLs, switch the rest to EXPORT_SYMBOL_GPL
>  - Drop code that is (yet) unused
>  - Rename variables, refactor their types (personal taste, for sure)
>  - Remove "extern" for function prototypes
>  - Remove MODULE_LICENSE/DESCRIPTION from kgdb core

Sorry, forgot to refresh for this hunk:

 - rename CHECK_EXCEPTION_STACK to EXCEPTION_STACK_READY

Signed-off-by: Jan Kiszka <jan.kiszka@....de>

---
 include/asm-x86/kgdb.h |    2 +-
 include/linux/kgdb.h   |    4 ++--
 kernel/kgdb.c          |    2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

Index: b/include/asm-x86/kgdb.h
===================================================================
--- a/include/asm-x86/kgdb.h
+++ b/include/asm-x86/kgdb.h
@@ -77,7 +77,7 @@ enum regnames { _AX,	/* 0 */
 #ifndef __ASSEMBLY__
 #define BREAKPOINT()		asm("   int $3");
 #ifndef CONFIG_X86_32
-#define CHECK_EXCEPTION_STACK()
((&__get_cpu_var(init_tss))[0].x86_tss.ist[0])
+#define EXCEPTION_STACK_READY()
((&__get_cpu_var(init_tss))[0].x86_tss.ist[0])
 #endif /* ! CONFIG_X86_32 */
 #define BREAK_INSTR_SIZE	1
 #define CACHE_FLUSH_IS_SAFE	1
Index: b/include/linux/kgdb.h
===================================================================
--- a/include/linux/kgdb.h
+++ b/include/linux/kgdb.h
@@ -23,8 +23,8 @@
 #include <linux/linkage.h>
 #include <linux/init.h>

-#ifndef CHECK_EXCEPTION_STACK
-#define CHECK_EXCEPTION_STACK()	1
+#ifndef EXCEPTION_STACK_READY
+#define EXCEPTION_STACK_READY()	1
 #endif

 struct tasklet_struct;
Index: b/kernel/kgdb.c
===================================================================
--- a/kernel/kgdb.c
+++ b/kernel/kgdb.c
@@ -1940,7 +1940,7 @@ static void __init kgdb_early_entry(void
 	 * architecture to hook in and look at us when they are ready.
 	 */

-	if (!CHECK_EXCEPTION_STACK()) {
+	if (!EXCEPTION_STACK_READY()) {
 		kgdb_state = KGDB_SEMI_INITIALIZED;
 		/* any kind of break point is deferred to late_init */
 		return;


Download attachment "signature.asc" of type "application/pgp-signature" (255 bytes)