This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [PATCH] Use multi-arch'd START_INFERIOR_TRAPS_EXPECTED on Alphatarget
> On Wed, Apr 24, 2002 at 08:06:31PM -0400, Andrew Cagney wrote:
>
> > #define STARTUP_WITH_SHELL 1
> > - #define START_INFERIOR_TRAPS_EXPECTED (STARTUP_WITH_SHELL + 1)
> > > Does STARTUP_WITH_SHELL need to be multi-arched?
> > > I'm wondering if it would be better to make it a variable (``set
> > startup-with-shell <boolean>''). Looking at its uses it appears that
> > fork-child.c:startup_inferior() would still work (if it did previously).
Hmm, ``HP Merge'' ~= ``there be dragons''.
> Hm. Actually, it occurs to me that this is more like what we want, but
> not quite. What we really want is:
>
> STARTUP_WITH_SHELL_ADDS_TRAPS
>
> So, then we get:
>
> #define START_INFERIOR_TRAPS_EXPECTED \
> (startup_with_shell ? (STARTUP_WITH_SHELL_ADDS_TRAPS + 1) : 1)
>
> I.e. the issue is really how many extra traps the shell adds.
I've included a few more excerts. I think this ``feature'' (only ever
worked on HP/UX. Can I suggest taking a step back and making the
feature work :-)
Andrew
PS: I think the relevant bits are:
inftarg.c:
startup_inferior (START_INFERIOR_TRAPS_EXPECTED);
fork-child.c:
if (STARTUP_WITH_SHELL)
inferior_ignoring_startup_exec_events = ntraps;
else
inferior_ignoring_startup_exec_events = 0;
inferior_ignoring_leading_exec_events =
target_reported_exec_events_per_exec_call () - 1;
while pre-merge it read (unchanged since '94):
/* Accept NTRAPS traps from the inferior. */
void
startup_inferior (ntraps)
int ntraps;
{
int pending_execs = ntraps;
int terminal_initted;
/* The process was started by the fork that created it,
but it will have stopped one instruction after execing the shell.
Here we must get it up to actual execution of the real program. */
clear_proceed_status ();
init_wait_for_inferior ();
terminal_initted = 0;
#ifdef STARTUP_INFERIOR
STARTUP_INFERIOR (pending_execs);
#else
while (1)
{
stop_soon_quietly = 1; /* Make wait_for_inferior be quiet */
wait_for_inferior ();
if (stop_signal != TARGET_SIGNAL_TRAP)
{
/* Let shell child handle its own signals in its own way */
/* FIXME, what if child has exit()ed? Must exit loop somehow */
resume (0, stop_signal);
}
else
{
/* We handle SIGTRAP, however; it means child did an exec. */
if (!terminal_initted)
{
/* Now that the child has exec'd we know it has already
set its
process group. On POSIX systems, tcsetpgrp will fail with
EPERM if we try it before the child's setpgid. */
/* Set up the "saved terminal modes" of the inferior
based on what modes we are starting it with. */
target_terminal_init ();
/* Install inferior's terminal modes. */
target_terminal_inferior ();
terminal_initted = 1;
}
if (0 == --pending_execs)
break;
resume (0, TARGET_SIGNAL_0); /* Just make it go on */
}
}
#endif /* STARTUP_INFERIOR */
stop_soon_quietly = 0;
}