This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v4 1/4] Rename nptl-signals.h to internal-signals.h
- From: Florian Weimer <fweimer at redhat dot com>
- To: Adhemerval Zanella <adhemerval dot zanella at linaro dot org>, libc-alpha at sourceware dot org
- Date: Tue, 20 Feb 2018 13:32:13 +0100
- Subject: Re: [PATCH v4 1/4] Rename nptl-signals.h to internal-signals.h
- Authentication-results: sourceware.org; auth=none
- References: <1518439345-6013-1-git-send-email-adhemerval.zanella@linaro.org>
On 02/12/2018 01:42 PM, Adhemerval Zanella wrote:
This patch renames the nptl-signals.h header to internal-signals.h.
On Linux the definitions and functions are not only NPTL related, but
used for other POSIX definitions as well (for instance SIGTIMER for
posix times, SIGSETXID for id functions, and signal block/restore
POSIX timers?
Signed-off-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
We don't use DCO, but have copyright assignments.
diff --git a/nptl/pthreadP.h b/nptl/pthreadP.h
diff --git a/sysdeps/unix/sysv/linux/nptl-signals.h b/sysdeps/unix/sysv/linux/internal-signals.h
similarity index 89%
rename from sysdeps/unix/sysv/linux/nptl-signals.h
rename to sysdeps/unix/sysv/linux/internal-signals.h
index e789198..e007372 100644
--- a/sysdeps/unix/sysv/linux/nptl-signals.h
+++ b/sysdeps/unix/sysv/linux/internal-signals.h
@@ -1,4 +1,4 @@
-/* Special use of signals in NPTL internals. Linux version.
+/* Special use of signals internally. Linux version.
Copyright (C) 2014-2018 Free Software Foundation, Inc.
This file is part of the GNU C Library.
@@ -16,6 +16,9 @@
License along with the GNU C Library; if not, see
<http://www.gnu.org/licenses/>. */
+#ifndef __INTERNAL_SIGNALS_H
+# define __INTERNAL_SIGNALS_H
+
#include <signal.h>
#include <sigsetops.h>
@@ -35,17 +38,16 @@
/* Return is sig is used internally. */
static inline int
-__nptl_is_internal_signal (int sig)
+__is_internal_signal (int sig)
{
- return (sig == SIGCANCEL) || (sig == SIGTIMER) || (sig == SIGSETXID);
+ return (sig == SIGCANCEL) || (sig == SIGSETXID);
Should this change be mentioned in the ChangeLog? You could remove the
unnecessary parens because you modify this line anyway.
/* Remove internal glibc signal from the mask. */
static inline void
-__nptl_clear_internal_signals (sigset_t *set)
+__clear_internal_signals (sigset_t *set)
{
__sigdelset (set, SIGCANCEL);
- __sigdelset (set, SIGTIMER);
Likewise, should be mentioned in the ChangeLog entry.
Looks okay otherwise.
Thanks,
Florian