X-Git-Url: http://git.lttng.org/?a=blobdiff_plain;ds=inline;f=liblttng-ust%2Fcompat.h;h=200d9771f85462b91ceabb7b5404254787e25fb9;hb=c0c0989ab70574e09b2f7e8b48c2da6af664a849;hp=003b480aa01d27dc428877dc9935963335e5340d;hpb=8db078dccf54f56edbcbd20cff2430b5e7540801;p=lttng-ust.git

diff --git a/liblttng-ust/compat.h b/liblttng-ust/compat.h
index 003b480a..200d9771 100644
--- a/liblttng-ust/compat.h
+++ b/liblttng-ust/compat.h
@@ -1,88 +1,99 @@
-#ifndef _UST_COMPAT_H
-#define _UST_COMPAT_H
-
 /*
+ * SPDX-License-Identifier: LGPL-2.1-only
+ *
  * Copyright (C) 2011 Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
  * Copyright (C) 2016 Raphaël Beamonte <raphael.beamonte@gmail.com>
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; only
- * version 2.1 of the License.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public
- * License along with this library; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ * Copyright (C) 2020 Michael Jeanson <mjeanson@efficios.com>
  */
 
-/*
- * lttng_ust_getprocname.
- */
-#ifdef __linux__
+#ifndef _UST_COMPAT_H
+#define _UST_COMPAT_H
 
-#include <sys/prctl.h>
+#include <pthread.h>
+#include <errno.h>
+#include <string.h>
+
+#include <lttng/ust-abi.h>
+
+#define LTTNG_UST_PROCNAME_SUFFIX "-ust"
 
-#define LTTNG_UST_PROCNAME_LEN 17
 
+#if defined(HAVE_PTHREAD_SETNAME_NP_WITH_TID)
 static inline
-void lttng_ust_getprocname(char *name)
+int lttng_pthread_setname_np(const char *name)
 {
-	(void) prctl(PR_GET_NAME, (unsigned long) name, 0, 0, 0);
+	/*
+	 * Some implementations don't error out, replicate this behavior for
+	 * consistency.
+	 */
+	if (strnlen(name, LTTNG_UST_ABI_PROCNAME_LEN) >= LTTNG_UST_ABI_PROCNAME_LEN) {
+		return ERANGE;
+	}
+
+	return pthread_setname_np(pthread_self(), name);
 }
 
-/*
- * If pthread_setname_np is available.
- */
-#ifdef HAVE_PTHREAD_SETNAME_NP
 static inline
-int lttng_pthread_setname_np(pthread_t thread, const char *name)
+int lttng_pthread_getname_np(char *name, size_t len)
 {
-	return pthread_setname_np(thread, name);
+	return pthread_getname_np(pthread_self(), name, len);
+}
+#elif defined(HAVE_PTHREAD_SETNAME_NP_WITHOUT_TID)
+static inline
+int lttng_pthread_setname_np(const char *name)
+{
+	return pthread_setname_np(name);
 }
-#endif
 
-#elif defined(__FreeBSD__)
+static inline
+int lttng_pthread_getname_np(char *name, size_t len)
+{
+	return pthread_getname_np(name, len);
+}
+#elif defined(HAVE_PTHREAD_SET_NAME_NP_WITH_TID)
 
-#include <stdlib.h>
-#include <string.h>
+#include <pthread_np.h>
+static inline
+int lttng_pthread_setname_np(const char *name)
+{
+	/* Replicate pthread_setname_np's behavior */
+	if (strnlen(name, LTTNG_UST_ABI_PROCNAME_LEN) >= LTTNG_UST_ABI_PROCNAME_LEN) {
+		return ERANGE;
+	}
 
-/*
- * Limit imposed by Linux UST-sessiond ABI.
- */
-#define LTTNG_UST_PROCNAME_LEN 17
+	pthread_set_name_np(pthread_self(), name);
+	return 0;
+}
 
-/*
- * Acts like linux prctl, the string is not necessarily 0-terminated if
- * 16-byte long.
- */
 static inline
-void lttng_ust_getprocname(char *name)
+int lttng_pthread_getname_np(char *name, size_t len)
 {
-	const char *bsd_name;
+	pthread_get_name_np(pthread_self(), name, len);
+	return 0;
+}
+#elif defined(__linux__)
+
+/* Fallback on prtctl on Linux */
+#include <sys/prctl.h>
 
-	bsd_name = getprogname();
-	if (!bsd_name)
-		name[0] = '\0';
-	else
-		strncpy(name, bsd_name, LTTNG_UST_PROCNAME_LEN - 1);
+static inline
+int lttng_pthread_setname_np(const char *name)
+{
+	/* Replicate pthread_setname_np's behavior */
+	if (strnlen(name, LTTNG_UST_ABI_PROCNAME_LEN) >= LTTNG_UST_ABI_PROCNAME_LEN) {
+		return ERANGE;
+	}
+	return prctl(PR_SET_NAME, name, 0, 0, 0);
 }
 
-/*
- * If pthread_set_name_np is available.
- */
-#ifdef HAVE_PTHREAD_SET_NAME_NP
 static inline
-int lttng_pthread_setname_np(pthread_t thread, const char *name)
+int lttng_pthread_getname_np(char *name, size_t len)
 {
-	return pthread_set_name_np(thread, name);
+	return prctl(PR_GET_NAME, name, 0, 0, 0);
 }
-#endif
 
+#else
+#error "Please add pthread name support for your OS."
 #endif
 
 /*
@@ -90,19 +101,20 @@ int lttng_pthread_setname_np(pthread_t thread, const char *name)
  * the setustprocname() function that will add '-ust' to the end
  * of the current process name, while truncating it if needed.
  */
-#if defined(HAVE_PTHREAD_SETNAME_NP) || defined(HAVE_PTHREAD_SETNAME_NP)
-#define LTTNG_UST_PROCNAME_SUFFIX "-ust"
-
-#include <pthread.h>
-
 static inline
 int lttng_ust_setustprocname(void)
 {
 	int ret = 0, len;
-	char name[LTTNG_UST_PROCNAME_LEN];
-	int limit = LTTNG_UST_PROCNAME_LEN - strlen(LTTNG_UST_PROCNAME_SUFFIX) - 1;
-
-	lttng_ust_getprocname(name);
+	char name[LTTNG_UST_ABI_PROCNAME_LEN];
+	int limit = LTTNG_UST_ABI_PROCNAME_LEN - strlen(LTTNG_UST_PROCNAME_SUFFIX) - 1;
+
+	/*
+	 * Get the current thread name.
+	 */
+	ret = lttng_pthread_getname_np(name, LTTNG_UST_ABI_PROCNAME_LEN);
+	if (ret) {
+		goto error;
+	}
 
 	len = strlen(name);
 	if (len > limit) {
@@ -114,18 +126,11 @@ int lttng_ust_setustprocname(void)
 		goto error;
 	}
 
-	ret = lttng_pthread_setname_np(pthread_self(), name);
+	ret = lttng_pthread_setname_np(name);
 
 error:
 	return ret;
 }
-#else
-static inline
-int lttng_ust_setustprocname(void)
-{
-	return 0;
-}
-#endif
 
 #include <errno.h>