This is the mail archive of the glibc-cvs@sourceware.org mailing list for the glibc project.
Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]
GNU C Library master sources branch master updated. glibc-2.21-487-g90dd591

From: jsm28 at sourceware dot org
To: glibc-cvs at sourceware dot org
Date: 17 Jun 2015 20:17:17 -0000
Subject: GNU C Library master sources branch master updated. glibc-2.21-487-g90dd591
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  90dd591393a03d023be06a33acfb42c1b328b0fa (commit)
      from  45dcd79f6eb37ad4e9e6967f1d459ffc3629eb1f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=90dd591393a03d023be06a33acfb42c1b328b0fa

commit 90dd591393a03d023be06a33acfb42c1b328b0fa
Author: Joseph Myers <joseph@codesourcery.com>
Date:   Wed Jun 17 20:16:56 2015 +0000

    Fix mq_notify pthread_barrier_* namespace (bug 18544).
    
    mq_notify (present in POSIX by 1996) brings in references to
    pthread_barrier_init and pthread_barrier_wait (new in the 2001 edition
    of POSIX).  This patch fixes this by making those functions into weak
    aliases of __pthread_barrier_*, exporting the __pthread_barrier_*
    names at version GLIBC_PRIVATE and using them in mq_notify.
    
    Tested for x86_64 and x86 (testsuite, and comparison of installed
    stripped shared libraries).  Changes in addresses from dynamic symbol
    table / PLT changes render most comparisons not particularly useful,
    but when the addresses of subsequent code don't change there's no sign
    of unexpected changes there.  This patch does not remove any
    linknamespace XFAILs because of other namespace issues remaining with
    mqueue.h functions.
    
    	[BZ #18544]
    	* nptl/pthread_barrier_init.c (pthread_barrier_init): Rename to
    	__pthread_barrier_init and define as weak alias of
    	__pthread_barrier_init.
    	* sysdeps/sparc/nptl/pthread_barrier_init.c
    	(pthread_barrier_init): Likewise.
    	* nptl/pthread_barrier_wait.c (pthread_barrier_wait): Rename to
    	__pthread_barrier_wait and define as weak alias of
    	__pthread_barrier_wait.
    	* sysdeps/sparc/nptl/pthread_barrier_wait.c
    	(pthread_barrier_wait): Likewise.
    	* sysdeps/sparc/sparc32/pthread_barrier_wait.c
    	(pthread_barrier_wait): Likewise.
    	* sysdeps/unix/sysv/linux/i386/i486/pthread_barrier_wait.S
    	(pthread_barrier_wait): Likewise.
    	* sysdeps/unix/sysv/linux/x86_64/pthread_barrier_wait.S
    	(pthread_barrier_wait): Likewise.
    	* nptl/Versions (libpthread): Export __pthread_barrier_init and
    	__pthread_barrier_wait at version GLIBC_PRIVATE.
    	* include/pthread.h (__pthread_barrier_init): Declare.
    	(__pthread_barrier_wait): Likewise.
    	* sysdeps/unix/sysv/linux/mq_notify.c (notification_function):
    	Call __pthread_barrier_wait instead of pthread_barrier_wait.
    	(helper_thread): Likewise.
    	(init_mq_netlink): Call __pthread_barrier_init instead of
    	pthread_barrier_init.

diff --git a/ChangeLog b/ChangeLog
index ea4aeeb..5b1226a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,32 @@
 2015-06-17  Joseph Myers  <joseph@codesourcery.com>
 
+	[BZ #18544]
+	* nptl/pthread_barrier_init.c (pthread_barrier_init): Rename to
+	__pthread_barrier_init and define as weak alias of
+	__pthread_barrier_init.
+	* sysdeps/sparc/nptl/pthread_barrier_init.c
+	(pthread_barrier_init): Likewise.
+	* nptl/pthread_barrier_wait.c (pthread_barrier_wait): Rename to
+	__pthread_barrier_wait and define as weak alias of
+	__pthread_barrier_wait.
+	* sysdeps/sparc/nptl/pthread_barrier_wait.c
+	(pthread_barrier_wait): Likewise.
+	* sysdeps/sparc/sparc32/pthread_barrier_wait.c
+	(pthread_barrier_wait): Likewise.
+	* sysdeps/unix/sysv/linux/i386/i486/pthread_barrier_wait.S
+	(pthread_barrier_wait): Likewise.
+	* sysdeps/unix/sysv/linux/x86_64/pthread_barrier_wait.S
+	(pthread_barrier_wait): Likewise.
+	* nptl/Versions (libpthread): Export __pthread_barrier_init and
+	__pthread_barrier_wait at version GLIBC_PRIVATE.
+	* include/pthread.h (__pthread_barrier_init): Declare.
+	(__pthread_barrier_wait): Likewise.
+	* sysdeps/unix/sysv/linux/mq_notify.c (notification_function):
+	Call __pthread_barrier_wait instead of pthread_barrier_wait.
+	(helper_thread): Likewise.
+	(init_mq_netlink): Call __pthread_barrier_init instead of
+	pthread_barrier_init.
+
 	[BZ #18542]
 	* libio/iovswscanf.c (__vswscanf): Use libc_hidden_def.
 	(vswscanf): Use ldbl_weak_alias instead of ldbl_strong_alias
diff --git a/NEWS b/NEWS
index d501b31..59586d4 100644
--- a/NEWS
+++ b/NEWS
@@ -22,7 +22,7 @@ Version 2.22
   18324, 18333, 18346, 18397, 18409, 18410, 18412, 18418, 18422, 18434,
   18444, 18468, 18469, 18470, 18479, 18483, 18495, 18496, 18497, 18498,
   18507, 18512, 18519, 18520, 18522, 18527, 18528, 18529, 18530, 18532,
-  18533, 18534, 18536, 18539, 18540, 18542.
+  18533, 18534, 18536, 18539, 18540, 18542, 18544.
 
 * Cache information can be queried via sysconf() function on s390 e.g. with
   _SC_LEVEL1_ICACHE_SIZE as argument.
diff --git a/include/pthread.h b/include/pthread.h
index 51854f6..858c869 100644
--- a/include/pthread.h
+++ b/include/pthread.h
@@ -1,6 +1,16 @@
 #include_next <pthread.h>
 
 #ifndef _ISOMAC
+/* Prototypes repeated instead of using __typeof because pthread.h is
+   included in C++ tests, and declaring functions with __typeof and
+   __THROW doesn't work for C++.  */
+extern int __pthread_barrier_init (pthread_barrier_t *__restrict __barrier,
+				 const pthread_barrierattr_t *__restrict
+				 __attr, unsigned int __count)
+     __THROW __nonnull ((1));
+extern int __pthread_barrier_wait (pthread_barrier_t *__barrier)
+     __THROWNL __nonnull ((1));
+
 /* This function is called to initialize the pthread library.  */
 extern void __pthread_initialize (void) __attribute__ ((weak));
 #endif
diff --git a/nptl/Versions b/nptl/Versions
index bc8a15b..34e4b46 100644
--- a/nptl/Versions
+++ b/nptl/Versions
@@ -273,6 +273,7 @@ libpthread {
     __pthread_initialize_minimal;
     __pthread_clock_gettime; __pthread_clock_settime;
     __pthread_unwind; __pthread_get_minstack;
+    __pthread_barrier_init; __pthread_barrier_wait;
     __shm_directory;
   }
 }
diff --git a/nptl/pthread_barrier_init.c b/nptl/pthread_barrier_init.c
index 82e13fb..5ea9fad 100644
--- a/nptl/pthread_barrier_init.c
+++ b/nptl/pthread_barrier_init.c
@@ -29,7 +29,7 @@ static const struct pthread_barrierattr default_barrierattr =
 
 
 int
-pthread_barrier_init (barrier, attr, count)
+__pthread_barrier_init (barrier, attr, count)
      pthread_barrier_t *barrier;
      const pthread_barrierattr_t *attr;
      unsigned int count;
@@ -68,3 +68,4 @@ pthread_barrier_init (barrier, attr, count)
 
   return 0;
 }
+weak_alias (__pthread_barrier_init, pthread_barrier_init)
diff --git a/nptl/pthread_barrier_wait.c b/nptl/pthread_barrier_wait.c
index 9e7090f..d69a929 100644
--- a/nptl/pthread_barrier_wait.c
+++ b/nptl/pthread_barrier_wait.c
@@ -24,7 +24,7 @@
 
 /* Wait on barrier.  */
 int
-pthread_barrier_wait (barrier)
+__pthread_barrier_wait (barrier)
      pthread_barrier_t *barrier;
 {
   struct pthread_barrier *ibarrier = (struct pthread_barrier *) barrier;
@@ -76,3 +76,4 @@ pthread_barrier_wait (barrier)
 
   return result;
 }
+weak_alias (__pthread_barrier_wait, pthread_barrier_wait)
diff --git a/sysdeps/sparc/nptl/pthread_barrier_init.c b/sysdeps/sparc/nptl/pthread_barrier_init.c
index 01bd18a..aa21a63 100644
--- a/sysdeps/sparc/nptl/pthread_barrier_init.c
+++ b/sysdeps/sparc/nptl/pthread_barrier_init.c
@@ -22,7 +22,7 @@
 #include <sparc-nptl.h>
 
 int
-pthread_barrier_init (barrier, attr, count)
+__pthread_barrier_init (barrier, attr, count)
      pthread_barrier_t *barrier;
      const pthread_barrierattr_t *attr;
      unsigned int count;
@@ -53,3 +53,4 @@ pthread_barrier_init (barrier, attr, count)
 
   return 0;
 }
