Re: [Kgdb-bugreport] [KGDB PATCH][1/7] Add / use kernel/Kconfig.kgdb

George Anzinger
2004-03-02T21:38:47Z

Amit S. Kale wrote:
> On Saturday 28 Feb 2004 6:38 am, George Anzinger wrote:
>
>>Pavel Machek wrote:
>>
>>>Hi!
>>>
>>>
>>>>>+config KGDB_THREAD
>>>>>+ bool "KGDB: Thread analysis"
>>>>>+ depends on KGDB
>>>>>+ help
>>>>>+ With thread analysis enabled, gdb can talk to kgdb stub to list
>>>>>+ threads and to get stack trace for a thread. This option also
>>>>>enables
>>>>>+ some code which helps gdb get exact status of thread. Thread
>>>>>analysis
>>>>>+ adds some overhead to schedule and down functions. You can disable
>>>>>+ this option if you do not want to compromise on speed.
>>>>
>>>>Lets remove the overhead and eliminate the need for this option in favor
>>>>of always having threads. Works in the mm kgdb...
>>>
>>>No. Thread analysis is unsuitable for the mainline (manipulates
>>>sched.c in ugly way). It may be okay for -mm, but in such case it
>>>should better be separated.
>>
>>Not in the -mm version. I agree that sched.c should NEVER be treated this
>>way and it is not in the -mm version. I also think that, most of the time,
>>it is useful to have the thread stuff, but that may be just my usage...
>
>
> If threads stuff didn't introduce any unclean code changes, I too would prefer
> to have it on all the time. As things stands, threads stuff is rather
> intrusive.

Lets put the threads stuff in the stub. The only stuff we need in the kernel is
the flag that indicateds that the pid hash table has been initialized.

Meanwhile, I would like to make a change to the gdb "info thread" command to do
a better job of displaying the threads. Here is what I am proposing:

Gdb would work as it does now if the following set is not done.

A new "set thread_level" command that would take the "bt" level to use on the
thread display.
A new "set thread_limits command that would take two expressions that would
reduce to two memory addresses.

Which ever of these is entered last will be active and used by "info thread" as
follows:

if thread_level is active gdb will do the indicated number of "up" operations
and display the result on the info thread line for that thread (note there is
other info on this line that will not be changed).

if thread_limits is active gdb will do 0 or more "up" commands until the
resultant PC is NOT between the given limits.

The kernel, at this time, has defined symbols for the thread_limits command (it
is used in the kernel for its internal display of threads). I would expect that
the thread_level version would be the answer for theaded application programs.

Daniel, how does this sound?

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