aboutsummaryrefslogtreecommitdiff
path: root/ChangeLog
diff options
context:
space:
mode:
authorAdhemerval Zanella <adhemerval.zanella@linaro.org>2018-05-10 17:24:56 -0300
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>2019-01-03 18:38:16 -0200
commitce7eb0e90315eb1939ac29f656d39b3db858c092 (patch)
tree66f46e348749230c9fb9c95e7d77b01eff8598c9 /ChangeLog
parent0b13e25581afb4ce95373f18249b91c104280a13 (diff)
downloadglibc-ce7eb0e90315eb1939ac29f656d39b3db858c092.tar.xz
glibc-ce7eb0e90315eb1939ac29f656d39b3db858c092.zip
nptl: Cleanup cancellation macros
This patch wraps all uses of *_{enable,disable}_asynccancel and and *_CANCEL_{ASYNC,RESET} in either already provided macros (lll_futex_timed_wait_cancel) or creates new ones if the functionality is not provided (SYSCALL_CANCEL_NCS, lll_futex_wait_cancel, and lll_futex_timed_wait_cancel). Also for some generic implementations, the direct call of the macros are removed since the underlying symbols are suppose to provide cancellation support. This is a priliminary patch intended to simplify the work required for BZ#12683 fix. It is a refactor change, no semantic changes are expected. Checked on x86_64-linux-gnu and i686-linux-gnu. * nptl/pthread_join_common.c (__pthread_timedjoin_ex): Use lll_wait_tid with timeout. * nptl/sem_wait.c (__old_sem_wait): Use lll_futex_wait_cancel. * sysdeps/nptl/aio_misc.h (AIO_MISC_WAIT): Use futex_reltimed_wait_cancelable for cancelabla mode. * sysdeps/nptl/gai_misc.h (GAI_MISC_WAIT): Likewise. * sysdeps/posix/open64.c (__libc_open64): Do not call cancelation macros. * sysdeps/posix/sigwait.c (__sigwait): Likewise. * sysdeps/posix/waitid.c (__sigwait): Likewise. * sysdeps/unix/sysdep.h (__SYSCALL_CANCEL_CALL, SYSCALL_CANCEL_NCS): New macro. * sysdeps/nptl/lowlevellock.h (lll_wait_tid): Add timeout argument. (lll_timedwait_tid): Remove macro. * sysdeps/unix/sysv/linux/i386/lowlevellock.h (lll_wait_tid): Likewise. (lll_timedwait_tid): Likewise. * sysdeps/unix/sysv/linux/sparc/lowlevellock.h (lll_wait_tid): Likewise. (lll_timedwait_tid): Likewise. * sysdeps/unix/sysv/linux/x86_64/lowlevellock.h (lll_wait_tid): Likewise. (lll_timedwait_tid): Likewise. * sysdeps/unix/sysv/linux/clock_nanosleep.c (__clock_nanosleep): Use INTERNAL_SYSCALL_CANCEL. * sysdeps/unix/sysv/linux/futex-internal.h (futex_reltimed_wait_cancelable): Use LIBC_CANCEL_{ASYNC,RESET} instead of __pthread_{enable,disable}_asynccancel. * sysdeps/unix/sysv/linux/lowlevellock-futex.h (lll_futex_wait_cancel): New macro.
Diffstat (limited to 'ChangeLog')
-rw-r--r--ChangeLog31
1 files changed, 31 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index 159808db85..c7b2feb21b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,36 @@
2019-01-03 Adhemerval Zanella <adhemerval.zanella@linaro.org>
+ * nptl/pthread_join_common.c (__pthread_timedjoin_ex): Use
+ lll_wait_tid with timeout.
+ * nptl/sem_wait.c (__old_sem_wait): Use lll_futex_wait_cancel.
+ * sysdeps/nptl/aio_misc.h (AIO_MISC_WAIT): Use
+ futex_reltimed_wait_cancelable for cancelabla mode.
+ * sysdeps/nptl/gai_misc.h (GAI_MISC_WAIT): Likewise.
+ * sysdeps/posix/open64.c (__libc_open64): Do not call cancelation
+ macros.
+ * sysdeps/posix/sigwait.c (__sigwait): Likewise.
+ * sysdeps/posix/waitid.c (__sigwait): Likewise.
+ * sysdeps/unix/sysdep.h (__SYSCALL_CANCEL_CALL,
+ SYSCALL_CANCEL_NCS): New macro.
+ * sysdeps/nptl/lowlevellock.h (lll_wait_tid): Add timeout argument.
+ (lll_timedwait_tid): Remove macro.
+ * sysdeps/unix/sysv/linux/i386/lowlevellock.h (lll_wait_tid):
+ Likewise.
+ (lll_timedwait_tid): Likewise.
+ * sysdeps/unix/sysv/linux/sparc/lowlevellock.h (lll_wait_tid):
+ Likewise.
+ (lll_timedwait_tid): Likewise.
+ * sysdeps/unix/sysv/linux/x86_64/lowlevellock.h (lll_wait_tid):
+ Likewise.
+ (lll_timedwait_tid): Likewise.
+ * sysdeps/unix/sysv/linux/clock_nanosleep.c (__clock_nanosleep):
+ Use INTERNAL_SYSCALL_CANCEL.
+ * sysdeps/unix/sysv/linux/futex-internal.h
+ (futex_reltimed_wait_cancelable): Use LIBC_CANCEL_{ASYNC,RESET}
+ instead of __pthread_{enable,disable}_asynccancel.
+ * sysdeps/unix/sysv/linux/lowlevellock-futex.h
+ (lll_futex_wait_cancel): New macro.
+
* sysdeps/i386/nptl/tls.h (THREAD_ATOMIC_CMPXCHG_VAL,
THREAD_ATOMIC_AND, THREAD_ATOMIC_BIT_SET): Remove macros.