lkml.org 
[lkml]   [2009]   [Dec]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC] [PATCH] In-kernel gdbstub based on utrace Infrastructure.

* Frank Ch. Eigler <fche@redhat.com> wrote:

> Hi -
>
> > > Only in name. One is highly invasive, for debugging the kernel across
> > > serial consoles. The other is highly noninvasive, for debugging user
> > > processes across normal userspace channels. They both happen to talk
> > > to gdb, but that's the end of the natural "overlap".
> > [...]
>
> > Well nothing that you mention here changes our obvious suggestion
> > that an in-kernel gdb stub should obviously either be a kgdb
> > extension, or a replacement of it.
>
> Help me out here: by "kgdb extension" do you imagine "something new
> that an unprivileged user can use to debug his own process"? Or do
> you imagine a new userspace facility that single-steps the kernel?

Is this a trick question? Single-stepping the kernel on the same system
[especially if it's an UP system] would certainly be a challenge ;-)

What i mean is what i said: if you provide a new framework (especially
if it's user visible - which both kgdb and the gdb stub is) you should
either fully replace existing functionality or extend it. Overlapping it
in an incomplete way is not useful to anyone.

Extending kgdb to allow the use of it as if we used gdb locally would
certainly be interesting - and then you could drop into the kernel
anytime as well. But i'm not siding with any particular solution - i'm
just seconding Peter's point that there's very clear overlap and
inconsistency, and that ought to be resolved one way or another.

> > We dont want to separate facilities for the same conceptual thing:
> > examining application state (be that in user-space and
> > kernel-space).
>
> This seems like a shallow sort of consistency. kgdb was added after
> ptrace existed -- why not extend ptrace instead to target the kernel?
> After all, it's "examining application state". The answer is that it
> doesn't make a heck of a lot of sense.

kgdb simply used gdb's preferred way of remote debugging. That's
certainly the ugliest bit of it btw - but it's an externality to kgdb.

Had it extended ptrace it wouldnt have gdb compatibility.

So i think this example of yours is inapposite as well.

Having said all that, i certainly subscribe to the view that neither
kgdb nor ptrace is particularly cleanly done. So i wouldnt mind if
something new existed that had a modern, flexible, extensible and
generally pleasant interface and implementation. If you are heading in
that direction, please let me know.

> > > > Btw., perf does meet that definition: it functionally replaces all
> > > > facilities that it overlaps/extends - such as Oprofile. [...]
> > >
> > > (And they currently separately coexist.)
> >
> > You didnt get my point apparently. Keeping the overlapped facility for
> > compatibility (and general user inertia) is fine. Creating a new
> > facility that doesnt do everything that the existing facility does, and
> > not integrating it either, is not fine.
>
> oprofile and perfctr are closer in concept than kgdb and ptrace, yet
> AFAIK perfctr doesn't "interface" to oprofile, except perhaps to the
> extent of resolving contention over the underlying physical resources.
> In any case this is not a great analogy.

(FYI, 'perfctr' is a different project that has existed for years, i
suspect you meant perf events?)

perf replaces oprofile functionally. If the in-kernel gdb stub replaced
kgdb functionally you'd hear no complaints from me.

Ingo


\
 
 \ /
  Last update: 2009-12-01 22:17    [from the cache]
©2003-2011 Jasper Spaans