Re: [Kgdb-bugreport] [KGDB PATCH][7/7] Move debugger_entry()

[George Anzinger <george@mvista.com>]

Tom Rini wrote:
> Hello.  When we use kgdboe, we can't use it until do_basic_setup() is done.
> So we have two options, not allow kgdboe to use the initial breakpoint
> or move debugger_entry() to be past the point where kgdboe will be usable.
> I've opted for the latter, as if an earlier breakpoint is needed you can
> still use serial and throw kgdb_schedule_breakpoint/breakpoint where desired.

Please don't do this.  How about configuring it along with the connection 
method.  I really don't want to have to modify the kernel just to get in early.

George
> 
> --- linux-2.6.3-rc4/init/main.c	2004-02-17 09:51:19.000000000 -0700
> +++ linux-2.6.3-rc4-kgdb/init/main.c	2004-02-17 11:33:51.854388988 -0700
> @@ -581,6 +582,7 @@ static int init(void * unused)
>  
>  	smp_init();
>  	do_basic_setup();
> +	debugger_entry();
>  
>  	prepare_namespace();
>  
> 

-- 
George Anzinger   george@mvista.com
High-res-timers:  http://sourceforge.net/projects/high-res-timers/
Preemption patch: http://www.kernel.org/pub/linux/kernel/people/rml