+weak_alias (__pthread_barrier_init, pthread_barrier_init)
diff --git a/sysdeps/sparc/nptl/pthread_barrier_wait.c b/sysdeps/sparc/nptl/pthread_barrier_wait.c
index b0437a1..dd4c336 100644
--- a/sysdeps/sparc/nptl/pthread_barrier_wait.c
+++ b/sysdeps/sparc/nptl/pthread_barrier_wait.c
@@ -24,7 +24,7 @@
 
 /* Wait on barrier.  */
 int
-pthread_barrier_wait (barrier)
+__pthread_barrier_wait (barrier)
      pthread_barrier_t *barrier;
 {
   union sparc_pthread_barrier *ibarrier
@@ -76,3 +76,4 @@ pthread_barrier_wait (barrier)
 
   return result;
 }
+weak_alias (__pthread_barrier_wait, pthread_barrier_wait)
diff --git a/sysdeps/sparc/sparc32/pthread_barrier_wait.c b/sysdeps/sparc/sparc32/pthread_barrier_wait.c
index afab203..81d22b0 100644
--- a/sysdeps/sparc/sparc32/pthread_barrier_wait.c
+++ b/sysdeps/sparc/sparc32/pthread_barrier_wait.c
@@ -24,7 +24,7 @@
 
 /* Wait on barrier.  */
 int
