From: Joel Brobecker <brobecker@adacore.com>
To: gdb-patches@sourceware.org
Subject: Re: [RFC] Support exec tracing on GNU/Linux and HP-UX
Date: Tue, 29 Jan 2008 21:26:00 -0000 [thread overview]
Message-ID: <20080129211657.GC16288@adacore.com> (raw)
In-Reply-To: <20080129170835.GC2815@caradoc.them.org>
Hi Daniel,
First of all, thanks very much for the mega-load of reviews you did
today. I was hoping for a more positive result, of course :-), but
it's interesting to get your opinion and we can start some discussions.
> > 2007-12-21 Joel Brobecker <brobecker@adacore.com>
> >
> > * infrun.c (wait_for_inferior): Add treat_exec_as_sigtrap parameter
> > and use it.
> > (proceed, start_remote): Update call to wait_for_inferior.
> > * inferior.h (wait_for_inferior): Update declaration.
> > * fork-child.c, infcmd.c, solib-irix.c, solib-osf.c, solib-sunos.c,
> > solib-svr4.c, win32-nat.c: Update calls to wait_for_inferior.
> > * inf-ttrace.c (inf_ttrace_wait): Report TTEVT_EXEC events as
> > TARGET_WAITKIND_EXECD instead of TARGET_WAITKIND_STOPPED.
> >
> > Tested on hppa-hpux, no regression.
> This looks good to me - would you add that comment and check it in?
Absolutely. Here is the comment that I added:
If TREAT_EXEC_AS_SIGTRAP is non-zero, then handle EXEC signals
as if they were SIGTRAP signals. This can be useful during
the startup sequence on some targets such as HP/UX, where
we receive an EXEC event instead of the expected SIGTRAP.
Hope this is clear enough.
Do you want to push your patch for the 6.8 release or would you
rather wait for the next one?
--
Joel
next prev parent reply other threads:[~2008-01-29 21:17 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-19 19:49 Daniel Jacobowitz
2007-10-22 6:01 ` Joel Brobecker
2007-10-22 12:09 ` Daniel Jacobowitz
2007-10-22 18:56 ` Joel Brobecker
2007-10-22 19:30 ` Joel Brobecker
2007-10-22 20:01 ` Daniel Jacobowitz
2007-12-21 16:04 ` Joel Brobecker
2007-12-21 16:13 ` Joel Brobecker
2008-01-29 17:09 ` Daniel Jacobowitz
2008-01-29 21:26 ` Joel Brobecker [this message]
2008-01-29 21:48 ` Daniel Jacobowitz
2008-01-29 21:48 ` Joel Brobecker
2008-01-29 22:17 ` Daniel Jacobowitz
2008-01-29 22:54 ` 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=20080129211657.GC16288@adacore.com \
--to=brobecker@adacore.com \
--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