Reviewers: xur, davidxl, iant2,

Message:
I backported the following patch:

2012-03-12  Richard Guenther  <rguent...@suse.de>

        * gthr.h (__GTHREAD_MUTEX_INIT_FUNCTION): Adjust specification.
        * gthr-posix.h (__GTHREAD_MUTEX_INIT_FUNCTION): Define.
        (__gthread_mutex_init_function): New function.
        * gthr-single.h (__GTHREAD_MUTEX_INIT_FUNCTION): Define.

        PR gcov/49484
        * libgcov.c: Include gthr.h.
        (__gcov_flush_mx): New global variable.
        (init_mx, init_mx_once): New functions.
        (__gcov_flush): Protect self with a mutex.
        (__gcov_fork): Re-initialize mutex after forking.
        * unwind-dw2-fde.c: Change condition under which to use
        __GTHREAD_MUTEX_INIT_FUNCTION.



Please review this at http://codereview.appspot.com/6139063/

Affected files:
   M    .
  M     gcc/ChangeLog.google-4_6
  M     gcc/gthr-posix.h
  M     gcc/gthr-single.h
  M     gcc/gthr.h
  M     gcc/libgcov.c
  M     gcc/unwind-dw2-fde.c
  M     libgcc/ChangeLog


Index: .
===================================================================
--- .   (revision 186999)
+++ .   (working copy)

Property changes on: .
___________________________________________________________________
Modified: svn:mergeinfo
   Merged /trunk:r185231
Index: libgcc/ChangeLog
===================================================================
--- libgcc/ChangeLog    (revision 186999)
+++ libgcc/ChangeLog    (working copy)
@@ -1,3 +1,19 @@
+2012-03-12  Richard Guenther  <rguent...@suse.de>
+
+       * gthr.h (__GTHREAD_MUTEX_INIT_FUNCTION): Adjust specification.
+       * gthr-posix.h (__GTHREAD_MUTEX_INIT_FUNCTION): Define.
+       (__gthread_mutex_init_function): New function.
+       * gthr-single.h (__GTHREAD_MUTEX_INIT_FUNCTION): Define.
+
+       PR gcov/49484
+       * libgcov.c: Include gthr.h.
+       (__gcov_flush_mx): New global variable.
+       (init_mx, init_mx_once): New functions.
+       (__gcov_flush): Protect self with a mutex.
+       (__gcov_fork): Re-initialize mutex after forking.
+       * unwind-dw2-fde.c: Change condition under which to use
+       __GTHREAD_MUTEX_INIT_FUNCTION.
+
 2012-03-01  Release Manager

        * GCC 4.6.3 released.
Index: gcc/unwind-dw2-fde.c
===================================================================
--- gcc/unwind-dw2-fde.c        (revision 186999)
+++ gcc/unwind-dw2-fde.c        (working copy)
@@ -46,11 +46,10 @@

 #ifdef __GTHREAD_MUTEX_INIT
 static __gthread_mutex_t object_mutex = __GTHREAD_MUTEX_INIT;
+#define init_object_mutex_once()
 #else
 static __gthread_mutex_t object_mutex;
-#endif

-#ifdef __GTHREAD_MUTEX_INIT_FUNCTION
 static void
 init_object_mutex (void)
 {
@@ -63,8 +62,6 @@
   static __gthread_once_t once = __GTHREAD_ONCE_INIT;
   __gthread_once (&once, init_object_mutex);
 }
-#else
-#define init_object_mutex_once()
 #endif

 /* Called from crtbegin.o to register the unwind info for an object.  */
Index: gcc/ChangeLog.google-4_6
===================================================================
--- gcc/ChangeLog.google-4_6    (revision 186999)
+++ gcc/ChangeLog.google-4_6    (working copy)
@@ -1,3 +1,22 @@
+2012-02-21   Ahmad Sharif  <asha...@google.com>
+
+       Backport from mainline r185231.
+       2012-03-12  Richard Guenther  <rguent...@suse.de>
+
+       * gthr.h (__GTHREAD_MUTEX_INIT_FUNCTION): Adjust specification.
+       * gthr-posix.h (__GTHREAD_MUTEX_INIT_FUNCTION): Define.
+       (__gthread_mutex_init_function): New function.
+       * gthr-single.h (__GTHREAD_MUTEX_INIT_FUNCTION): Define.
+
+       PR gcov/49484
+       * libgcov.c: Include gthr.h.
+       (__gcov_flush_mx): New global variable.
+       (init_mx, init_mx_once): New functions.
+       (__gcov_flush): Protect self with a mutex.
+       (__gcov_fork): Re-initialize mutex after forking.
+       * unwind-dw2-fde.c: Change condition under which to use
+       __GTHREAD_MUTEX_INIT_FUNCTION.
+
 2012-04-25   Rong Xu  <x...@google.com>

        * gcc/gcc.c (ripa_lto_spec): Support -S in streaming LIPO.