-pthread_barrier_wait (barrier)
+__pthread_barrier_wait (barrier)
      pthread_barrier_t *barrier;
 {
   union sparc_pthread_barrier *ibarrier
@@ -92,3 +92,4 @@ pthread_barrier_wait (barrier)
 
   return result;
 }
+weak_alias (__pthread_barrier_wait, pthread_barrier_wait)
diff --git a/sysdeps/unix/sysv/linux/i386/i486/pthread_barrier_wait.S b/sysdeps/unix/sysv/linux/i386/i486/pthread_barrier_wait.S
index 1d8200f..ad1a774 100644
--- a/sysdeps/unix/sysv/linux/i386/i486/pthread_barrier_wait.S
+++ b/sysdeps/unix/sysv/linux/i386/i486/pthread_barrier_wait.S
@@ -22,10 +22,10 @@
 
 	.text
 
-	.globl	pthread_barrier_wait
-	.type	pthread_barrier_wait,@function
+	.globl	__pthread_barrier_wait
+	.type	__pthread_barrier_wait,@function
 	.align	16
-pthread_barrier_wait:
+__pthread_barrier_wait:
 	cfi_startproc
 	pushl	%ebx
 	cfi_adjust_cfa_offset(4)
@@ -183,4 +183,5 @@ pthread_barrier_wait:
 	call	__lll_unlock_wake
 	jmp	10b
 	cfi_endproc
