1. Patchew
  2. QEMU
  3. View series

[PATCH] thread-posix: add support for setting threads name on OpenBSD

Brad Smith posted 1 patch 1 year, 4 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/Y57NrCmPTVSXLWC4@humpty.home.comstyle.com
Maintainers: Paolo Bonzini <pbonzini@redhat.com>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, "Daniel P. Berrangé" <berrange@redhat.com>, Thomas Huth <thuth@redhat.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Brad Smith <brad@comstyle.com>
meson.build              | 12 ++++++++++++
util/qemu-thread-posix.c |  9 ++++++++-
2 files changed, 20 insertions(+), 1 deletion(-)
[PATCH] thread-posix: add support for setting threads name on OpenBSD
Posted by Brad Smith 1 year, 4 months ago 
Make use of pthread_set_name_np() to be able to set the threads name
on OpenBSD.

Signed-off-by: Brad Smith <brad@comstyle.com>
---
 meson.build              | 12 ++++++++++++
 util/qemu-thread-posix.c |  9 ++++++++-
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/meson.build b/meson.build
index 5c6b5a1c75..68adcb6291 100644
--- a/meson.build
+++ b/meson.build
@@ -2123,6 +2123,18 @@ config_host_data.set('CONFIG_PTHREAD_SETNAME_NP_WO_TID', cc.links(gnu_source_pre
     pthread_create(&thread, 0, f, 0);
     return 0;
   }''', dependencies: threads))
+config_host_data.set('CONFIG_PTHREAD_SET_NAME_NP', cc.links(gnu_source_prefix + '''
+  #include <pthread.h>
+  #include <pthread_np.h>
+
+  static void *f(void *p) { return NULL; }
+  int main(void)
+  {
+    pthread_t thread;
+    pthread_create(&thread, 0, f, 0);
+    pthread_set_name_np(thread, "QEMU");
+    return 0;
+  }''', dependencies: threads))
 config_host_data.set('CONFIG_PTHREAD_CONDATTR_SETCLOCK', cc.links(gnu_source_prefix + '''
   #include <pthread.h>
   #include <time.h>
diff --git a/util/qemu-thread-posix.c b/util/qemu-thread-posix.c
index bae938c670..412caa45ef 100644
--- a/util/qemu-thread-posix.c
+++ b/util/qemu-thread-posix.c
@@ -18,6 +18,10 @@
 #include "qemu/tsan.h"
 #include "qemu/bitmap.h"
 
+#ifdef CONFIG_PTHREAD_SET_NAME_NP
+#include <pthread_np.h>
+#endif
+
 static bool name_threads;
 
 void qemu_thread_naming(bool enable)
@@ -25,7 +29,8 @@ void qemu_thread_naming(bool enable)
     name_threads = enable;
 
 #if !defined CONFIG_PTHREAD_SETNAME_NP_W_TID && \
-    !defined CONFIG_PTHREAD_SETNAME_NP_WO_TID
+    !defined CONFIG_PTHREAD_SETNAME_NP_WO_TID && \
+    !defined CONFIG_PTHREAD_SET_NAME_NP
     /* This is a debugging option, not fatal */
     if (enable) {
         fprintf(stderr, "qemu: thread naming not supported on this host\n");
@@ -480,6 +485,8 @@ static void *qemu_thread_start(void *args)
         pthread_setname_np(pthread_self(), qemu_thread_args->name);
 # elif defined(CONFIG_PTHREAD_SETNAME_NP_WO_TID)
         pthread_setname_np(qemu_thread_args->name);
+# elif defined(CONFIG_PTHREAD_SET_NAME_NP)
+        pthread_set_name_np(pthread_self(), qemu_thread_args->name);
 # endif
     }
     QEMU_TSAN_ANNOTATE_THREAD_NAME(qemu_thread_args->name);
-- 
2.38.1
Re: [PATCH] thread-posix: add support for setting threads name on OpenBSD
Posted by Paolo Bonzini 1 year, 2 months ago 
Queued, thanks.

Paolo
Re: [PATCH] thread-posix: add support for setting threads name on OpenBSD
Posted by Brad Smith 1 year, 2 months ago 
ping.

On 2022-12-18 3:22 a.m., Brad Smith wrote:
> Make use of pthread_set_name_np() to be able to set the threads name
> on OpenBSD.
>
> Signed-off-by: Brad Smith <brad@comstyle.com>
> ---
>   meson.build              | 12 ++++++++++++
>   util/qemu-thread-posix.c |  9 ++++++++-
>   2 files changed, 20 insertions(+), 1 deletion(-)
>
> diff --git a/meson.build b/meson.build
> index 5c6b5a1c75..68adcb6291 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -2123,6 +2123,18 @@ config_host_data.set('CONFIG_PTHREAD_SETNAME_NP_WO_TID', cc.links(gnu_source_pre
>       pthread_create(&thread, 0, f, 0);
>       return 0;
>     }''', dependencies: threads))
> +config_host_data.set('CONFIG_PTHREAD_SET_NAME_NP', cc.links(gnu_source_prefix + '''
> +  #include <pthread.h>
> +  #include <pthread_np.h>
> +
> +  static void *f(void *p) { return NULL; }
> +  int main(void)
> +  {
> +    pthread_t thread;
> +    pthread_create(&thread, 0, f, 0);
> +    pthread_set_name_np(thread, "QEMU");
> +    return 0;
> +  }''', dependencies: threads))
>   config_host_data.set('CONFIG_PTHREAD_CONDATTR_SETCLOCK', cc.links(gnu_source_prefix + '''
>     #include <pthread.h>
>     #include <time.h>
> diff --git a/util/qemu-thread-posix.c b/util/qemu-thread-posix.c
> index bae938c670..412caa45ef 100644
> --- a/util/qemu-thread-posix.c
> +++ b/util/qemu-thread-posix.c
> @@ -18,6 +18,10 @@
>   #include "qemu/tsan.h"
>   #include "qemu/bitmap.h"
>   
> +#ifdef CONFIG_PTHREAD_SET_NAME_NP
> +#include <pthread_np.h>
> +#endif
> +
>   static bool name_threads;
>   
>   void qemu_thread_naming(bool enable)
> @@ -25,7 +29,8 @@ void qemu_thread_naming(bool enable)
>       name_threads = enable;
>   
>   #if !defined CONFIG_PTHREAD_SETNAME_NP_W_TID && \
> -    !defined CONFIG_PTHREAD_SETNAME_NP_WO_TID
> +    !defined CONFIG_PTHREAD_SETNAME_NP_WO_TID && \
> +    !defined CONFIG_PTHREAD_SET_NAME_NP
>       /* This is a debugging option, not fatal */
>       if (enable) {
>           fprintf(stderr, "qemu: thread naming not supported on this host\n");
> @@ -480,6 +485,8 @@ static void *qemu_thread_start(void *args)
>           pthread_setname_np(pthread_self(), qemu_thread_args->name);
>   # elif defined(CONFIG_PTHREAD_SETNAME_NP_WO_TID)
>           pthread_setname_np(qemu_thread_args->name);
> +# elif defined(CONFIG_PTHREAD_SET_NAME_NP)
> +        pthread_set_name_np(pthread_self(), qemu_thread_args->name);
>   # endif
>       }
>       QEMU_TSAN_ANNOTATE_THREAD_NAME(qemu_thread_args->name);
Re: [PATCH] thread-posix: add support for setting threads name on OpenBSD
Posted by Philippe Mathieu-Daudé 1 year, 4 months ago 
On 18/12/22 09:22, Brad Smith wrote:
> Make use of pthread_set_name_np() to be able to set the threads name
> on OpenBSD.
> 
> Signed-off-by: Brad Smith <brad@comstyle.com>
> ---
>   meson.build              | 12 ++++++++++++
>   util/qemu-thread-posix.c |  9 ++++++++-
>   2 files changed, 20 insertions(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Re: [PATCH] thread-posix: add support for setting threads name on OpenBSD
Posted by Richard Henderson 1 year, 4 months ago 
On 12/18/22 00:22, Brad Smith wrote:
> Make use of pthread_set_name_np() to be able to set the threads name
> on OpenBSD.
> 
> Signed-off-by: Brad Smith<brad@comstyle.com>
> ---
>   meson.build              | 12 ++++++++++++
>   util/qemu-thread-posix.c |  9 ++++++++-
>   2 files changed, 20 insertions(+), 1 deletion(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.