Re: [patch 1/8] A different KGDB stub for -mm

[Tom Rini <trini@kernel.crashing.org>]

> diff -puN kernel/sched.c~core-lite kernel/sched.c
> --- linux-2.6.10-rc1/kernel/sched.c~core-lite	2004-10-29 11:26:42.888818087 -0700
> +++ linux-2.6.10-rc1-trini/kernel/sched.c	2004-10-29 11:26:42.904814331 -0700
[snip]
> @@ -4567,6 +4568,12 @@ void __might_sleep(char *file, int line)
>  #if defined(in_atomic)
>  	static unsigned long prev_jiffy;	/* ratelimiting */
>  
> +	if (atomic_read(&debugger_active))
> +		return;
> +
> +	if (atomic_read(&debugger_active))
> +		return;
> +
>  	if ((in_atomic() || irqs_disabled()) &&
>  	    system_state == SYSTEM_RUNNING) {
>  		if (time_before(jiffies, prev_jiffy + HZ) && prev_jiffy)

Olaf Hering pointed this out to me privately.  Obvious patch follows.

(And, er, I fogot this on all of the other patches too, as I said, not a
good start to the day...)

Signed-off-by: Tom Rini <trini@kernel.crashing.org>

diff -u linux-2.6.10-rc1/kernel/sched.c linux-2.6.10-rc1/kernel/sched.c
--- linux-2.6.10-rc1/kernel/sched.c
+++ linux-2.6.10-rc1/kernel/sched.c
@@ -4571,9 +4571,6 @@
 	if (atomic_read(&debugger_active))
 		return;
 
-	if (atomic_read(&debugger_active))
-		return;
-
 	if ((in_atomic() || irqs_disabled()) &&
 	    system_state == SYSTEM_RUNNING) {
 		if (time_before(jiffies, prev_jiffy + HZ) && prev_jiffy)

-- 
Tom Rini
http://gate.crashing.org/~trini/