This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [csl-am] missing SIGTRAP


On Mon, Jul 25, 2005 at 07:59:04PM +0200, Mark Kettenis wrote:
>    I was going to ignore this patch for mainline, since it indeed isn't
>    the right way.  Then I took a look at the simulators and how much of a
>    PAIN it's going to be to migrate them off of host signal numbers.
> 
>    Does anyone object to the inclusion of this patch?
> 
> The problem here is the fact that the simulator tries to map host
> events and simulator events to the same thing isn't it?  I'm not too
> concerned with the simulators, but I think someone who does care
> should really fix this.  Meanwhile I don't object to a local hack in
> gdb/remote-sim.c.

No, I think that the problem is simpler than that.  The simulator
doesn't care about host events.  It's mapping target events to signal
numbers; it just picked the SIG* numbers instead of the TARGET_SIGNAL_*
numbers.  Which are only recently (in the grand scheme of things -
about three years ago) available in include/gdb/.

> However, I *do* object to the change to gdb/signals/signals.c.  This
> is supposed to be a host to target mapping of signals.  Making it
> pretend that the host has SIGTRAP while it doesn't is a truly bad
> thing to do.

Well then the hack in remote-sim.c will have to be a bit more
intrusive... it calls target_signal_to_host and target_signal_from_host
on this value.  That's doable.

-- 
Daniel Jacobowitz
CodeSourcery, LLC


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]