Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Mark Kettenis <mark.kettenis@xs4all.nl>
To: drow@false.org
Cc: mark.kettenis@xs4all.nl, gdb-patches@sourceware.org
Subject: Re: [commit] Unwinder for OpenBSD/sparc64 kernel trap frames
Date: Sun, 31 Dec 2006 12:36:00 -0000	[thread overview]
Message-ID: <200612311236.kBVCacgf020195@brahms.sibelius.xs4all.nl> (raw)
In-Reply-To: <20061231022259.GB719@nevyn.them.org> (message from Daniel 	Jacobowitz on Sat, 30 Dec 2006 21:22:59 -0500)

> Date: Sat, 30 Dec 2006 21:22:59 -0500
> From: Daniel Jacobowitz <drow@false.org>
> 
> On Sun, Dec 31, 2006 at 02:28:13AM +0100, Mark Kettenis wrote:
> > +  find_pc_partial_function (frame_pc_unwind (next_frame), &name, NULL, NULL);
> > +  if (name && strcmp (name, "Lslowtrap_reenter") == 0)
> > +    return &sparc64obsd_trapframe_unwind;
> 
> Does this really belong in GDB?  I ask because we've resisted adding
> similar things for other targets before - at least I have for Linux
> and for Xfree86, but maybe I should have been more accepting.

Heh, I believe you raised the same question when I added the same code
to i386 and amd64.  At that time I believe I convinced you that this
was acceptable.  This is not very invasive and nicely contained in
*BSD-specific files.  And the layout of kernel trapframes is pretty
stable.

GDB has been shipping with *BSD kernel debugging support for ages,
although it probably was almost completely broken until I cleaned
things up and replaced it with the kvm target a while ago.

I have looked into adding the XFree86 support too, but that was much
more invasive (because weirdness in their "dynamic linker"
implementation and problems with our shared library framework).  And
these days on OpenBSD, XOrg uses the normal ld.so to load modules.

Adding Linux kernel debugging support is much harder.  The rate of
change is much higher, there are kernel modules (which have a slightly
odd format).  And of course Linux doesn't have kernel core dumps, so
it would only be useful to debug a live kernel.

> Anyway, I have some thoughts on how to extend the unwind mechanism so
> that the OpenBSD kernel could ship a script that knew how to unwind its
> trap frames.

Did you really mean to say OpenBSD here?

Mark


  reply	other threads:[~2006-12-31 12:36 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-31  1:28 Mark Kettenis
2006-12-31  2:23 ` Daniel Jacobowitz
2006-12-31 12:36   ` Mark Kettenis [this message]
2006-12-31 15:05     ` Daniel Jacobowitz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200612311236.kBVCacgf020195@brahms.sibelius.xs4all.nl \
    --to=mark.kettenis@xs4all.nl \
    --cc=drow@false.org \
    --cc=gdb-patches@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox