Sat Feb 12 14:59:32 2022 UTC ()
libpthread: Move namespacing include to top of .c files.

Stuff like libc's namespace.h, or atomic_op_namespace.h, which does
namespacing tricks like `#define atomic_cas_uint _atomic_cas_uint',
has to go at the top of each .c file.  If it goes in the middle, it
might be too late to affect the declarations, and result in compile
errors.

I tripped over this by including <sys/atomic.h> in mips
<machine/lock.h>.

(Maybe we should create a new pthread_namespace.h file for the
purpose, but this'll do for now.)


(riastradh)
diff -r1.179 -r1.180 src/lib/libpthread/pthread.c
diff -r1.19 -r1.20 src/lib/libpthread/pthread_attr.c
diff -r1.22 -r1.23 src/lib/libpthread/pthread_barrier.c
diff -r1.40 -r1.41 src/lib/libpthread/pthread_cancelstub.c
diff -r1.6 -r1.7 src/lib/libpthread/pthread_compat.c
diff -r1.6 -r1.7 src/lib/libpthread/res_state.c
diff -r1.76 -r1.77 src/lib/libpthread/pthread_cond.c
diff -r1.3 -r1.4 src/lib/libpthread/pthread_getcpuclockid.c
diff -r1.3 -r1.4 src/lib/libpthread/pthread_once.c
diff -r1.107 -r1.108 src/lib/libpthread/pthread_int.h
diff -r1.34 -r1.35 src/lib/libpthread/pthread_lock.c
diff -r1.2 -r1.3 src/lib/libpthread/pthread_makelwp_netbsd.c
diff -r1.17 -r1.18 src/lib/libpthread/pthread_misc.c
diff -r1.81 -r1.82 src/lib/libpthread/pthread_mutex.c
diff -r1.43 -r1.44 src/lib/libpthread/pthread_rwlock.c
diff -r1.26 -r1.27 src/lib/libpthread/pthread_specific.c
diff -r1.8 -r1.9 src/lib/libpthread/pthread_spin.c
diff -r1.23 -r1.24 src/lib/libpthread/pthread_tsd.c

cvs diff -r1.179 -r1.180 src/lib/libpthread/pthread.c (expand / switch to context diff)
--- src/lib/libpthread/pthread.c 2021/04/13 00:31:54 1.179
+++ src/lib/libpthread/pthread.c 2022/02/12 14:59:32 1.180
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread.c,v 1.179 2021/04/13 00:31:54 mrg Exp $	*/
+/*	$NetBSD: pthread.c,v 1.180 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2002, 2003, 2006, 2007, 2008, 2020
@@ -31,9 +31,12 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread.c,v 1.179 2021/04/13 00:31:54 mrg Exp $");
+__RCSID("$NetBSD: pthread.c,v 1.180 2022/02/12 14:59:32 riastradh Exp $");
 
 #define	__EXPOSE_STACK	1
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <sys/param.h>
 #include <sys/exec_elf.h>

cvs diff -r1.19 -r1.20 src/lib/libpthread/pthread_attr.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_attr.c 2020/01/29 13:47:31 1.19
+++ src/lib/libpthread/pthread_attr.c 2022/02/12 14:59:32 1.20
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_attr.c,v 1.19 2020/01/29 13:47:31 kamil Exp $	*/
+/*	$NetBSD: pthread_attr.c,v 1.20 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2002, 2003, 2008 The NetBSD Foundation, Inc.
@@ -30,7 +30,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_attr.c,v 1.19 2020/01/29 13:47:31 kamil Exp $");
+__RCSID("$NetBSD: pthread_attr.c,v 1.20 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <errno.h>
 #include <stdio.h>

cvs diff -r1.22 -r1.23 src/lib/libpthread/pthread_barrier.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_barrier.c 2020/05/16 22:53:37 1.22
+++ src/lib/libpthread/pthread_barrier.c 2022/02/12 14:59:32 1.23
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_barrier.c,v 1.22 2020/05/16 22:53:37 ad Exp $	*/
+/*	$NetBSD: pthread_barrier.c,v 1.23 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2003, 2006, 2007, 2009, 2020 The NetBSD Foundation, Inc.
@@ -30,7 +30,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_barrier.c,v 1.22 2020/05/16 22:53:37 ad Exp $");
+__RCSID("$NetBSD: pthread_barrier.c,v 1.23 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <errno.h>
 

cvs diff -r1.40 -r1.41 src/lib/libpthread/pthread_cancelstub.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_cancelstub.c 2021/10/01 20:13:38 1.40
+++ src/lib/libpthread/pthread_cancelstub.c 2022/02/12 14:59:32 1.41
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_cancelstub.c,v 1.40 2021/10/01 20:13:38 christos Exp $	*/
+/*	$NetBSD: pthread_cancelstub.c,v 1.41 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2002, 2007 The NetBSD Foundation, Inc.
@@ -33,7 +33,10 @@
 #undef _FORTIFY_SOURCE
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_cancelstub.c,v 1.40 2021/10/01 20:13:38 christos Exp $");
+__RCSID("$NetBSD: pthread_cancelstub.c,v 1.41 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #ifndef lint
 

cvs diff -r1.6 -r1.7 src/lib/libpthread/pthread_compat.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_compat.c 2017/12/08 09:59:26 1.6
+++ src/lib/libpthread/pthread_compat.c 2022/02/12 14:59:32 1.7
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_compat.c,v 1.6 2017/12/08 09:59:26 kre Exp $	*/
+/*	$NetBSD: pthread_compat.c,v 1.7 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2008 The NetBSD Foundation, Inc.
@@ -34,7 +34,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_compat.c,v 1.6 2017/12/08 09:59:26 kre Exp $");
+__RCSID("$NetBSD: pthread_compat.c,v 1.7 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <sys/param.h>
 #include <sys/syscall.h>

cvs diff -r1.6 -r1.7 src/lib/libpthread/res_state.c (expand / switch to context diff)
--- src/lib/libpthread/res_state.c 2008/04/28 20:23:02 1.6
+++ src/lib/libpthread/res_state.c 2022/02/12 14:59:32 1.7
@@ -1,4 +1,4 @@
-/*	$NetBSD: res_state.c,v 1.6 2008/04/28 20:23:02 martin Exp $	*/
+/*	$NetBSD: res_state.c,v 1.7 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2004 The NetBSD Foundation, Inc.
@@ -31,8 +31,11 @@
 
 #include <sys/cdefs.h>
 #if defined(LIBC_SCCS) && !defined(lint)
-__RCSID("$NetBSD: res_state.c,v 1.6 2008/04/28 20:23:02 martin Exp $");
+__RCSID("$NetBSD: res_state.c,v 1.7 2022/02/12 14:59:32 riastradh Exp $");
 #endif
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <sys/types.h>
 #include <sys/queue.h>

cvs diff -r1.76 -r1.77 src/lib/libpthread/pthread_cond.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_cond.c 2020/06/14 21:33:28 1.76
+++ src/lib/libpthread/pthread_cond.c 2022/02/12 14:59:32 1.77
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_cond.c,v 1.76 2020/06/14 21:33:28 ad Exp $	*/
+/*	$NetBSD: pthread_cond.c,v 1.77 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2006, 2007, 2008, 2020 The NetBSD Foundation, Inc.
@@ -30,7 +30,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_cond.c,v 1.76 2020/06/14 21:33:28 ad Exp $");
+__RCSID("$NetBSD: pthread_cond.c,v 1.77 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <stdlib.h>
 #include <errno.h>

cvs diff -r1.3 -r1.4 src/lib/libpthread/pthread_getcpuclockid.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_getcpuclockid.c 2020/01/29 16:03:44 1.3
+++ src/lib/libpthread/pthread_getcpuclockid.c 2022/02/12 14:59:32 1.4
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_getcpuclockid.c,v 1.3 2020/01/29 16:03:44 kamil Exp $	*/
+/*	$NetBSD: pthread_getcpuclockid.c,v 1.4 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2016 The NetBSD Foundation, Inc.
@@ -30,8 +30,11 @@
  */
 #include <sys/cdefs.h>
 #if defined(LIBC_SCCS) && !defined(lint)
-__RCSID("$NetBSD: pthread_getcpuclockid.c,v 1.3 2020/01/29 16:03:44 kamil Exp $");
+__RCSID("$NetBSD: pthread_getcpuclockid.c,v 1.4 2022/02/12 14:59:32 riastradh Exp $");
 #endif /* LIBC_SCCS and not lint */
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <sys/types.h>
 #include <errno.h>

cvs diff -r1.3 -r1.4 src/lib/libpthread/pthread_once.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_once.c 2013/03/21 16:49:12 1.3
+++ src/lib/libpthread/pthread_once.c 2022/02/12 14:59:32 1.4
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_once.c,v 1.3 2013/03/21 16:49:12 christos Exp $	*/
+/*	$NetBSD: pthread_once.c,v 1.4 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2003 The NetBSD Foundation, Inc.
@@ -37,7 +37,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_once.c,v 1.3 2013/03/21 16:49:12 christos Exp $");
+__RCSID("$NetBSD: pthread_once.c,v 1.4 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include "pthread.h"
 #include "pthread_int.h"

cvs diff -r1.107 -r1.108 src/lib/libpthread/pthread_int.h (expand / switch to context diff)
--- src/lib/libpthread/pthread_int.h 2020/06/10 22:45:15 1.107
+++ src/lib/libpthread/pthread_int.h 2022/02/12 14:59:32 1.108
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_int.h,v 1.107 2020/06/10 22:45:15 ad Exp $	*/
+/*	$NetBSD: pthread_int.h,v 1.108 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2002, 2003, 2006, 2007, 2008, 2020
@@ -40,9 +40,6 @@
 #include "pthread_types.h"
 #include "pthread_queue.h"
 #include "pthread_md.h"
-
-/* Need to use libc-private names for atomic operations. */
-#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <sys/atomic.h>
 #include <sys/rbtree.h>

cvs diff -r1.34 -r1.35 src/lib/libpthread/pthread_lock.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_lock.c 2008/04/28 20:23:01 1.34
+++ src/lib/libpthread/pthread_lock.c 2022/02/12 14:59:32 1.35
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_lock.c,v 1.34 2008/04/28 20:23:01 martin Exp $	*/
+/*	$NetBSD: pthread_lock.c,v 1.35 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2006, 2007 The NetBSD Foundation, Inc.
@@ -34,7 +34,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_lock.c,v 1.34 2008/04/28 20:23:01 martin Exp $");
+__RCSID("$NetBSD: pthread_lock.c,v 1.35 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <sys/types.h>
 #include <sys/ras.h>

cvs diff -r1.2 -r1.3 src/lib/libpthread/pthread_makelwp_netbsd.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_makelwp_netbsd.c 2014/12/17 01:49:08 1.2
+++ src/lib/libpthread/pthread_makelwp_netbsd.c 2022/02/12 14:59:32 1.3
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_makelwp_netbsd.c,v 1.2 2014/12/17 01:49:08 pooka Exp $	*/
+/*	$NetBSD: pthread_makelwp_netbsd.c,v 1.3 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2002, 2003, 2006, 2007, 2008 The NetBSD Foundation, Inc.
@@ -30,7 +30,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_makelwp_netbsd.c,v 1.2 2014/12/17 01:49:08 pooka Exp $");
+__RCSID("$NetBSD: pthread_makelwp_netbsd.c,v 1.3 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <sys/param.h>
 

cvs diff -r1.17 -r1.18 src/lib/libpthread/pthread_misc.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_misc.c 2020/01/29 16:34:09 1.17
+++ src/lib/libpthread/pthread_misc.c 2022/02/12 14:59:32 1.18
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_misc.c,v 1.17 2020/01/29 16:34:09 kamil Exp $	*/
+/*	$NetBSD: pthread_misc.c,v 1.18 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2006, 2007, 2008 The NetBSD Foundation, Inc.
@@ -30,7 +30,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_misc.c,v 1.17 2020/01/29 16:34:09 kamil Exp $");
+__RCSID("$NetBSD: pthread_misc.c,v 1.18 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <errno.h>
 #include <string.h>

cvs diff -r1.81 -r1.82 src/lib/libpthread/pthread_mutex.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_mutex.c 2020/06/11 18:41:22 1.81
+++ src/lib/libpthread/pthread_mutex.c 2022/02/12 14:59:32 1.82
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_mutex.c,v 1.81 2020/06/11 18:41:22 ad Exp $	*/
+/*	$NetBSD: pthread_mutex.c,v 1.82 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2003, 2006, 2007, 2008, 2020 The NetBSD Foundation, Inc.
@@ -47,7 +47,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_mutex.c,v 1.81 2020/06/11 18:41:22 ad Exp $");
+__RCSID("$NetBSD: pthread_mutex.c,v 1.82 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <sys/types.h>
 #include <sys/lwpctl.h>

cvs diff -r1.43 -r1.44 src/lib/libpthread/pthread_rwlock.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_rwlock.c 2022/02/11 21:40:58 1.43
+++ src/lib/libpthread/pthread_rwlock.c 2022/02/12 14:59:32 1.44
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_rwlock.c,v 1.43 2022/02/11 21:40:58 riastradh Exp $ */
+/*	$NetBSD: pthread_rwlock.c,v 1.44 2022/02/12 14:59:32 riastradh Exp $ */
 
 /*-
  * Copyright (c) 2002, 2006, 2007, 2008, 2020 The NetBSD Foundation, Inc.
@@ -30,7 +30,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_rwlock.c,v 1.43 2022/02/11 21:40:58 riastradh Exp $");
+__RCSID("$NetBSD: pthread_rwlock.c,v 1.44 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <sys/types.h>
 #include <sys/lwpctl.h>

cvs diff -r1.26 -r1.27 src/lib/libpthread/pthread_specific.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_specific.c 2013/03/21 16:49:12 1.26
+++ src/lib/libpthread/pthread_specific.c 2022/02/12 14:59:32 1.27
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_specific.c,v 1.26 2013/03/21 16:49:12 christos Exp $	*/
+/*	$NetBSD: pthread_specific.c,v 1.27 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2007 The NetBSD Foundation, Inc.
@@ -30,7 +30,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_specific.c,v 1.26 2013/03/21 16:49:12 christos Exp $");
+__RCSID("$NetBSD: pthread_specific.c,v 1.27 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 /* Functions and structures dealing with thread-specific data */
 

cvs diff -r1.8 -r1.9 src/lib/libpthread/pthread_spin.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_spin.c 2020/02/05 11:05:10 1.8
+++ src/lib/libpthread/pthread_spin.c 2022/02/12 14:59:32 1.9
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_spin.c,v 1.8 2020/02/05 11:05:10 kamil Exp $	*/
+/*	$NetBSD: pthread_spin.c,v 1.9 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2006, 2007 The NetBSD Foundation, Inc.
@@ -34,7 +34,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_spin.c,v 1.8 2020/02/05 11:05:10 kamil Exp $");
+__RCSID("$NetBSD: pthread_spin.c,v 1.9 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 #include <sys/types.h>
 #include <sys/ras.h>

cvs diff -r1.23 -r1.24 src/lib/libpthread/pthread_tsd.c (expand / switch to context diff)
--- src/lib/libpthread/pthread_tsd.c 2020/06/11 18:42:02 1.23
+++ src/lib/libpthread/pthread_tsd.c 2022/02/12 14:59:32 1.24
@@ -1,4 +1,4 @@
-/*	$NetBSD: pthread_tsd.c,v 1.23 2020/06/11 18:42:02 ad Exp $	*/
+/*	$NetBSD: pthread_tsd.c,v 1.24 2022/02/12 14:59:32 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2001, 2007, 2020 The NetBSD Foundation, Inc.
@@ -30,7 +30,10 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pthread_tsd.c,v 1.23 2020/06/11 18:42:02 ad Exp $");
+__RCSID("$NetBSD: pthread_tsd.c,v 1.24 2022/02/12 14:59:32 riastradh Exp $");
+
+/* Need to use libc-private names for atomic operations. */
+#include "../../common/lib/libc/atomic/atomic_op_namespace.h"
 
 /* Functions and structures dealing with thread-specific data */
 #include <errno.h>