This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [Patch 6/7]: 68HC11 port of gdb (sim-misc single step fix)
- To: Stephane Carrez <Stephane dot Carrez at worldnet dot fr>
- Subject: Re: [Patch 6/7]: 68HC11 port of gdb (sim-misc single step fix)
- From: Andrew Cagney <ac131313 at cygnus dot com>
- Date: Mon, 31 Jul 2000 23:45:22 +1000
- CC: gdb-patches at sourceware dot cygnus dot com
- References: <39568A82.A0B2BFEB@free.fr> <3980227F.A6FC263B@cygnus.com> <3982002E.1F34EFF9@worldnet.fr>
> > > diff -Nrup --exclude-from=gdb-exclude.lst /src/gnu/cygnus/gdb/sim/common/sim-resume.c gdb/sim/common/sim-resume.c
> > > --- /src/gnu/cygnus/gdb/sim/common/sim-resume.c Mon Jul 12 13:14:09 1999
> > > +++ gdb/sim/common/sim-resume.c Mon Feb 21 09:00:23 2000
> > > @@ -51,8 +51,6 @@ sim_resume (SIM_DESC sd,
> > > sim_events_deschedule (sd, engine->stepper);
> > > engine->stepper = NULL;
> > > }
> > > - if (step)
> > > - engine->stepper = sim_events_schedule (sd, 1, has_stepped, sd);
> > >
> > > sim_module_resume (sd);
> > >
> > > @@ -69,6 +67,9 @@ sim_resume (SIM_DESC sd,
> > > sim_events_preprocess (sd, last_cpu_nr >= nr_cpus, next_cpu_nr >= nr_cpus);
> > > if (next_cpu_nr >= nr_cpus)
> > > next_cpu_nr = 0;
> > > +
> > > + if (step)
> > > + engine->stepper = sim_events_schedule (sd, 1, has_stepped, sd);
> > >
> > > /* Only deliver the siggnal ]sic] the first time through - don't
> > > re-deliver any siggnal during a restart. */
> >
> > Note the comment about siggnals[sic].
> >
> > Andrew
>
> ssignal does not seem to be ever used. It seemed to be ignored in most
> simulators (and in sim_engine_run).
From memory, siggnal will be non-zero, if, from the GDB cli, someone
enters:
(gdb) signal NNNN
Most simulators ignore it because most don't allow trapping of system
interrupts.
> I don't reach that point anyway when the problem occurs. I don't return
> from 'sim_events_preprocess' because it calls 'sim_engine_halt' that longjumps
> to return (due to single step event we just scheduled).
What about a sim_engine_restart()?
Andrew