Index: gcc/libgcov.c
===================================================================
--- gcc/libgcov.c       (revision 186999)
+++ gcc/libgcov.c       (working copy)
@@ -46,6 +46,7 @@
 #include "tsystem.h"
 #include "coretypes.h"
 #include "tm.h"
+#include "gthr.h"
 #endif /* __KERNEL__ */

 #if 1
@@ -667,6 +668,26 @@
   info->version = 0;
 }

+#ifdef __GTHREAD_MUTEX_INIT
+ATTRIBUTE_HIDDEN __gthread_mutex_t __gcov_flush_mx = __GTHREAD_MUTEX_INIT;
+#define init_mx_once()
+#else
+__gthread_mutex_t __gcov_flush_mx ATTRIBUTE_HIDDEN;
+
+static void
+init_mx (void)
+{
+  __GTHREAD_MUTEX_INIT_FUNCTION (&__gcov_flush_mx);
+}
+static void
+init_mx_once (void)
+{
+  static __gthread_once_t once = __GTHREAD_ONCE_INIT;
+  __gthread_once (&once, init_mx);
+}
+#endif
+
+
 /* Called before fork or exec - write out profile information gathered so
    far and reset it to zero.  This avoids duplication or loss of the
    profile information gathered so far.  */
@@ -676,6 +697,9 @@
 {
   const struct gcov_info *gi_ptr;

+  init_mx_once ();
+  __gthread_mutex_lock (&__gcov_flush_mx);
+
   __gcov_stop_pmu_profiler ();
   gcov_exit ();
   for (gi_ptr = __gcov_list; gi_ptr; gi_ptr = gi_ptr->next)
@@ -691,6 +715,7 @@
          }
     }
   __gcov_start_pmu_profiler ();
+  __gthread_mutex_unlock (&__gcov_flush_mx);
 }

 #else /* __GCOV_KERNEL__ */
@@ -1756,8 +1781,13 @@
 pid_t
 __gcov_fork (void)
 {
+  pid_t pid;
+  extern __gthread_mutex_t __gcov_flush_mx;
   __gcov_flush ();
-  return fork ();
+  pid = fork ();
+  if (pid == 0)
+    __GTHREAD_MUTEX_INIT_FUNCTION (&__gcov_flush_mx);
+  return pid;
 }
 #endif

Index: gcc/gthr-posix.h
===================================================================
--- gcc/gthr-posix.h    (revision 186999)
+++ gcc/gthr-posix.h    (working copy)
@@ -54,6 +54,7 @@
 #define __GTHREAD_HAS_COND     1

 #define __GTHREAD_MUTEX_INIT PTHREAD_MUTEX_INITIALIZER
+#define __GTHREAD_MUTEX_INIT_FUNCTION __gthread_mutex_init_function
 #define __GTHREAD_ONCE_INIT PTHREAD_ONCE_INIT
 #if defined(PTHREAD_RECURSIVE_MUTEX_INITIALIZER)
 #define __GTHREAD_RECURSIVE_MUTEX_INIT PTHREAD_RECURSIVE_MUTEX_INITIALIZER
@@ -726,6 +727,13 @@
   return __gthrw_(pthread_setspecific) (__key, __ptr);
 }

+static inline void
+__gthread_mutex_init_function (__gthread_mutex_t *__mutex)
+{
+  if (__gthread_active_p ())
+    __gthrw_(pthread_mutex_init) (__mutex, NULL);
+}
+
 static inline int
 __gthread_mutex_destroy (__gthread_mutex_t *__mutex)
 {
Index: gcc/gthr-single.h
===================================================================
--- gcc/gthr-single.h   (revision 186999)
+++ gcc/gthr-single.h   (working copy)
@@ -36,6 +36,7 @@

 #define __GTHREAD_ONCE_INIT 0
 #define __GTHREAD_MUTEX_INIT 0
+#define __GTHREAD_MUTEX_INIT_FUNCTION(mx)
 #define __GTHREAD_RECURSIVE_MUTEX_INIT 0

 #define UNUSED __attribute__((unused))
Index: gcc/gthr.h
===================================================================
--- gcc/gthr.h  (revision 186999)
+++ gcc/gthr.h  (working copy)
@@ -51,11 +51,12 @@
                to initialize __gthread_mutex_t to get a fast
                non-recursive mutex.
      __GTHREAD_MUTEX_INIT_FUNCTION
-               some systems can't initialize a mutex without a
-               function call.  On such systems, define this to a
-               function which looks like this:
+               to initialize __gthread_mutex_t to get a fast
+               non-recursive mutex.
+               Define this to a function which looks like this:
                  void __GTHREAD_MUTEX_INIT_FUNCTION (__gthread_mutex_t *)
-               Don't define __GTHREAD_MUTEX_INIT in this case
+               Some systems can't initialize a mutex without a
+               function call.  Don't define __GTHREAD_MUTEX_INIT in this case.
      __GTHREAD_RECURSIVE_MUTEX_INIT
      __GTHREAD_RECURSIVE_MUTEX_INIT_FUNCTION
                as above, but for a recursive mutex.


Reply via email to