This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC] Support exec tracing on GNU/Linux and HP-UX
- From: Daniel Jacobowitz <drow at false dot org>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 29 Jan 2008 12:08:35 -0500
- Subject: Re: [RFC] Support exec tracing on GNU/Linux and HP-UX
- References: <20071019175920.GA548@caradoc.them.org> <20071022043831.GD764@adacore.com> <20071022114328.GA1421@caradoc.them.org> <20071022184702.GG764@adacore.com> <20071022185627.GH764@adacore.com> <20071022193024.GA16312@caradoc.them.org> <20071221153039.GO6154@adacore.com>
On Fri, Dec 21, 2007 at 07:30:39PM +0400, Joel Brobecker wrote:
> You may not remember as this was a couple of months ago (already!),
> but we had determined at the time that we could not easily disable
> exec events during inferior startup on HP/UX. So we decided to find
> a way to treat these exec events as SIGTRAPs.
>
> Here is a possible solution that does not involve the use of a global.
> It involves the addition of a new parameter to wait_for_inferior. Most
> of the time, we will set it so that events are not handled as is. But
> during the inferior startup sequence, we tell it to translate EXEC
> events into SIGTRAPs. I should probably add a comment at the beginning
> of wait_for_inferior explaining the intent of that new parameter -
> I will do that if you like the idea.
>
> 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.
Hi Joel,
This looks good to me - would you add that comment and check it in?
--
Daniel Jacobowitz
CodeSourcery