Re: [FFmpeg-devel] [PATCH v2] lavu/thread: add support for setting thread name on *bsd and solaris

[Brad Smith <brad-at-comstyle.com@ffmpeg.org>]

On 2024-01-07 12:55 a.m., Brad Smith wrote:
> lavu/thread: add support for setting thread name on *bsd and solaris
>
> FreeBSD/DragonFly/Solaris use pthread_setname_np(). OpenBSD uses pthread_set_name_np().
>
> Signed-off-by: Brad Smith<brad@comstyle.com>
> ---
>   configure          | 10 ++++++++++
>   libavutil/thread.h | 14 ++++++++++++--
>   2 files changed, 22 insertions(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index 0b5e83bd20..67660b6292 100755
> --- a/configure
> +++ b/configure
> @@ -2239,6 +2239,7 @@ HEADERS_LIST="
>       opencv2_core_core_c_h
>       OpenGL_gl3_h
>       poll_h
> +    pthread_np_h
>       sys_param_h
>       sys_resource_h
>       sys_select_h
> @@ -2341,6 +2342,8 @@ SYSTEM_FUNCS="
>       posix_memalign
>       prctl
>       pthread_cancel
> +    pthread_set_name_np
> +    pthread_setname_np
>       sched_getaffinity
>       SecItemImport
>       SetConsoleTextAttribute
> @@ -6523,6 +6526,7 @@ check_headers malloc.h
>   check_headers mftransform.h
>   check_headers net/udplite.h
>   check_headers poll.h
> +check_headers pthread_np.h
>   check_headers sys/param.h
>   check_headers sys/resource.h
>   check_headers sys/select.h
> @@ -6691,6 +6695,12 @@ if ! disabled pthreads && ! enabled w32threads && ! enabled os2threads; then
>       if enabled pthreads; then
>           check_builtin sem_timedwait semaphore.h "sem_t *s; sem_init(s,0,0); sem_timedwait(s,0); sem_destroy(s)" $pthreads_extralibs
>           check_func pthread_cancel $pthreads_extralibs
> +        hdrs=pthread.h
> +        if enabled pthread_np_h; then
> +            hdrs="$hdrs pthread_np.h"
> +        fi
> +        check_lib pthread_set_name_np "$hdrs" pthread_set_name_np -lpthread
> +        check_lib pthread_setname_np "$hdrs" pthread_setname_np -lpthread
>       fi
>   fi
>   
> diff --git a/libavutil/thread.h b/libavutil/thread.h
> index 2ded498c89..fa74dd2ea7 100644
> --- a/libavutil/thread.h
> +++ b/libavutil/thread.h
> @@ -26,6 +26,8 @@
>   
>   #if HAVE_PRCTL
>   #include <sys/prctl.h>
> +#elif (HAVE_PTHREAD_SETNAME_NP || HAVE_PTHREAD_SET_NAME_NP) && HAVE_PTHREAD_NP_H
> +#include <pthread_np.h>
>   #endif
>   
>   #include "error.h"
> @@ -213,11 +215,19 @@ static inline int ff_thread_once(char *control, void (*routine)(void))
>   
>   static inline int ff_thread_setname(const char *name)
>   {
> +    int ret = 0;
> +
>   #if HAVE_PRCTL
> -    return AVERROR(prctl(PR_SET_NAME, name));
> +    ret = AVERROR(prctl(PR_SET_NAME, name));
> +#elif HAVE_PTHREAD_SETNAME_NP
> +    ret = AVERROR(pthread_setname_np(pthread_self(), name));
> +#elif HAVE_PTHREAD_SET_NAME_NP
> +    pthread_set_name_np(pthread_self(), name);
> +#else
> +    ret = AVERROR(ENOSYS);
>   #endif
>   
> -    return AVERROR(ENOSYS);
> +    return ret;
>   }
>   
>   #endif /* AVUTIL_THREAD_H */


ping.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".