lkml.org 
[lkml]   [2004]   [Jan]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: PPC KGDB changes and some help?
Tom Rini wrote:
>
>
> FWIW, this is true of KGDB on all PPCs. IIRC, so long as the serial
> definitions are filled out statically, the stub currently in kernel.org
> for PPC can do first-line-of-C already.
>
>
>>How about changing the code in kgdbstub to allow kgdb to be configured in one
>>of the following ways:
>>Late kgdb - kgdb comes up after smp_init in the kernel boot sequence. kgdb8250
>>can be used with more flexibility through kernel command line options. One
>>can boot a kgdb kernel without activating kgdb. Works with the interface
>>chosen by kernel command line (kgdb8250 and kgdbeth for the moment).
>
A further thought on this. I think kgdb should take control on oops, panic and
other bad news things. This without being anthing but configured in. Thus the
command line options to set up the interface, etc, should not automatically
connect to gdb.
>
> Which is basically how it goes now, right?
>
>
>>Early kgdb - kgdb comes up right at the begining of start_kernel at the cost
>>of flexibility. It doesn't show any messages such as "waiting for gdb".

I question the utility of ever using this message. Either the user has asked
for the connection, either via a ^C or the command line or a breakpoint was hit.
If he asked, why tell him? If it is a breakpoint, it is foolish and dangerous
to call any outside function, such as printk.

--
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

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

\
 
 \ /
  Last update: 2005-03-22 14:00    [from the cache]
©2003-2011 Jasper Spaans