[1/4] Clean pthread functions namespaces for C11 threads
Commit Message
This patch adds internal definition (through {libc_}hidden_{proto,def}) and
also change some strong to weak alias for symbols that might be used by C11
threads implementations.
The patchset should not change libc/libpthread functional, although object
changes are expected (since now internal symbols are used instead) and final
exported symbols through GLIBC_PRIVATE is also expanded (to cover libpthread
usage of __mmap{64}, __munmap, __mprotect).
Checked with a build for all major ABI (aarch64-linux-gnu, alpha-linux-gnu,
arm-linux-gnueabi, i386-linux-gnu, ia64-linux-gnu, m68k-linux-gnu,
microblaze-linux-gnu [1], mips{64}-linux-gnu, nios2-linux-gnu,
powerpc{64le}-linux-gnu, s390{x}-linux-gnu, sparc{64}-linux-gnu,
tile{pro,gx}-linux-gnu, and x86_64-linux-gnu).
* include/sched.h (__sched_get_priority_max): Add libc hidden proto.
(__sched_get_prioriry_min): Likewise.
* include/sys/mman.h (__mmap): Likewise.
(__mmap64): Likewise.
(__munmap): Likewise.
(__mprotect): Likewise.
* include/termios.h (__tcsetattr): Likewise.
* include/time.h (__nanosleep): Use hidden_proto instead of
libc_hidden_proto.
* posix/nanosleep.c (__nanosleep): Likewise.
* misc/Versions (libc): Export __mmap, __munmap, __mprotect,
__sched_get_priority_min, and __sched_get_priority_max under
GLIBC_PRIVATE.
* nptl/allocatestack.c (__free_stacks): Use internal definition for
libc symbols.
(setup_stack_prot): Likewise.
(change_stack_perm): Likewise.
(allocate_stack): Likewise.
* sysdeps/posix/gethostname.c: Likewise.
* nptl/tpp.c (__init_sched_fifo_prio): Likewise.
* sysdeps/unix/sysv/linux/i386/smp.h (is_smp_system): Likewise.
* sysdeps/unix/sysv/linux/powerpc/ioctl.c (__ioctl): Likewise.
* nptl/pthreadP.h (__pthread_mutex_timedlock): Add definition.
(__pthread_key_delete): Likewise.
(__pthread_detach): Likewise.
(__pthread_cancel): Likewise.
(__pthread_mutex_trylock): Likewise.
(__pthread_mutexattr_init): Likewise.
(__pthread_mutexattr_settype): Likewise.
* nptl/pthread_cancel.c (pthread_cancel): Change to internal name and
create alias for exported one.
* nptl/pthread_join.c (pthread_join): Likewise.
* nptl/pthread_detach.c (pthread_detach): Likewise.
* nptl/pthread_key_delete.c (pthread_key_delete): Likewise.
* nptl/pthread_mutex_timedlock.c (pthread_mutex_timedlock): Likewise.
* nptl/pthread_create.c: Change static requirements for pthread
symbols.
* nptl/pthread_equal.c (__pthread_equal): Change strong alias to weak
for internal definition.
* nptl/pthread_exit.c (__pthread_exit): Likewise.
* nptl/pthread_getspecific.c (__pthread_getspecific): Likewise.
* nptl/pthread_key_create.c (__pthread_key_create): Likewise.
* nptl/pthread_mutex_destroy.c (__pthread_mutex_destroy): Likewise.
* nptl/pthread_mutex_init.c (__pthread_mutex_init): Likewise.
* nptl/pthread_mutex_lock.c (__pthread_mutex_lock): Likewise.
* nptl/pthread_mutex_trylock.c (__pthread_mutex_trylock): Likewise.
* nptl/pthread_mutex_unlock.c (__pthread_mutex_unlock): Likewise.
* nptl/pthread_mutexattr_init.c (__pthread_mutexattr_init): Likwise.
* nptl/pthread_mutexattr_settype.c (__pthread_mutexattr_settype):
Likewise.
* nptl/pthread_self.c (__pthread_self): Likewise.
* nptl/pthread_setspecific.c (__pthread_setspecific): Likewise.
* sysdeps/unix/sysv/linux/tcsetattr.c (tcsetattr): Likewise.
* misc/mmap.c (__mmap): Add internal symbol definition.
* misc/mmap.c (__mmap64): Likewise.
* sysdeps/unix/sysv/linux/mmap.c (__mmap): Likewise.
* sysdeps/unix/sysv/linux/mmap64.c (__mmap): Likewise.
(__mmap64): Likewise.
* sysdeps/unix/sysv/linux/i386/Versions (libc) [GLIBC_PRIVATE):
Add __uname.
---
ChangeLog | 64 +++++++++++++++++++++++++++++++++
include/sched.h | 2 ++
include/sys/mman.h | 4 +++
include/termios.h | 2 ++
include/time.h | 2 +-
misc/Versions | 2 ++
misc/mmap.c | 1 +
misc/mmap64.c | 1 +
nptl/allocatestack.c | 32 ++++++++---------
nptl/pthreadP.h | 9 +++++
nptl/pthread_cancel.c | 7 ++--
nptl/pthread_create.c | 18 +++++-----
nptl/pthread_detach.c | 3 +-
nptl/pthread_equal.c | 2 +-
nptl/pthread_exit.c | 4 +--
nptl/pthread_getspecific.c | 2 +-
nptl/pthread_join.c | 3 +-
nptl/pthread_key_create.c | 2 +-
nptl/pthread_key_delete.c | 3 +-
nptl/pthread_mutex_destroy.c | 2 +-
nptl/pthread_mutex_init.c | 2 +-
nptl/pthread_mutex_lock.c | 2 +-
nptl/pthread_mutex_timedlock.c | 5 +--
nptl/pthread_mutex_trylock.c | 3 +-
nptl/pthread_mutex_unlock.c | 2 +-
nptl/pthread_mutexattr_init.c | 3 +-
nptl/pthread_mutexattr_settype.c | 3 +-
nptl/pthread_self.c | 2 +-
nptl/pthread_setspecific.c | 2 +-
nptl/tpp.c | 4 +--
posix/nanosleep.c | 2 +-
sysdeps/posix/gethostname.c | 2 +-
sysdeps/unix/sysv/linux/i386/Versions | 2 +-
sysdeps/unix/sysv/linux/i386/smp.h | 2 +-
sysdeps/unix/sysv/linux/mmap.c | 1 +
sysdeps/unix/sysv/linux/mmap64.c | 2 ++
sysdeps/unix/sysv/linux/nanosleep.c | 2 +-
sysdeps/unix/sysv/linux/powerpc/ioctl.c | 6 ++--
sysdeps/unix/sysv/linux/tcsetattr.c | 3 +-
39 files changed, 156 insertions(+), 59 deletions(-)
Comments
On Tuesday 20 June 2017 12:22 AM, Adhemerval Zanella wrote:
> This patch adds internal definition (through {libc_}hidden_{proto,def}) and
> also change some strong to weak alias for symbols that might be used by C11
> threads implementations.
>
> The patchset should not change libc/libpthread functional, although object
> changes are expected (since now internal symbols are used instead) and final
> exported symbols through GLIBC_PRIVATE is also expanded (to cover libpthread
> usage of __mmap{64}, __munmap, __mprotect).
>
> Checked with a build for all major ABI (aarch64-linux-gnu, alpha-linux-gnu,
> arm-linux-gnueabi, i386-linux-gnu, ia64-linux-gnu, m68k-linux-gnu,
> microblaze-linux-gnu [1], mips{64}-linux-gnu, nios2-linux-gnu,
> powerpc{64le}-linux-gnu, s390{x}-linux-gnu, sparc{64}-linux-gnu,
> tile{pro,gx}-linux-gnu, and x86_64-linux-gnu).
Looks OK with one small nit/comment below.
Siddhesh
>
> * include/sched.h (__sched_get_priority_max): Add libc hidden proto.
> (__sched_get_prioriry_min): Likewise.
> * include/sys/mman.h (__mmap): Likewise.
> (__mmap64): Likewise.
> (__munmap): Likewise.
> (__mprotect): Likewise.
> * include/termios.h (__tcsetattr): Likewise.
> * include/time.h (__nanosleep): Use hidden_proto instead of
> libc_hidden_proto.
> * posix/nanosleep.c (__nanosleep): Likewise.
> * misc/Versions (libc): Export __mmap, __munmap, __mprotect,
> __sched_get_priority_min, and __sched_get_priority_max under
> GLIBC_PRIVATE.
> * nptl/allocatestack.c (__free_stacks): Use internal definition for
> libc symbols.
> (setup_stack_prot): Likewise.
> (change_stack_perm): Likewise.
> (allocate_stack): Likewise.
> * sysdeps/posix/gethostname.c: Likewise.
> * nptl/tpp.c (__init_sched_fifo_prio): Likewise.
> * sysdeps/unix/sysv/linux/i386/smp.h (is_smp_system): Likewise.
> * sysdeps/unix/sysv/linux/powerpc/ioctl.c (__ioctl): Likewise.
> * nptl/pthreadP.h (__pthread_mutex_timedlock): Add definition.
> (__pthread_key_delete): Likewise.
> (__pthread_detach): Likewise.
> (__pthread_cancel): Likewise.
> (__pthread_mutex_trylock): Likewise.
> (__pthread_mutexattr_init): Likewise.
> (__pthread_mutexattr_settype): Likewise.
> * nptl/pthread_cancel.c (pthread_cancel): Change to internal name and
> create alias for exported one.
> * nptl/pthread_join.c (pthread_join): Likewise.
> * nptl/pthread_detach.c (pthread_detach): Likewise.
> * nptl/pthread_key_delete.c (pthread_key_delete): Likewise.
> * nptl/pthread_mutex_timedlock.c (pthread_mutex_timedlock): Likewise.
> * nptl/pthread_create.c: Change static requirements for pthread
> symbols.
> * nptl/pthread_equal.c (__pthread_equal): Change strong alias to weak
> for internal definition.
> * nptl/pthread_exit.c (__pthread_exit): Likewise.
> * nptl/pthread_getspecific.c (__pthread_getspecific): Likewise.
> * nptl/pthread_key_create.c (__pthread_key_create): Likewise.
> * nptl/pthread_mutex_destroy.c (__pthread_mutex_destroy): Likewise.
> * nptl/pthread_mutex_init.c (__pthread_mutex_init): Likewise.
> * nptl/pthread_mutex_lock.c (__pthread_mutex_lock): Likewise.
> * nptl/pthread_mutex_trylock.c (__pthread_mutex_trylock): Likewise.
> * nptl/pthread_mutex_unlock.c (__pthread_mutex_unlock): Likewise.
> * nptl/pthread_mutexattr_init.c (__pthread_mutexattr_init): Likwise.
> * nptl/pthread_mutexattr_settype.c (__pthread_mutexattr_settype):
> Likewise.
> * nptl/pthread_self.c (__pthread_self): Likewise.
> * nptl/pthread_setspecific.c (__pthread_setspecific): Likewise.
> * sysdeps/unix/sysv/linux/tcsetattr.c (tcsetattr): Likewise.
> * misc/mmap.c (__mmap): Add internal symbol definition.
> * misc/mmap.c (__mmap64): Likewise.
> * sysdeps/unix/sysv/linux/mmap.c (__mmap): Likewise.
> * sysdeps/unix/sysv/linux/mmap64.c (__mmap): Likewise.
> (__mmap64): Likewise.
> * sysdeps/unix/sysv/linux/i386/Versions (libc) [GLIBC_PRIVATE):
> Add __uname.
> ---
> ChangeLog | 64 +++++++++++++++++++++++++++++++++
> include/sched.h | 2 ++
> include/sys/mman.h | 4 +++
> include/termios.h | 2 ++
> include/time.h | 2 +-
> misc/Versions | 2 ++
> misc/mmap.c | 1 +
> misc/mmap64.c | 1 +
> nptl/allocatestack.c | 32 ++++++++---------
> nptl/pthreadP.h | 9 +++++
> nptl/pthread_cancel.c | 7 ++--
> nptl/pthread_create.c | 18 +++++-----
> nptl/pthread_detach.c | 3 +-
> nptl/pthread_equal.c | 2 +-
> nptl/pthread_exit.c | 4 +--
> nptl/pthread_getspecific.c | 2 +-
> nptl/pthread_join.c | 3 +-
> nptl/pthread_key_create.c | 2 +-
> nptl/pthread_key_delete.c | 3 +-
> nptl/pthread_mutex_destroy.c | 2 +-
> nptl/pthread_mutex_init.c | 2 +-
> nptl/pthread_mutex_lock.c | 2 +-
> nptl/pthread_mutex_timedlock.c | 5 +--
> nptl/pthread_mutex_trylock.c | 3 +-
> nptl/pthread_mutex_unlock.c | 2 +-
> nptl/pthread_mutexattr_init.c | 3 +-
> nptl/pthread_mutexattr_settype.c | 3 +-
> nptl/pthread_self.c | 2 +-
> nptl/pthread_setspecific.c | 2 +-
> nptl/tpp.c | 4 +--
> posix/nanosleep.c | 2 +-
> sysdeps/posix/gethostname.c | 2 +-
> sysdeps/unix/sysv/linux/i386/Versions | 2 +-
> sysdeps/unix/sysv/linux/i386/smp.h | 2 +-
> sysdeps/unix/sysv/linux/mmap.c | 1 +
> sysdeps/unix/sysv/linux/mmap64.c | 2 ++
> sysdeps/unix/sysv/linux/nanosleep.c | 2 +-
> sysdeps/unix/sysv/linux/powerpc/ioctl.c | 6 ++--
> sysdeps/unix/sysv/linux/tcsetattr.c | 3 +-
> 39 files changed, 156 insertions(+), 59 deletions(-)
>
> diff --git a/include/sched.h b/include/sched.h
> index b4d7406..f75faed 100644
> --- a/include/sched.h
> +++ b/include/sched.h
> @@ -13,7 +13,9 @@ extern int __sched_getscheduler (__pid_t __pid);
> extern int __sched_yield (void);
> libc_hidden_proto (__sched_yield)
> extern int __sched_get_priority_max (int __algorithm);
> +libc_hidden_proto (__sched_get_priority_max)
> extern int __sched_get_priority_min (int __algorithm);
> +libc_hidden_proto (__sched_get_priority_min)
> extern int __sched_rr_get_interval (__pid_t __pid, struct timespec *__t);
>
> /* These are Linux specific. */
> diff --git a/include/sys/mman.h b/include/sys/mman.h
> index 7026f69..8b996fc 100644
> --- a/include/sys/mman.h
> +++ b/include/sys/mman.h
> @@ -5,10 +5,14 @@
> /* Now define the internal interfaces. */
> extern void *__mmap (void *__addr, size_t __len, int __prot,
> int __flags, int __fd, __off_t __offset);
> +libc_hidden_proto (__mmap)
> extern void *__mmap64 (void *__addr, size_t __len, int __prot,
> int __flags, int __fd, __off64_t __offset);
> +libc_hidden_proto (__mmap64)
> extern int __munmap (void *__addr, size_t __len);
> +libc_hidden_proto (__munmap)
> extern int __mprotect (void *__addr, size_t __len, int __prot);
> +libc_hidden_proto (__mprotect)
>
> extern int __madvise (void *__addr, size_t __len, int __advice);
> libc_hidden_proto (__madvise)
> diff --git a/include/termios.h b/include/termios.h
> index fad51f8..1a36e22 100644
> --- a/include/termios.h
> +++ b/include/termios.h
> @@ -4,6 +4,8 @@
> #ifndef _ISOMAC
> /* Now define the internal interfaces. */
> extern int __tcgetattr (int __fd, struct termios *__termios_p);
> +extern int __tcsetattr (int __fd, int __optional_actions,
> + const struct termios *__termios_p);
>
> extern int __libc_tcdrain (int __fd);
>
> diff --git a/include/time.h b/include/time.h
> index 3a828e0..b72a3a3 100644
> --- a/include/time.h
> +++ b/include/time.h
> @@ -78,7 +78,7 @@ extern struct tm *__tz_convert (const time_t *timer, int use_localtime, struct t
>
> extern int __nanosleep (const struct timespec *__requested_time,
> struct timespec *__remaining);
> -libc_hidden_proto (__nanosleep)
> +hidden_proto (__nanosleep)
Wouldn't it be more correct to define libc_hidden_* for IS_IN(libc) as
well as IS_IN(pthread) in libc-symbols.h? Basically just enable usage
of all of the libc_* macros there for libc as well as libpthread.
> extern int __nanosleep_nocancel (const struct timespec *__requested_time,
> struct timespec *__remaining)
> attribute_hidden;
> diff --git a/misc/Versions b/misc/Versions
> index bafda78..bfbda50 100644
> --- a/misc/Versions
> +++ b/misc/Versions
> @@ -163,5 +163,7 @@ libc {
> __mktemp;
> __libc_ifunc_impl_list;
> __tdelete; __tfind; __tsearch; __twalk;
> + __mmap; __munmap; __mprotect;
> + __sched_get_priority_min; __sched_get_priority_max;
> }
> }
> diff --git a/misc/mmap.c b/misc/mmap.c
> index 576d66a..4172d76 100644
> --- a/misc/mmap.c
> +++ b/misc/mmap.c
> @@ -37,3 +37,4 @@ __mmap (__ptr_t addr, size_t len, int prot, int flags, int fd, off_t offset)
>
> stub_warning (mmap)
> weak_alias (__mmap, mmap)
> +libc_hidden_def (__mmap)
> diff --git a/misc/mmap64.c b/misc/mmap64.c
> index 6b1ac41..845b847 100644
> --- a/misc/mmap64.c
> +++ b/misc/mmap64.c
> @@ -45,3 +45,4 @@ __mmap64 (__ptr_t addr, size_t len, int prot, int flags, int fd,
> }
>
> weak_alias (__mmap64, mmap64)
> +libc_hidden_def (__mmap64)
> diff --git a/nptl/allocatestack.c b/nptl/allocatestack.c
> index 8364406..ec7d42e 100644
> --- a/nptl/allocatestack.c
> +++ b/nptl/allocatestack.c
> @@ -278,7 +278,7 @@ __free_stacks (size_t limit)
>
> /* Remove this block. This should never fail. If it does
> something is really wrong. */
> - if (munmap (curr->stackblock, curr->stackblock_size) != 0)
> + if (__munmap (curr->stackblock, curr->stackblock_size) != 0)
> abort ();
>
> /* Maybe we have freed enough. */
> @@ -328,7 +328,7 @@ change_stack_perm (struct pthread *pd
> #else
> # error "Define either _STACK_GROWS_DOWN or _STACK_GROWS_UP"
> #endif
> - if (mprotect (stack, len, PROT_READ | PROT_WRITE | PROT_EXEC) != 0)
> + if (__mprotect (stack, len, PROT_READ | PROT_WRITE | PROT_EXEC) != 0)
> return errno;
>
> return 0;
> @@ -359,14 +359,14 @@ setup_stack_prot (char *mem, size_t size, char *guard, size_t guardsize,
> #if _STACK_GROWS_DOWN
> /* As defined at guard_position, for architectures with downward stack
> the guard page is always at start of the allocated area. */
> - if (mprotect (guardend, size - guardsize, prot) != 0)
> + if (__mprotect (guardend, size - guardsize, prot) != 0)
> return errno;
> #else
> size_t mprots1 = (uintptr_t) guard - (uintptr_t) mem;
> - if (mprotect (mem, mprots1, prot) != 0)
> + if (__mprotect (mem, mprots1, prot) != 0)
> return errno;
> size_t mprots2 = ((uintptr_t) mem + size) - (uintptr_t) guardend;
> - if (mprotect (guardend, mprots2, prot) != 0)
> + if (__mprotect (guardend, mprots2, prot) != 0)
> return errno;
> #endif
> return 0;
> @@ -530,8 +530,8 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
> /* If a guard page is required, avoid committing memory by first
> allocate with PROT_NONE and then reserve with required permission
> excluding the guard page. */
> - mem = mmap (NULL, size, (guardsize == 0) ? prot : PROT_NONE,
> - MAP_PRIVATE | MAP_ANONYMOUS | MAP_STACK, -1, 0);
> + mem = __mmap (NULL, size, (guardsize == 0) ? prot : PROT_NONE,
> + MAP_PRIVATE | MAP_ANONYMOUS | MAP_STACK, -1, 0);
>
> if (__glibc_unlikely (mem == MAP_FAILED))
> return errno;
> @@ -557,7 +557,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
> pagesize_m1);
> if (setup_stack_prot (mem, size, guard, guardsize, prot) != 0)
> {
> - munmap (mem, size);
> + __munmap (mem, size);
> return errno;
> }
> }
> @@ -600,7 +600,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
> assert (errno == ENOMEM);
>
> /* Free the stack memory we just allocated. */
> - (void) munmap (mem, size);
> + (void) __munmap (mem, size);
>
> return errno;
> }
> @@ -630,7 +630,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
> if (err != 0)
> {
> /* Free the stack memory we just allocated. */
> - (void) munmap (mem, size);
> + (void) __munmap (mem, size);
>
> return err;
> }
> @@ -650,7 +650,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
> {
> char *guard = guard_position (mem, size, guardsize, pd,
> pagesize_m1);
> - if (mprotect (guard, guardsize, PROT_NONE) != 0)
> + if (__mprotect (guard, guardsize, PROT_NONE) != 0)
> {
> mprot_error:
> lll_lock (stack_cache_lock, LLL_PRIVATE);
> @@ -668,7 +668,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
> of memory caused problems we better do not use it
> anymore. Uh, and we ignore possible errors. There
> is nothing we could do. */
> - (void) munmap (mem, size);
> + (void) __munmap (mem, size);
>
> return errno;
> }
> @@ -685,19 +685,19 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
> char *oldguard = mem + (((size - pd->guardsize) / 2) & ~pagesize_m1);
>
> if (oldguard < guard
> - && mprotect (oldguard, guard - oldguard, prot) != 0)
> + && __mprotect (oldguard, guard - oldguard, prot) != 0)
> goto mprot_error;
>
> - if (mprotect (guard + guardsize,
> + if (__mprotect (guard + guardsize,
> oldguard + pd->guardsize - guard - guardsize,
> prot) != 0)
> goto mprot_error;
> #elif _STACK_GROWS_DOWN
> - if (mprotect ((char *) mem + guardsize, pd->guardsize - guardsize,
> + if (__mprotect ((char *) mem + guardsize, pd->guardsize - guardsize,
> prot) != 0)
> goto mprot_error;
> #elif _STACK_GROWS_UP
> - if (mprotect ((char *) pd - pd->guardsize,
> + if (__mprotect ((char *) pd - pd->guardsize,
> pd->guardsize - guardsize, prot) != 0)
> goto mprot_error;
> #endif
> diff --git a/nptl/pthreadP.h b/nptl/pthreadP.h
> index 7fc1e50..6e7d6ff 100644
> --- a/nptl/pthreadP.h
> +++ b/nptl/pthreadP.h
> @@ -428,6 +428,8 @@ extern int __pthread_mutex_init (pthread_mutex_t *__mutex,
> extern int __pthread_mutex_destroy (pthread_mutex_t *__mutex);
> extern int __pthread_mutex_trylock (pthread_mutex_t *_mutex);
> extern int __pthread_mutex_lock (pthread_mutex_t *__mutex);
> +extern int __pthread_mutex_timedlock (pthread_mutex_t *__mutex,
> + const struct timespec *__abstime);
> extern int __pthread_mutex_cond_lock (pthread_mutex_t *__mutex)
> attribute_hidden internal_function;
> extern void __pthread_mutex_cond_lock_adjust (pthread_mutex_t *__mutex)
> @@ -491,6 +493,7 @@ extern int __pthread_cond_timedwait (pthread_cond_t *cond,
> extern int __pthread_condattr_destroy (pthread_condattr_t *attr);
> extern int __pthread_condattr_init (pthread_condattr_t *attr);
> extern int __pthread_key_create (pthread_key_t *key, void (*destr) (void *));
> +extern int __pthread_key_delete (pthread_key_t key);
> extern void *__pthread_getspecific (pthread_key_t key);
> extern int __pthread_setspecific (pthread_key_t key, const void *value);
> extern int __pthread_once (pthread_once_t *once_control,
> @@ -499,8 +502,11 @@ extern int __pthread_atfork (void (*prepare) (void), void (*parent) (void),
> void (*child) (void));
> extern pthread_t __pthread_self (void);
> extern int __pthread_equal (pthread_t thread1, pthread_t thread2);
> +extern int __pthread_detach (pthread_t th);
> +extern int __pthread_cancel (pthread_t th);
> extern int __pthread_kill (pthread_t threadid, int signo);
> extern void __pthread_exit (void *value) __attribute__ ((__noreturn__));
> +extern int __pthread_join (pthread_t threadid, void **thread_return);
> extern int __pthread_setcanceltype (int type, int *oldtype);
> extern int __pthread_enable_asynccancel (void) attribute_hidden;
> extern void __pthread_disable_asynccancel (int oldtype)
> @@ -511,6 +517,7 @@ extern void __pthread_testcancel (void);
> hidden_proto (__pthread_mutex_init)
> hidden_proto (__pthread_mutex_destroy)
> hidden_proto (__pthread_mutex_lock)
> +hidden_proto (__pthread_mutex_trylock)
> hidden_proto (__pthread_mutex_unlock)
> hidden_proto (__pthread_rwlock_rdlock)
> hidden_proto (__pthread_rwlock_wrlock)
> @@ -521,6 +528,8 @@ hidden_proto (__pthread_setspecific)
> hidden_proto (__pthread_once)
> hidden_proto (__pthread_setcancelstate)
> hidden_proto (__pthread_testcancel)
> +hidden_proto (__pthread_mutexattr_init)
> +hidden_proto (__pthread_mutexattr_settype)
> #endif
>
> extern int __pthread_cond_broadcast_2_0 (pthread_cond_2_0_t *cond);
> diff --git a/nptl/pthread_cancel.c b/nptl/pthread_cancel.c
> index 231a58d..742dfe6 100644
> --- a/nptl/pthread_cancel.c
> +++ b/nptl/pthread_cancel.c
> @@ -25,7 +25,7 @@
> #include <unistd.h>
>
> int
> -pthread_cancel (pthread_t th)
> +__pthread_cancel (pthread_t th)
> {
> volatile struct pthread *pd = (volatile struct pthread *) th;
>
> @@ -66,7 +66,7 @@ pthread_cancel (pthread_t th)
> #ifdef SIGCANCEL
> /* The cancellation handler will take care of marking the
> thread as canceled. */
> - pid_t pid = getpid ();
> + pid_t pid = __getpid ();
>
> INTERNAL_SYSCALL_DECL (err);
> int val = INTERNAL_SYSCALL_CALL (tgkill, err, pid, pd->tid,
> @@ -99,5 +99,6 @@ pthread_cancel (pthread_t th)
>
> return result;
> }
> +weak_alias (__pthread_cancel, pthread_cancel)
>
> -PTHREAD_STATIC_FN_REQUIRE (pthread_create)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_create)
> diff --git a/nptl/pthread_create.c b/nptl/pthread_create.c
> index c7d1b8f..7a970ff 100644
> --- a/nptl/pthread_create.c
> +++ b/nptl/pthread_create.c
> @@ -920,14 +920,14 @@ compat_symbol (libpthread, __pthread_create_2_0, pthread_create,
>
> /* If pthread_create is present, libgcc_eh.a and libsupc++.a expects some other POSIX thread
> functions to be present as well. */
> -PTHREAD_STATIC_FN_REQUIRE (pthread_mutex_lock)
> -PTHREAD_STATIC_FN_REQUIRE (pthread_mutex_trylock)
> -PTHREAD_STATIC_FN_REQUIRE (pthread_mutex_unlock)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_mutex_lock)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_mutex_trylock)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_mutex_unlock)
>
> -PTHREAD_STATIC_FN_REQUIRE (pthread_once)
> -PTHREAD_STATIC_FN_REQUIRE (pthread_cancel)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_once)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_cancel)
>
> -PTHREAD_STATIC_FN_REQUIRE (pthread_key_create)
> -PTHREAD_STATIC_FN_REQUIRE (pthread_key_delete)
> -PTHREAD_STATIC_FN_REQUIRE (pthread_setspecific)
> -PTHREAD_STATIC_FN_REQUIRE (pthread_getspecific)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_key_create)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_key_delete)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_setspecific)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_getspecific)
> diff --git a/nptl/pthread_detach.c b/nptl/pthread_detach.c
> index 8a2e943..5c4c8f7 100644
> --- a/nptl/pthread_detach.c
> +++ b/nptl/pthread_detach.c
> @@ -22,7 +22,7 @@
>
>
> int
> -pthread_detach (pthread_t th)
> +__pthread_detach (pthread_t th)
> {
> struct pthread *pd = (struct pthread *) th;
>
> @@ -53,3 +53,4 @@ pthread_detach (pthread_t th)
>
> return result;
> }
> +weak_alias (__pthread_detach, pthread_detach)
> diff --git a/nptl/pthread_equal.c b/nptl/pthread_equal.c
> index e304add..b5244ee 100644
> --- a/nptl/pthread_equal.c
> +++ b/nptl/pthread_equal.c
> @@ -24,4 +24,4 @@ __pthread_equal (pthread_t thread1, pthread_t thread2)
> {
> return thread1 == thread2;
> }
> -strong_alias (__pthread_equal, pthread_equal)
> +weak_alias (__pthread_equal, pthread_equal)
> diff --git a/nptl/pthread_exit.c b/nptl/pthread_exit.c
> index dffab09..7209769 100644
> --- a/nptl/pthread_exit.c
> +++ b/nptl/pthread_exit.c
> @@ -27,8 +27,8 @@ __pthread_exit (void *value)
>
> __do_cancel ();
> }
> -strong_alias (__pthread_exit, pthread_exit)
> +weak_alias (__pthread_exit, pthread_exit)
>
> /* After a thread terminates, __libc_start_main decrements
> __nptl_nthreads defined in pthread_create.c. */
> -PTHREAD_STATIC_FN_REQUIRE (pthread_create)
> +PTHREAD_STATIC_FN_REQUIRE (__pthread_create)
> diff --git a/nptl/pthread_getspecific.c b/nptl/pthread_getspecific.c
> index ddedcf2..114d6da 100644
> --- a/nptl/pthread_getspecific.c
> +++ b/nptl/pthread_getspecific.c
> @@ -63,5 +63,5 @@ __pthread_getspecific (pthread_key_t key)
>
> return result;
> }
> -strong_alias (__pthread_getspecific, pthread_getspecific)
> +weak_alias (__pthread_getspecific, pthread_getspecific)
> hidden_def (__pthread_getspecific)
> diff --git a/nptl/pthread_join.c b/nptl/pthread_join.c
> index 0192f69..afc8c37 100644
> --- a/nptl/pthread_join.c
> +++ b/nptl/pthread_join.c
> @@ -37,7 +37,7 @@ cleanup (void *arg)
>
>
> int
> -pthread_join (pthread_t threadid, void **thread_return)
> +__pthread_join (pthread_t threadid, void **thread_return)
> {
> struct pthread *pd = (struct pthread *) threadid;
>
> @@ -115,3 +115,4 @@ pthread_join (pthread_t threadid, void **thread_return)
>
> return result;
> }
> +weak_alias (__pthread_join, pthread_join)
> diff --git a/nptl/pthread_key_create.c b/nptl/pthread_key_create.c
> index 4f51c76..70c0e12 100644
> --- a/nptl/pthread_key_create.c
> +++ b/nptl/pthread_key_create.c
> @@ -47,5 +47,5 @@ __pthread_key_create (pthread_key_t *key, void (*destr) (void *))
>
> return EAGAIN;
> }
> -strong_alias (__pthread_key_create, pthread_key_create)
> +weak_alias (__pthread_key_create, pthread_key_create)
> hidden_def (__pthread_key_create)
> diff --git a/nptl/pthread_key_delete.c b/nptl/pthread_key_delete.c
> index 605f93b..426163d 100644
> --- a/nptl/pthread_key_delete.c
> +++ b/nptl/pthread_key_delete.c
> @@ -22,7 +22,7 @@
>
>
> int
> -pthread_key_delete (pthread_key_t key)
> +__pthread_key_delete (pthread_key_t key)
> {
> int result = EINVAL;
>
> @@ -39,3 +39,4 @@ pthread_key_delete (pthread_key_t key)
>
> return result;
> }
> +weak_alias (__pthread_key_delete, pthread_key_delete)
> diff --git a/nptl/pthread_mutex_destroy.c b/nptl/pthread_mutex_destroy.c
> index a300832..ed3b63f 100644
> --- a/nptl/pthread_mutex_destroy.c
> +++ b/nptl/pthread_mutex_destroy.c
> @@ -36,5 +36,5 @@ __pthread_mutex_destroy (pthread_mutex_t *mutex)
>
> return 0;
> }
> -strong_alias (__pthread_mutex_destroy, pthread_mutex_destroy)
> +weak_alias (__pthread_mutex_destroy, pthread_mutex_destroy)
> hidden_def (__pthread_mutex_destroy)
> diff --git a/nptl/pthread_mutex_init.c b/nptl/pthread_mutex_init.c
> index 138e144..6f2fc80 100644
> --- a/nptl/pthread_mutex_init.c
> +++ b/nptl/pthread_mutex_init.c
> @@ -144,5 +144,5 @@ __pthread_mutex_init (pthread_mutex_t *mutex,
>
> return 0;
> }
> -strong_alias (__pthread_mutex_init, pthread_mutex_init)
> +weak_alias (__pthread_mutex_init, pthread_mutex_init)
> hidden_def (__pthread_mutex_init)
> diff --git a/nptl/pthread_mutex_lock.c b/nptl/pthread_mutex_lock.c
> index dc9ca4c..b76475b 100644
> --- a/nptl/pthread_mutex_lock.c
> +++ b/nptl/pthread_mutex_lock.c
> @@ -597,7 +597,7 @@ __pthread_mutex_lock_full (pthread_mutex_t *mutex)
> return 0;
> }
> #ifndef __pthread_mutex_lock
> -strong_alias (__pthread_mutex_lock, pthread_mutex_lock)
> +weak_alias (__pthread_mutex_lock, pthread_mutex_lock)
> hidden_def (__pthread_mutex_lock)
> #endif
>
> diff --git a/nptl/pthread_mutex_timedlock.c b/nptl/pthread_mutex_timedlock.c
> index a4beb7b..be53381 100644
> --- a/nptl/pthread_mutex_timedlock.c
> +++ b/nptl/pthread_mutex_timedlock.c
> @@ -41,8 +41,8 @@
> #endif
>
> int
> -pthread_mutex_timedlock (pthread_mutex_t *mutex,
> - const struct timespec *abstime)
> +__pthread_mutex_timedlock (pthread_mutex_t *mutex,
> + const struct timespec *abstime)
> {
> int oldval;
> pid_t id = THREAD_GETMEM (THREAD_SELF, tid);
> @@ -634,3 +634,4 @@ pthread_mutex_timedlock (pthread_mutex_t *mutex,
> out:
> return result;
> }
> +weak_alias (__pthread_mutex_timedlock, pthread_mutex_timedlock)
> diff --git a/nptl/pthread_mutex_trylock.c b/nptl/pthread_mutex_trylock.c
> index e514997..ec7da61 100644
> --- a/nptl/pthread_mutex_trylock.c
> +++ b/nptl/pthread_mutex_trylock.c
> @@ -403,6 +403,7 @@ __pthread_mutex_trylock (pthread_mutex_t *mutex)
>
> #ifndef __pthread_mutex_trylock
> #ifndef pthread_mutex_trylock
> -strong_alias (__pthread_mutex_trylock, pthread_mutex_trylock)
> +weak_alias (__pthread_mutex_trylock, pthread_mutex_trylock)
> +hidden_def (__pthread_mutex_trylock)
> #endif
> #endif
> diff --git a/nptl/pthread_mutex_unlock.c b/nptl/pthread_mutex_unlock.c
> index f701d4e..e1a8a5c 100644
> --- a/nptl/pthread_mutex_unlock.c
> +++ b/nptl/pthread_mutex_unlock.c
> @@ -346,5 +346,5 @@ __pthread_mutex_unlock (pthread_mutex_t *mutex)
> {
> return __pthread_mutex_unlock_usercnt (mutex, 1);
> }
> -strong_alias (__pthread_mutex_unlock, pthread_mutex_unlock)
> +weak_alias (__pthread_mutex_unlock, pthread_mutex_unlock)
> hidden_def (__pthread_mutex_unlock)
> diff --git a/nptl/pthread_mutexattr_init.c b/nptl/pthread_mutexattr_init.c
> index 210d490..dcad522 100644
> --- a/nptl/pthread_mutexattr_init.c
> +++ b/nptl/pthread_mutexattr_init.c
> @@ -33,4 +33,5 @@ __pthread_mutexattr_init (pthread_mutexattr_t *attr)
>
> return 0;
> }
> -strong_alias (__pthread_mutexattr_init, pthread_mutexattr_init)
> +weak_alias (__pthread_mutexattr_init, pthread_mutexattr_init)
> +hidden_def (__pthread_mutexattr_init)
> diff --git a/nptl/pthread_mutexattr_settype.c b/nptl/pthread_mutexattr_settype.c
> index 8c4d11f..988793b 100644
> --- a/nptl/pthread_mutexattr_settype.c
> +++ b/nptl/pthread_mutexattr_settype.c
> @@ -40,4 +40,5 @@ __pthread_mutexattr_settype (pthread_mutexattr_t *attr, int kind)
> return 0;
> }
> weak_alias (__pthread_mutexattr_settype, pthread_mutexattr_setkind_np)
> -strong_alias (__pthread_mutexattr_settype, pthread_mutexattr_settype)
> +weak_alias (__pthread_mutexattr_settype, pthread_mutexattr_settype)
> +hidden_def (__pthread_mutexattr_settype)
> diff --git a/nptl/pthread_self.c b/nptl/pthread_self.c
> index 4c978e1..8e21775 100644
> --- a/nptl/pthread_self.c
> +++ b/nptl/pthread_self.c
> @@ -25,4 +25,4 @@ __pthread_self (void)
> {
> return (pthread_t) THREAD_SELF;
> }
> -strong_alias (__pthread_self, pthread_self)
> +weak_alias (__pthread_self, pthread_self)
> diff --git a/nptl/pthread_setspecific.c b/nptl/pthread_setspecific.c
> index c5416a5..214af3b 100644
> --- a/nptl/pthread_setspecific.c
> +++ b/nptl/pthread_setspecific.c
> @@ -89,5 +89,5 @@ __pthread_setspecific (pthread_key_t key, const void *value)
>
> return 0;
> }
> -strong_alias (__pthread_setspecific, pthread_setspecific)
> +weak_alias (__pthread_setspecific, pthread_setspecific)
> hidden_def (__pthread_setspecific)
> diff --git a/nptl/tpp.c b/nptl/tpp.c
> index 57eb026..7eb2b96 100644
> --- a/nptl/tpp.c
> +++ b/nptl/tpp.c
> @@ -43,9 +43,9 @@ void
> __init_sched_fifo_prio (void)
> {
> atomic_store_relaxed (&__sched_fifo_max_prio,
> - sched_get_priority_max (SCHED_FIFO));
> + __sched_get_priority_max (SCHED_FIFO));
> atomic_store_relaxed (&__sched_fifo_min_prio,
> - sched_get_priority_min (SCHED_FIFO));
> + __sched_get_priority_min (SCHED_FIFO));
> }
>
> int
> diff --git a/posix/nanosleep.c b/posix/nanosleep.c
> index 60a93ca..dbda103 100644
> --- a/posix/nanosleep.c
> +++ b/posix/nanosleep.c
> @@ -29,5 +29,5 @@ __nanosleep (const struct timespec *requested_time,
> }
> stub_warning (nanosleep)
>
> -libc_hidden_def (__nanosleep)
> +hidden_def (__nanosleep)
> weak_alias (__nanosleep, nanosleep)
> diff --git a/sysdeps/posix/gethostname.c b/sysdeps/posix/gethostname.c
> index 03a5d3f..a132482 100644
> --- a/sysdeps/posix/gethostname.c
> +++ b/sysdeps/posix/gethostname.c
> @@ -29,7 +29,7 @@ __gethostname (char *name, size_t len)
> struct utsname buf;
> size_t node_len;
>
> - if (uname (&buf))
> + if (__uname (&buf))
> return -1;
>
> node_len = strlen (buf.nodename) + 1;
> diff --git a/sysdeps/unix/sysv/linux/i386/Versions b/sysdeps/unix/sysv/linux/i386/Versions
> index f3544ac..b59ace4 100644
> --- a/sysdeps/unix/sysv/linux/i386/Versions
> +++ b/sysdeps/unix/sysv/linux/i386/Versions
> @@ -46,6 +46,6 @@ libc {
> fallocate64;
> }
> GLIBC_PRIVATE {
> - __modify_ldt;
> + __modify_ldt; __uname;
> }
> }
> diff --git a/sysdeps/unix/sysv/linux/i386/smp.h b/sysdeps/unix/sysv/linux/i386/smp.h
> index 7d1a7f2..3109ea9 100644
> --- a/sysdeps/unix/sysv/linux/i386/smp.h
> +++ b/sysdeps/unix/sysv/linux/i386/smp.h
> @@ -36,7 +36,7 @@ is_smp_system (void)
> char *cp;
>
> /* Try reading the number using `sysctl' first. */
> - if (uname (&u.uts) == 0)
> + if (__uname (&u.uts) == 0)
> cp = u.uts.version;
> else
> {
> diff --git a/sysdeps/unix/sysv/linux/mmap.c b/sysdeps/unix/sysv/linux/mmap.c
> index b1f229f..98c2f88 100644
> --- a/sysdeps/unix/sysv/linux/mmap.c
> +++ b/sysdeps/unix/sysv/linux/mmap.c
> @@ -47,5 +47,6 @@ __mmap (void *addr, size_t len, int prot, int flags, int fd, off_t offset)
> #endif
> }
> weak_alias (__mmap, mmap)
> +libc_hidden_def (__mmap)
>
> #endif /* __OFF_T_MATCHES_OFF64_T */
> diff --git a/sysdeps/unix/sysv/linux/mmap64.c b/sysdeps/unix/sysv/linux/mmap64.c
> index d3ea279..e8d519b 100644
> --- a/sysdeps/unix/sysv/linux/mmap64.c
> +++ b/sysdeps/unix/sysv/linux/mmap64.c
> @@ -53,8 +53,10 @@ __mmap64 (void *addr, size_t len, int prot, int flags, int fd, off64_t offset)
> #endif
> }
> weak_alias (__mmap64, mmap64)
> +libc_hidden_def (__mmap64)
>
> #ifdef __OFF_T_MATCHES_OFF64_T
> weak_alias (__mmap64, mmap)
> weak_alias (__mmap64, __mmap)
> +libc_hidden_def (__mmap)
> #endif
> diff --git a/sysdeps/unix/sysv/linux/nanosleep.c b/sysdeps/unix/sysv/linux/nanosleep.c
> index b352f84..2d15328 100644
> --- a/sysdeps/unix/sysv/linux/nanosleep.c
> +++ b/sysdeps/unix/sysv/linux/nanosleep.c
> @@ -26,5 +26,5 @@ __nanosleep (const struct timespec *requested_time,
> {
> return SYSCALL_CANCEL (nanosleep, requested_time, remaining);
> }
> -libc_hidden_def (__nanosleep)
> +hidden_def (__nanosleep)
> weak_alias (__nanosleep, nanosleep)
> diff --git a/sysdeps/unix/sysv/linux/powerpc/ioctl.c b/sysdeps/unix/sysv/linux/powerpc/ioctl.c
> index e2e3d33..1437d1d 100644
> --- a/sysdeps/unix/sysv/linux/powerpc/ioctl.c
> +++ b/sysdeps/unix/sysv/linux/powerpc/ioctl.c
> @@ -41,15 +41,15 @@ __ioctl (int fd, unsigned long int request, ...)
> break;
>
> case TCSETS:
> - result = tcsetattr (fd, TCSANOW, (struct termios *) arg);
> + result = __tcsetattr (fd, TCSANOW, (struct termios *) arg);
> break;
>
> case TCSETSW:
> - result = tcsetattr (fd, TCSADRAIN, (struct termios *) arg);
> + result = __tcsetattr (fd, TCSADRAIN, (struct termios *) arg);
> break;
>
> case TCSETSF:
> - result = tcsetattr (fd, TCSAFLUSH, (struct termios *) arg);
> + result = __tcsetattr (fd, TCSAFLUSH, (struct termios *) arg);
> break;
>
> default:
> diff --git a/sysdeps/unix/sysv/linux/tcsetattr.c b/sysdeps/unix/sysv/linux/tcsetattr.c
> index c7d1d65..49acd16 100644
> --- a/sysdeps/unix/sysv/linux/tcsetattr.c
> +++ b/sysdeps/unix/sysv/linux/tcsetattr.c
> @@ -41,7 +41,7 @@
>
> /* Set the state of FD to *TERMIOS_P. */
> int
> -tcsetattr (int fd, int optional_actions, const struct termios *termios_p)
> +__tcsetattr (int fd, int optional_actions, const struct termios *termios_p)
> {
> struct __kernel_termios k_termios;
> unsigned long int cmd;
> @@ -77,4 +77,5 @@ tcsetattr (int fd, int optional_actions, const struct termios *termios_p)
>
> return INLINE_SYSCALL (ioctl, 3, fd, cmd, &k_termios);
> }
> +weak_alias (__tcsetattr, tcsetattr)
> libc_hidden_def (tcsetattr)
>
On 22/06/2017 16:35, Siddhesh Poyarekar wrote:
> On Tuesday 20 June 2017 12:22 AM, Adhemerval Zanella wrote:
>> This patch adds internal definition (through {libc_}hidden_{proto,def}) and
>> also change some strong to weak alias for symbols that might be used by C11
>> threads implementations.
>>
>> The patchset should not change libc/libpthread functional, although object
>> changes are expected (since now internal symbols are used instead) and final
>> exported symbols through GLIBC_PRIVATE is also expanded (to cover libpthread
>> usage of __mmap{64}, __munmap, __mprotect).
>>
>> Checked with a build for all major ABI (aarch64-linux-gnu, alpha-linux-gnu,
>> arm-linux-gnueabi, i386-linux-gnu, ia64-linux-gnu, m68k-linux-gnu,
>> microblaze-linux-gnu [1], mips{64}-linux-gnu, nios2-linux-gnu,
>> powerpc{64le}-linux-gnu, s390{x}-linux-gnu, sparc{64}-linux-gnu,
>> tile{pro,gx}-linux-gnu, and x86_64-linux-gnu).
>
> Looks OK with one small nit/comment below.
Thanks, I will push it upstream and split the C11 patch in small parts.
>> diff --git a/include/termios.h b/include/termios.h
>> index fad51f8..1a36e22 100644
>> --- a/include/termios.h
>> +++ b/include/termios.h
>> @@ -4,6 +4,8 @@
>> #ifndef _ISOMAC
>> /* Now define the internal interfaces. */
>> extern int __tcgetattr (int __fd, struct termios *__termios_p);
>> +extern int __tcsetattr (int __fd, int __optional_actions,
>> + const struct termios *__termios_p);
>>
>> extern int __libc_tcdrain (int __fd);
>>
>> diff --git a/include/time.h b/include/time.h
>> index 3a828e0..b72a3a3 100644
>> --- a/include/time.h
>> +++ b/include/time.h
>> @@ -78,7 +78,7 @@ extern struct tm *__tz_convert (const time_t *timer, int use_localtime, struct t
>>
>> extern int __nanosleep (const struct timespec *__requested_time,
>> struct timespec *__remaining);
>> -libc_hidden_proto (__nanosleep)
>> +hidden_proto (__nanosleep)
>
> Wouldn't it be more correct to define libc_hidden_* for IS_IN(libc) as
> well as IS_IN(pthread) in libc-symbols.h? Basically just enable usage
> of all of the libc_* macros there for libc as well as libpthread.
It could be case, but then I think the 'libc_' will be misleading. I would
rather to just change to hidden_{proto,def} to symbol built on both
libc and libpthread.
@@ -13,7 +13,9 @@ extern int __sched_getscheduler (__pid_t __pid);
extern int __sched_yield (void);
libc_hidden_proto (__sched_yield)
extern int __sched_get_priority_max (int __algorithm);
+libc_hidden_proto (__sched_get_priority_max)
extern int __sched_get_priority_min (int __algorithm);
+libc_hidden_proto (__sched_get_priority_min)
extern int __sched_rr_get_interval (__pid_t __pid, struct timespec *__t);
/* These are Linux specific. */
@@ -5,10 +5,14 @@
/* Now define the internal interfaces. */
extern void *__mmap (void *__addr, size_t __len, int __prot,
int __flags, int __fd, __off_t __offset);
+libc_hidden_proto (__mmap)
extern void *__mmap64 (void *__addr, size_t __len, int __prot,
int __flags, int __fd, __off64_t __offset);
+libc_hidden_proto (__mmap64)
extern int __munmap (void *__addr, size_t __len);
+libc_hidden_proto (__munmap)
extern int __mprotect (void *__addr, size_t __len, int __prot);
+libc_hidden_proto (__mprotect)
extern int __madvise (void *__addr, size_t __len, int __advice);
libc_hidden_proto (__madvise)
@@ -4,6 +4,8 @@
#ifndef _ISOMAC
/* Now define the internal interfaces. */
extern int __tcgetattr (int __fd, struct termios *__termios_p);
+extern int __tcsetattr (int __fd, int __optional_actions,
+ const struct termios *__termios_p);
extern int __libc_tcdrain (int __fd);
@@ -78,7 +78,7 @@ extern struct tm *__tz_convert (const time_t *timer, int use_localtime, struct t
extern int __nanosleep (const struct timespec *__requested_time,
struct timespec *__remaining);
-libc_hidden_proto (__nanosleep)
+hidden_proto (__nanosleep)
extern int __nanosleep_nocancel (const struct timespec *__requested_time,
struct timespec *__remaining)
attribute_hidden;
@@ -163,5 +163,7 @@ libc {
__mktemp;
__libc_ifunc_impl_list;
__tdelete; __tfind; __tsearch; __twalk;
+ __mmap; __munmap; __mprotect;
+ __sched_get_priority_min; __sched_get_priority_max;
}
}
@@ -37,3 +37,4 @@ __mmap (__ptr_t addr, size_t len, int prot, int flags, int fd, off_t offset)
stub_warning (mmap)
weak_alias (__mmap, mmap)
+libc_hidden_def (__mmap)
@@ -45,3 +45,4 @@ __mmap64 (__ptr_t addr, size_t len, int prot, int flags, int fd,
}
weak_alias (__mmap64, mmap64)
+libc_hidden_def (__mmap64)
@@ -278,7 +278,7 @@ __free_stacks (size_t limit)
/* Remove this block. This should never fail. If it does
something is really wrong. */
- if (munmap (curr->stackblock, curr->stackblock_size) != 0)
+ if (__munmap (curr->stackblock, curr->stackblock_size) != 0)
abort ();
/* Maybe we have freed enough. */
@@ -328,7 +328,7 @@ change_stack_perm (struct pthread *pd
#else
# error "Define either _STACK_GROWS_DOWN or _STACK_GROWS_UP"
#endif
- if (mprotect (stack, len, PROT_READ | PROT_WRITE | PROT_EXEC) != 0)
+ if (__mprotect (stack, len, PROT_READ | PROT_WRITE | PROT_EXEC) != 0)
return errno;
return 0;
@@ -359,14 +359,14 @@ setup_stack_prot (char *mem, size_t size, char *guard, size_t guardsize,
#if _STACK_GROWS_DOWN
/* As defined at guard_position, for architectures with downward stack
the guard page is always at start of the allocated area. */
- if (mprotect (guardend, size - guardsize, prot) != 0)
+ if (__mprotect (guardend, size - guardsize, prot) != 0)
return errno;
#else
size_t mprots1 = (uintptr_t) guard - (uintptr_t) mem;
- if (mprotect (mem, mprots1, prot) != 0)
+ if (__mprotect (mem, mprots1, prot) != 0)
return errno;
size_t mprots2 = ((uintptr_t) mem + size) - (uintptr_t) guardend;
- if (mprotect (guardend, mprots2, prot) != 0)
+ if (__mprotect (guardend, mprots2, prot) != 0)
return errno;
#endif
return 0;
@@ -530,8 +530,8 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
/* If a guard page is required, avoid committing memory by first
allocate with PROT_NONE and then reserve with required permission
excluding the guard page. */
- mem = mmap (NULL, size, (guardsize == 0) ? prot : PROT_NONE,
- MAP_PRIVATE | MAP_ANONYMOUS | MAP_STACK, -1, 0);
+ mem = __mmap (NULL, size, (guardsize == 0) ? prot : PROT_NONE,
+ MAP_PRIVATE | MAP_ANONYMOUS | MAP_STACK, -1, 0);
if (__glibc_unlikely (mem == MAP_FAILED))
return errno;
@@ -557,7 +557,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
pagesize_m1);
if (setup_stack_prot (mem, size, guard, guardsize, prot) != 0)
{
- munmap (mem, size);
+ __munmap (mem, size);
return errno;
}
}
@@ -600,7 +600,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
assert (errno == ENOMEM);
/* Free the stack memory we just allocated. */
- (void) munmap (mem, size);
+ (void) __munmap (mem, size);
return errno;
}
@@ -630,7 +630,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
if (err != 0)
{
/* Free the stack memory we just allocated. */
- (void) munmap (mem, size);
+ (void) __munmap (mem, size);
return err;
}
@@ -650,7 +650,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
{
char *guard = guard_position (mem, size, guardsize, pd,
pagesize_m1);
- if (mprotect (guard, guardsize, PROT_NONE) != 0)
+ if (__mprotect (guard, guardsize, PROT_NONE) != 0)
{
mprot_error:
lll_lock (stack_cache_lock, LLL_PRIVATE);
@@ -668,7 +668,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
of memory caused problems we better do not use it
anymore. Uh, and we ignore possible errors. There
is nothing we could do. */
- (void) munmap (mem, size);
+ (void) __munmap (mem, size);
return errno;
}
@@ -685,19 +685,19 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
char *oldguard = mem + (((size - pd->guardsize) / 2) & ~pagesize_m1);
if (oldguard < guard
- && mprotect (oldguard, guard - oldguard, prot) != 0)
+ && __mprotect (oldguard, guard - oldguard, prot) != 0)
goto mprot_error;
- if (mprotect (guard + guardsize,
+ if (__mprotect (guard + guardsize,
oldguard + pd->guardsize - guard - guardsize,
prot) != 0)
goto mprot_error;
#elif _STACK_GROWS_DOWN
- if (mprotect ((char *) mem + guardsize, pd->guardsize - guardsize,
+ if (__mprotect ((char *) mem + guardsize, pd->guardsize - guardsize,
prot) != 0)
goto mprot_error;
#elif _STACK_GROWS_UP
- if (mprotect ((char *) pd - pd->guardsize,
+ if (__mprotect ((char *) pd - pd->guardsize,
pd->guardsize - guardsize, prot) != 0)
goto mprot_error;
#endif
@@ -428,6 +428,8 @@ extern int __pthread_mutex_init (pthread_mutex_t *__mutex,
extern int __pthread_mutex_destroy (pthread_mutex_t *__mutex);
extern int __pthread_mutex_trylock (pthread_mutex_t *_mutex);
extern int __pthread_mutex_lock (pthread_mutex_t *__mutex);
+extern int __pthread_mutex_timedlock (pthread_mutex_t *__mutex,
+ const struct timespec *__abstime);
extern int __pthread_mutex_cond_lock (pthread_mutex_t *__mutex)
attribute_hidden internal_function;
extern void __pthread_mutex_cond_lock_adjust (pthread_mutex_t *__mutex)
@@ -491,6 +493,7 @@ extern int __pthread_cond_timedwait (pthread_cond_t *cond,
extern int __pthread_condattr_destroy (pthread_condattr_t *attr);
extern int __pthread_condattr_init (pthread_condattr_t *attr);
extern int __pthread_key_create (pthread_key_t *key, void (*destr) (void *));
+extern int __pthread_key_delete (pthread_key_t key);
extern void *__pthread_getspecific (pthread_key_t key);
extern int __pthread_setspecific (pthread_key_t key, const void *value);
extern int __pthread_once (pthread_once_t *once_control,
@@ -499,8 +502,11 @@ extern int __pthread_atfork (void (*prepare) (void), void (*parent) (void),
void (*child) (void));
extern pthread_t __pthread_self (void);
extern int __pthread_equal (pthread_t thread1, pthread_t thread2);
+extern int __pthread_detach (pthread_t th);
+extern int __pthread_cancel (pthread_t th);
extern int __pthread_kill (pthread_t threadid, int signo);
extern void __pthread_exit (void *value) __attribute__ ((__noreturn__));
+extern int __pthread_join (pthread_t threadid, void **thread_return);
extern int __pthread_setcanceltype (int type, int *oldtype);
extern int __pthread_enable_asynccancel (void) attribute_hidden;
extern void __pthread_disable_asynccancel (int oldtype)
@@ -511,6 +517,7 @@ extern void __pthread_testcancel (void);
hidden_proto (__pthread_mutex_init)
hidden_proto (__pthread_mutex_destroy)
hidden_proto (__pthread_mutex_lock)
+hidden_proto (__pthread_mutex_trylock)
hidden_proto (__pthread_mutex_unlock)
hidden_proto (__pthread_rwlock_rdlock)
hidden_proto (__pthread_rwlock_wrlock)
@@ -521,6 +528,8 @@ hidden_proto (__pthread_setspecific)
hidden_proto (__pthread_once)
hidden_proto (__pthread_setcancelstate)
hidden_proto (__pthread_testcancel)
+hidden_proto (__pthread_mutexattr_init)
+hidden_proto (__pthread_mutexattr_settype)
#endif
extern int __pthread_cond_broadcast_2_0 (pthread_cond_2_0_t *cond);
@@ -25,7 +25,7 @@
#include <unistd.h>
int
-pthread_cancel (pthread_t th)
+__pthread_cancel (pthread_t th)
{
volatile struct pthread *pd = (volatile struct pthread *) th;
@@ -66,7 +66,7 @@ pthread_cancel (pthread_t th)
#ifdef SIGCANCEL
/* The cancellation handler will take care of marking the
thread as canceled. */
- pid_t pid = getpid ();
+ pid_t pid = __getpid ();
INTERNAL_SYSCALL_DECL (err);
int val = INTERNAL_SYSCALL_CALL (tgkill, err, pid, pd->tid,
@@ -99,5 +99,6 @@ pthread_cancel (pthread_t th)
return result;
}
+weak_alias (__pthread_cancel, pthread_cancel)
-PTHREAD_STATIC_FN_REQUIRE (pthread_create)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_create)
@@ -920,14 +920,14 @@ compat_symbol (libpthread, __pthread_create_2_0, pthread_create,
/* If pthread_create is present, libgcc_eh.a and libsupc++.a expects some other POSIX thread
functions to be present as well. */
-PTHREAD_STATIC_FN_REQUIRE (pthread_mutex_lock)
-PTHREAD_STATIC_FN_REQUIRE (pthread_mutex_trylock)
-PTHREAD_STATIC_FN_REQUIRE (pthread_mutex_unlock)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_mutex_lock)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_mutex_trylock)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_mutex_unlock)
-PTHREAD_STATIC_FN_REQUIRE (pthread_once)
-PTHREAD_STATIC_FN_REQUIRE (pthread_cancel)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_once)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_cancel)
-PTHREAD_STATIC_FN_REQUIRE (pthread_key_create)
-PTHREAD_STATIC_FN_REQUIRE (pthread_key_delete)
-PTHREAD_STATIC_FN_REQUIRE (pthread_setspecific)
-PTHREAD_STATIC_FN_REQUIRE (pthread_getspecific)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_key_create)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_key_delete)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_setspecific)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_getspecific)
@@ -22,7 +22,7 @@
int
-pthread_detach (pthread_t th)
+__pthread_detach (pthread_t th)
{
struct pthread *pd = (struct pthread *) th;
@@ -53,3 +53,4 @@ pthread_detach (pthread_t th)
return result;
}
+weak_alias (__pthread_detach, pthread_detach)
@@ -24,4 +24,4 @@ __pthread_equal (pthread_t thread1, pthread_t thread2)
{
return thread1 == thread2;
}
-strong_alias (__pthread_equal, pthread_equal)
+weak_alias (__pthread_equal, pthread_equal)
@@ -27,8 +27,8 @@ __pthread_exit (void *value)
__do_cancel ();
}
-strong_alias (__pthread_exit, pthread_exit)
+weak_alias (__pthread_exit, pthread_exit)
/* After a thread terminates, __libc_start_main decrements
__nptl_nthreads defined in pthread_create.c. */
-PTHREAD_STATIC_FN_REQUIRE (pthread_create)
+PTHREAD_STATIC_FN_REQUIRE (__pthread_create)
@@ -63,5 +63,5 @@ __pthread_getspecific (pthread_key_t key)
return result;
}
-strong_alias (__pthread_getspecific, pthread_getspecific)
+weak_alias (__pthread_getspecific, pthread_getspecific)
hidden_def (__pthread_getspecific)
@@ -37,7 +37,7 @@ cleanup (void *arg)
int
-pthread_join (pthread_t threadid, void **thread_return)
+__pthread_join (pthread_t threadid, void **thread_return)
{
struct pthread *pd = (struct pthread *) threadid;
@@ -115,3 +115,4 @@ pthread_join (pthread_t threadid, void **thread_return)
return result;
}
+weak_alias (__pthread_join, pthread_join)
@@ -47,5 +47,5 @@ __pthread_key_create (pthread_key_t *key, void (*destr) (void *))
return EAGAIN;
}
-strong_alias (__pthread_key_create, pthread_key_create)
+weak_alias (__pthread_key_create, pthread_key_create)
hidden_def (__pthread_key_create)
@@ -22,7 +22,7 @@
int
-pthread_key_delete (pthread_key_t key)
+__pthread_key_delete (pthread_key_t key)
{
int result = EINVAL;
@@ -39,3 +39,4 @@ pthread_key_delete (pthread_key_t key)
return result;
}
+weak_alias (__pthread_key_delete, pthread_key_delete)
@@ -36,5 +36,5 @@ __pthread_mutex_destroy (pthread_mutex_t *mutex)
return 0;
}
-strong_alias (__pthread_mutex_destroy, pthread_mutex_destroy)
+weak_alias (__pthread_mutex_destroy, pthread_mutex_destroy)
hidden_def (__pthread_mutex_destroy)
@@ -144,5 +144,5 @@ __pthread_mutex_init (pthread_mutex_t *mutex,
return 0;
}
-strong_alias (__pthread_mutex_init, pthread_mutex_init)
+weak_alias (__pthread_mutex_init, pthread_mutex_init)
hidden_def (__pthread_mutex_init)
@@ -597,7 +597,7 @@ __pthread_mutex_lock_full (pthread_mutex_t *mutex)
return 0;
}
#ifndef __pthread_mutex_lock
-strong_alias (__pthread_mutex_lock, pthread_mutex_lock)
+weak_alias (__pthread_mutex_lock, pthread_mutex_lock)
hidden_def (__pthread_mutex_lock)
#endif
@@ -41,8 +41,8 @@
#endif
int
-pthread_mutex_timedlock (pthread_mutex_t *mutex,
- const struct timespec *abstime)
+__pthread_mutex_timedlock (pthread_mutex_t *mutex,
+ const struct timespec *abstime)
{
int oldval;
pid_t id = THREAD_GETMEM (THREAD_SELF, tid);
@@ -634,3 +634,4 @@ pthread_mutex_timedlock (pthread_mutex_t *mutex,
out:
return result;
}
+weak_alias (__pthread_mutex_timedlock, pthread_mutex_timedlock)
@@ -403,6 +403,7 @@ __pthread_mutex_trylock (pthread_mutex_t *mutex)
#ifndef __pthread_mutex_trylock
#ifndef pthread_mutex_trylock
-strong_alias (__pthread_mutex_trylock, pthread_mutex_trylock)
+weak_alias (__pthread_mutex_trylock, pthread_mutex_trylock)
+hidden_def (__pthread_mutex_trylock)
#endif
#endif
@@ -346,5 +346,5 @@ __pthread_mutex_unlock (pthread_mutex_t *mutex)
{
return __pthread_mutex_unlock_usercnt (mutex, 1);
}
-strong_alias (__pthread_mutex_unlock, pthread_mutex_unlock)
+weak_alias (__pthread_mutex_unlock, pthread_mutex_unlock)
hidden_def (__pthread_mutex_unlock)
@@ -33,4 +33,5 @@ __pthread_mutexattr_init (pthread_mutexattr_t *attr)
return 0;
}
-strong_alias (__pthread_mutexattr_init, pthread_mutexattr_init)
+weak_alias (__pthread_mutexattr_init, pthread_mutexattr_init)
+hidden_def (__pthread_mutexattr_init)
@@ -40,4 +40,5 @@ __pthread_mutexattr_settype (pthread_mutexattr_t *attr, int kind)
return 0;
}
weak_alias (__pthread_mutexattr_settype, pthread_mutexattr_setkind_np)
-strong_alias (__pthread_mutexattr_settype, pthread_mutexattr_settype)
+weak_alias (__pthread_mutexattr_settype, pthread_mutexattr_settype)
+hidden_def (__pthread_mutexattr_settype)
@@ -25,4 +25,4 @@ __pthread_self (void)
{
return (pthread_t) THREAD_SELF;
}
-strong_alias (__pthread_self, pthread_self)
+weak_alias (__pthread_self, pthread_self)
@@ -89,5 +89,5 @@ __pthread_setspecific (pthread_key_t key, const void *value)
return 0;
}
-strong_alias (__pthread_setspecific, pthread_setspecific)
+weak_alias (__pthread_setspecific, pthread_setspecific)
hidden_def (__pthread_setspecific)
@@ -43,9 +43,9 @@ void
__init_sched_fifo_prio (void)
{
atomic_store_relaxed (&__sched_fifo_max_prio,
- sched_get_priority_max (SCHED_FIFO));
+ __sched_get_priority_max (SCHED_FIFO));
atomic_store_relaxed (&__sched_fifo_min_prio,
- sched_get_priority_min (SCHED_FIFO));
+ __sched_get_priority_min (SCHED_FIFO));
}
int
@@ -29,5 +29,5 @@ __nanosleep (const struct timespec *requested_time,
}
stub_warning (nanosleep)
-libc_hidden_def (__nanosleep)
+hidden_def (__nanosleep)
weak_alias (__nanosleep, nanosleep)
@@ -29,7 +29,7 @@ __gethostname (char *name, size_t len)
struct utsname buf;
size_t node_len;
- if (uname (&buf))
+ if (__uname (&buf))
return -1;
node_len = strlen (buf.nodename) + 1;
@@ -46,6 +46,6 @@ libc {
fallocate64;
}
GLIBC_PRIVATE {
- __modify_ldt;
+ __modify_ldt; __uname;
}
}
@@ -36,7 +36,7 @@ is_smp_system (void)
char *cp;
/* Try reading the number using `sysctl' first. */
- if (uname (&u.uts) == 0)
+ if (__uname (&u.uts) == 0)
cp = u.uts.version;
else
{
@@ -47,5 +47,6 @@ __mmap (void *addr, size_t len, int prot, int flags, int fd, off_t offset)
#endif
}
weak_alias (__mmap, mmap)
+libc_hidden_def (__mmap)
#endif /* __OFF_T_MATCHES_OFF64_T */
@@ -53,8 +53,10 @@ __mmap64 (void *addr, size_t len, int prot, int flags, int fd, off64_t offset)
#endif
}
weak_alias (__mmap64, mmap64)
+libc_hidden_def (__mmap64)
#ifdef __OFF_T_MATCHES_OFF64_T
weak_alias (__mmap64, mmap)
weak_alias (__mmap64, __mmap)
+libc_hidden_def (__mmap)
#endif
@@ -26,5 +26,5 @@ __nanosleep (const struct timespec *requested_time,
{
return SYSCALL_CANCEL (nanosleep, requested_time, remaining);
}
-libc_hidden_def (__nanosleep)
+hidden_def (__nanosleep)
weak_alias (__nanosleep, nanosleep)
@@ -41,15 +41,15 @@ __ioctl (int fd, unsigned long int request, ...)
break;
case TCSETS:
- result = tcsetattr (fd, TCSANOW, (struct termios *) arg);
+ result = __tcsetattr (fd, TCSANOW, (struct termios *) arg);
break;
case TCSETSW:
- result = tcsetattr (fd, TCSADRAIN, (struct termios *) arg);
+ result = __tcsetattr (fd, TCSADRAIN, (struct termios *) arg);
break;
case TCSETSF:
- result = tcsetattr (fd, TCSAFLUSH, (struct termios *) arg);
+ result = __tcsetattr (fd, TCSAFLUSH, (struct termios *) arg);
break;
default:
@@ -41,7 +41,7 @@
/* Set the state of FD to *TERMIOS_P. */
int
-tcsetattr (int fd, int optional_actions, const struct termios *termios_p)
+__tcsetattr (int fd, int optional_actions, const struct termios *termios_p)
{
struct __kernel_termios k_termios;
unsigned long int cmd;
@@ -77,4 +77,5 @@ tcsetattr (int fd, int optional_actions, const struct termios *termios_p)
return INLINE_SYSCALL (ioctl, 3, fd, cmd, &k_termios);
}
+weak_alias (__tcsetattr, tcsetattr)
libc_hidden_def (tcsetattr)