Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Richard Henderson <rth@twiddle.net>
To: Mark Kettenis <mark.kettenis@xs4all.nl>
Cc: drow@false.org, amodra@bigpond.net.au, gdb-patches@gcc.gnu.org,
	Richard Henderson <rth@twiddle.net>
Subject: Re: ppc32 debugging ppc64, part 1
Date: Mon, 12 Sep 2005 22:37:00 -0000	[thread overview]
Message-ID: <20050912223637.GA3587@twiddle.net> (raw)
In-Reply-To: <200509122119.j8CLJDNE020689@elgar.sibelius.xs4all.nl>

On Mon, Sep 12, 2005 at 11:19:13PM +0200, Mark Kettenis wrote:
> ...but in that case Richard is making things hopelessly complicated by
> doing using syscall() instead of ptrace().

I'm undecided as to whether it's cleaner or more complicated.  On
the one hand, I don't have to add odd casts and can just use a 
pointer as the real interface allows.

But for the record, I was seeing odd things and wanted to get the
libc code out of the way.  I later discovered the typo in my own
code, but I never got around to reverting the use of syscall.

> Fair enough.  But this code is really getting difficult to read.  It
> uses several constructs that really only made sense in the generic
> code where this was copied from.

Huh?  I removed the bulk of those.  What in particular did you have
in mind?

> What really frustrates me is that the different Linux ports are
> reinventing the wheel, all in a different way.

I appreciate your frustration, but it's not something that's in my
purvey to think about.  Ptrace is so hopelessly obtuse, slow, and
indirect that any effort fixing it would be better spent designing
a proper interface.  And in the meantime there's the issue of what's
actually present in extant kernels.


r~


  reply	other threads:[~2005-09-12 22:37 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-09-12 12:52 Richard Henderson
2005-09-12 19:47 ` Mark Kettenis
2005-09-12 20:32   ` Daniel Jacobowitz
2005-09-12 21:07     ` David S. Miller
2005-09-12 21:20     ` Mark Kettenis
2005-09-12 22:37       ` Richard Henderson [this message]
2005-09-12 23:20         ` Richard Henderson
2005-09-12 22:39   ` Richard Henderson
2005-09-17 21:24 ` Daniel Jacobowitz
2005-09-18  2:02   ` Richard Henderson
2005-09-18  2:14     ` 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=20050912223637.GA3587@twiddle.net \
    --to=rth@twiddle.net \
    --cc=amodra@bigpond.net.au \
    --cc=drow@false.org \
    --cc=gdb-patches@gcc.gnu.org \
    --cc=mark.kettenis@xs4all.nl \
    /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