]> granicus.if.org Git - gc/commitdiff
Improve comment in store_stack_ptr in pthread_stop_world.c
authorIvan Maidanski <ivmai@mail.ru>
Wed, 29 Nov 2017 18:14:59 +0000 (21:14 +0300)
committerIvan Maidanski <ivmai@mail.ru>
Wed, 29 Nov 2017 18:18:15 +0000 (21:18 +0300)
(fix commit a194093)

* pthread_stop_world.c (GC_store_stack_ptr): Refine comment about the
atomic primitives usage.

pthread_stop_world.c

index b1f3745edfcd02a85f6d164f26a247a5b45e9f78..c012f66b552b0ec6504b8807d1513a0f05bd90b2 100644 (file)
@@ -267,9 +267,9 @@ GC_INLINE void GC_store_stack_ptr(GC_thread me)
   /* stack_ptr) and GC_push_all_stacks (fetching stack_ptr) because     */
   /* GC_push_all_stacks is executed after GC_stop_world exits and the   */
   /* latter runs sem_[try]wait repeatedly waiting for all the suspended */
-  /* threads to call sem_post.  At the same time, stack_ptr is stored   */
-  /* (here) and fetched (by GC_push_all_stacks) atomically to avoid the */
-  /* related TSan warning.                                              */
+  /* threads to call sem_post.  Nonetheless, stack_ptr is stored (here) */
+  /* and fetched (by GC_push_all_stacks) using the atomic primitives to */
+  /* avoid the related TSan warning.                                    */
 # ifdef SPARC
     AO_store((volatile AO_t *)&me->stop_info.stack_ptr,
              (AO_t)GC_save_regs_in_stack());