Skip to content

Commit

Permalink
Unbreak FreeBSD cross-build on MacOS broken in 051460b
Browse files Browse the repository at this point in the history
MacOS used FreeBSD-compatible getprogname() and pthread_getname_np().
But pthread_getthreadid_np() does not exist on MacOS. This implements
libspl_gettid() using pthread_threadid_np() to get the thread id
of the current thread.

Tested with FreeBSD GitHub actions
freebsd-src/.github/workflows/cross-bootstrap-tools.yml

Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: Rob Norris <[email protected]>
Signed-off-by: Martin Matuska <[email protected]>
Closes #16167
  • Loading branch information
mmatuska authored May 9, 2024
1 parent 3400127 commit 414acbd
Showing 1 changed file with 20 additions and 1 deletion.
21 changes: 20 additions & 1 deletion lib/libspl/assert.c
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,11 @@
#define libspl_getprogname() (program_invocation_short_name)
#define libspl_getthreadname(buf, len) \
prctl(PR_GET_NAME, (unsigned long)(buf), 0, 0, 0)
#elif defined(__FreeBSD__)
#elif defined(__FreeBSD__) || defined(__APPLE__)
#if !defined(__APPLE__)
#include <pthread_np.h>
#define libspl_gettid() pthread_getthreadid_np()
#endif
#define libspl_getprogname() getprogname()
#define libspl_getthreadname(buf, len) \
pthread_getname_np(pthread_self(), buf, len);
Expand Down Expand Up @@ -98,6 +100,19 @@ libspl_dump_backtrace(void)
#define libspl_dump_backtrace()
#endif

#if defined(__APPLE__)
static inline uint64_t
libspl_gettid(void)
{
uint64_t tid;

if (pthread_threadid_np(NULL, &tid) != 0)
tid = 0;

return (tid);
}
#endif

static boolean_t libspl_assert_ok = B_FALSE;

void
Expand Down Expand Up @@ -128,7 +143,11 @@ libspl_assertf(const char *file, const char *func, int line,

fprintf(stderr, "\n"
" PID: %-8u COMM: %s\n"
#if defined(__APPLE__)
" TID: %-8" PRIu64 " NAME: %s\n",
#else
" TID: %-8u NAME: %s\n",
#endif
getpid(), libspl_getprogname(),
libspl_gettid(), tname);

Expand Down

0 comments on commit 414acbd

Please sign in to comment.