Factor out shared definitions from bits/signum.h.
commit8082d91e1c449e0cb137468b731004a5e605c8c6
authorZack Weinberg <zackw@panix.com>
Sat, 3 Jun 2017 20:22:24 +0000 (3 16:22 -0400)
committerZack Weinberg <zackw@panix.com>
Wed, 21 Jun 2017 00:32:50 +0000 (20 20:32 -0400)
tree0d90492e10b87d6d30a0c094fae8d7375439938a
parentaf85385f311c574590381f7897461643f8a2fe6f
Factor out shared definitions from bits/signum.h.

Many of the things defined by bits/signum.h are invariant across all
supported operating systems.  This patch factors out all of them to a
new header bits/signum-generic.h, which each bits/signum.h will include
and then override whichever things need adjustment.  Normally that will
mean, at most, adding or changing a few signal numbers.

A user-visible side effect is that the obsolete signal constant SIGUNUSED
(which is an alias for SIGSYS on all platforms that define it) is no
longer exposed by any version of bits/signum.h.

A side effect only relevant to glibc hackers is that _NSIG is now defined
in terms of __SIGRTMAX, instead of the other way around.  This is because
__SIGRTMAX varies from platform to platform, but _NSIG==__SIGRTMAX+1 is
true universally.  If your platform doesn't support realtime signals,
leave __SIGRTMAX equal to __SIGRTMIN.

I also added a Linux-specific test to make sure that our signal constants
match the ones in <asm/signal.h>, since we can't use that header (it's
not even vaguely namespace-clean).

* bits/signum-generic.h: Renamed from bits/signum.h.
Add proper multiple include guard and misuse check.
Define __SIGRTMIN = __SIGRTMAX = 32, and define _NSIG = __SIGRTMAX+1.
Move definition of SIGIO to "archaic names for compatibility" section.
* bits/signum.h: New file which just includes bits/signum-generic.h.
* sysdeps/unix/bsd/bits/signum.h
* sysdeps/unix/sysv/linux/bits/signum.h
* sysdeps/unix/sysv/linux/alpha/bits/signum.h
* sysdeps/unix/sysv/linux/hppa/bits/signum.h
* sysdeps/unix/sysv/linux/mips/bits/signum.h
* sysdeps/unix/sysv/linux/sparc/bits/signum.h
Just include <bits/signum-generic.h> and then add or adjust
signal constants.  Do not define SIGUNUSED, SIGRTMIN, or SIGRTMAX.

* signal/Makefile: Install bits/signum-generic.h.
* signal/signal.h: Define SIGRTMIN and SIGRTMAX here.

* sysdeps/generic/siglist.h: SIGSYS and SIGWINCH are
universal.  Prefer SIGPOLL to SIGIO. Simplify #ifdeffage.

* sysdeps/unix/sysv/linux/tst-signal-numbers.sh: New test.
* sysdeps/unix/sysv/linux/Makefile: Run it.
15 files changed:
ChangeLog
NEWS
bits/signum-generic.h [copied from bits/signum.h with 66% similarity]
bits/signum.h
signal/Makefile
signal/signal.h
sysdeps/generic/siglist.h
sysdeps/unix/bsd/bits/signum.h
sysdeps/unix/sysv/linux/Makefile
sysdeps/unix/sysv/linux/alpha/bits/signum.h
sysdeps/unix/sysv/linux/bits/signum.h
sysdeps/unix/sysv/linux/hppa/bits/signum.h
sysdeps/unix/sysv/linux/mips/bits/signum.h
sysdeps/unix/sysv/linux/sparc/bits/signum.h
sysdeps/unix/sysv/linux/tst-signal-numbers.sh [new file with mode: 0644]