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]

Re: [RFA] target.c: Add support for Solaris realtime signals


"Peter.Schauer" wrote:
> 
> This patch adds support for Solaris realtime signals, they are modelled after
> (ummh, more or less copied from) the REALTIME_LO/HI support.
> 
> Unfortunately I can't add a testsuite case to sigall.exp, as the definitions
> of the signal numbers are dynamic.
> 
>         * target.c (target_signal_from_host, do_target_signal_to_host):
>         Add support for Solaris realtime signals.

I suspect it should call internal_error() rather than error() - if the
signal turns out to be outside of that range then there is something
serious.  However, you mention that the signal numbering is dynamic -
does this mean that the number of realtime signals can be increased to
something more than 32 at runtime?) - hence the reason I'm not 100%
sure.

Other than this, I've no problems with your patch.

	enjoy,
		Andrew

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