Re: kgdb for mainline kernel: core-lite [patch 1/3]

Amit S. Kale
2004-03-09T04:29:43Z

On Monday 08 Mar 2004 10:48 pm, Tom Rini wrote:
> On Mon, Mar 08, 2004 at 05:57:26PM +0100, Andi Kleen wrote:
> > Tom Rini <trini@kernel.crashing.org> writes:
> > > Here's where what Andi said about being able to get the pt_regs stuff
> > > off the stack (I think that's what he said at least) started to confuse
> > > me slightly. But if I understand it right (and I never got around to
> > > testing this) we can replace the do_schedule() stuffs at least with
> > > something like kgdb_get_pt_regs(), since (and I lost my notes on this,
> > > so it's probably not quite right) (thread_info->esp0)-1
> >
> > No, that's the user space registers.
> >
> > You don't need these registers really as long as you have the
> > correct dwarf2 CFI description of all the code involved. gdb
> > is then able to reconstruct them using the C stack by itself.
> >
> > All it needs for that is esp and eip.
>
> Ah, OK. Amit, how about for the expanded T-packet thing, we do what you
> suggested, except call it kgdb_arch_extra_regs() and let the arch fill
> out whatever regs it needs to. ESP/EIP for x86_64/i386, PC/SP for
> ppc32, etc, etc?

That'll be great, though it won't provide any more info to gdb, as gdb can get
that info by 'g' packet.

Thread listing problem is with 'g' packet returning incomplete information for
threads other than the current thread.

-Amit