lkml.org 
[lkml]   [2004]   [Feb]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH][1/6] A different KGDB stub
On Tue, Feb 17, 2004 at 03:50:36PM -0800, Andrew Morton wrote:

> Tom Rini <trini@kernel.crashing.org> wrote:
> >
> > The following is the core bits to this KGDB stub.
>
> This still contains the kern_do_schedule() gunk. Andi raised this issue
> last week. He identified several other significant issues as well, but
> there was no followup. Could you please dig out his email and address the
> points which he raised? (I can't find the email - perhaps Andi could
> re-review this patch?)

By my read of Andi's email, the kern_do_schedule() gunk is "I really
don't like this change. It is completely useless because you can get the
pt_regs as well from the stack. Please don't add it. George's stub also
didn't need it."

But I don't see how it does. But I'll look again tomorrow.

The next issue was about adding debuggerinfo to thread_struct. By my
read of the code, it's because of the thread handling bits that Amit's
version does that George's does not. So I'm not sure how it's not
needed (unless all of the relevant code goes. If that's too heavy, I
can remove all of that).

Next was that KGDB should use the notify_die hooks that are there, and
I've done that.

Finally, the save_context_frame stuff can go (x86_64-specific stuffs)
but as I don't have x86_64 hw (I'll try and whip up a toolchain
tomorrow, but I leave for FOSDEM Thursday) so I didn't touch that, in
hopes that someone who could test it would.

--
Tom Rini
http://gate.crashing.org/~trini/
-
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:01    [W:1.627 / U:0.180 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site