-	.size	pthread_barrier_wait,.-pthread_barrier_wait
+	.size	__pthread_barrier_wait,.-__pthread_barrier_wait
+weak_alias (__pthread_barrier_wait, pthread_barrier_wait)
diff --git a/sysdeps/unix/sysv/linux/mq_notify.c b/sysdeps/unix/sysv/linux/mq_notify.c
index 36b327d..5a27274 100644
--- a/sysdeps/unix/sysv/linux/mq_notify.c
+++ b/sysdeps/unix/sysv/linux/mq_notify.c
@@ -92,7 +92,7 @@ notification_function (void *arg)
   union sigval param = data->param;
 
   /* Let the parent go.  */
-  (void) pthread_barrier_wait (&notify_barrier);
+  (void) __pthread_barrier_wait (&notify_barrier);
 
   /* Make the thread detached.  */
   (void) pthread_detach (pthread_self ());
@@ -132,7 +132,7 @@ helper_thread (void *arg)
 				== 0, 0))
 	    /* Since we passed a pointer to DATA to the new thread we have
 	       to wait until it is done with it.  */
-	    (void) pthread_barrier_wait (&notify_barrier);
+	    (void) __pthread_barrier_wait (&notify_barrier);
 	}
       else if (data.raw[NOTIFY_COOKIE_LEN - 1] == NOTIFY_REMOVED)
 	/* The only state we keep is the copy of the thread attributes.  */
@@ -166,7 +166,7 @@ init_mq_netlink (void)
   int err = 1;
 
   /* Initialize the barrier.  */
-  if (__builtin_expect (pthread_barrier_init (&notify_barrier, NULL, 2) == 0,
+  if (__builtin_expect (__pthread_barrier_init (&notify_barrier, NULL, 2) == 0,
 			0))
     {
       /* Create the helper thread.  */
diff --git a/sysdeps/unix/sysv/linux/x86_64/pthread_barrier_wait.S b/sysdeps/unix/sysv/linux/x86_64/pthread_barrier_wait.S
index 69ecc18..e0f7d6c 100644
--- a/sysdeps/unix/sysv/linux/x86_64/pthread_barrier_wait.S
+++ b/sysdeps/unix/sysv/linux/x86_64/pthread_barrier_wait.S
@@ -23,10 +23,10 @@
 
 	.text
 
-	.globl	pthread_barrier_wait
-	.type	pthread_barrier_wait,@function
+	.globl	__pthread_barrier_wait
+	.type	__pthread_barrier_wait,@function
 	.align	16
-pthread_barrier_wait:
+__pthread_barrier_wait:
 	/* Get the mutex.  */
 	xorl	%eax, %eax
 	movl	$1, %esi
@@ -157,4 +157,5 @@ pthread_barrier_wait:
 	xorl	$LLL_SHARED, %esi
 	callq	__lll_unlock_wake
 	jmp	10b
-	.size	pthread_barrier_wait,.-pthread_barrier_wait
+	.size	__pthread_barrier_wait,.-__pthread_barrier_wait
+weak_alias (__pthread_barrier_wait, pthread_barrier_wait)

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog                                          |   27 ++++++++++++++++++++
 NEWS                                               |    2 +-
 include/pthread.h                                  |   10 +++++++
 nptl/Versions                                      |    1 +
 nptl/pthread_barrier_init.c                        |    3 +-
 nptl/pthread_barrier_wait.c                        |    3 +-
 sysdeps/sparc/nptl/pthread_barrier_init.c          |    3 +-
 sysdeps/sparc/nptl/pthread_barrier_wait.c          |    3 +-
 sysdeps/sparc/sparc32/pthread_barrier_wait.c       |    3 +-
 .../sysv/linux/i386/i486/pthread_barrier_wait.S    |    9 +++---
 sysdeps/unix/sysv/linux/mq_notify.c                |    6 ++--
 .../unix/sysv/linux/x86_64/pthread_barrier_wait.S  |    9 +++---
 12 files changed, 62 insertions(+), 17 deletions(-)


hooks/post-receive
-- 
GNU C Library master sources
Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]