1358214 - profiler does not build with musl libc

Status: RESOLVED FIXED

(Core :: Gecko Profiler, defect, P2)

Description

[Duncan Overbruck]

Attachment: fix_gettid_musl.patch

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_3) AppleWebKit/600.5.17 (KHTML, like Gecko) Version/8.0.5 Safari/600.5.17
Build ID: 20170319182602

Steps to reproduce:

Building firefox with musl libc on a linux system.


Actual results:

gettid(2) is not defined.

```
/builddir/firefox-53.0/tools/profiler/core/platform-linux.cc: In static member function 'static Thread::tid_t Thread::GetCurrentId()':
/builddir/firefox-53.0/tools/profiler/core/platform-linux.cc:120:17: error: 'gettid' was not declared in this scope
   return gettid();
                 ^
/builddir/firefox-53.0/tools/profiler/core/platform-linux.cc: In function 'void* SignalSender(void*)':
/builddir/firefox-53.0/tools/profiler/core/platform-linux.cc:307:34: error: 'gettid' was not declared in this scope
   DebugOnly<int> my_tid = gettid();
                                  ^

/builddir/firefox-53.0/tools/profiler/lul/LulMain.cpp: In constructor 'lul::LUL::LUL(void (*)(const char*))':
/builddir/firefox-53.0/tools/profiler/lul/LulMain.cpp:841:27: error: 'gettid' was not declared in this scope
   , mAdminThreadId(gettid())
                           ^
/builddir/firefox-53.0/tools/profiler/lul/LulMain.cpp: In destructor 'lul::LUL::~LUL()':
/builddir/firefox-53.0/tools/profiler/lul/LulMain.cpp:833:37: error: 'gettid' was not declared in this scope
                    getpid(), gettid(), this, (_str));   \
                                     ^
/builddir/firefox-53.0/tools/profiler/lul/LulMain.cpp:852:3: note: in expansion of macro 'LUL_LOG'
   LUL_LOG("LUL::~LUL: Destroyed object");
   ^~~~~~~
/builddir/firefox-53.0/tools/profiler/lul/LulMain.cpp: In member function 'void lul::LUL::EnableUnwinding()':
/builddir/firefox-53.0/tools/profiler/lul/LulMain.cpp:833:37: error: 'gettid' was not declared in this scope
                    getpid(), gettid(), this, (_str));   \
                                     ^
/builddir/firefox-53.0/tools/profiler/lul/LulMain.cpp:888:3: note: in expansion of macro 'LUL_LOG'
   LUL_LOG("LUL::EnableUnwinding");
   ^~~~~~~
```


Expected results:

`gettid(2)` should be defined for all libcs as discussed in Bug 1261392.

Comment 1

[Thinker Li [:sinker]]

I think you are requesting for a feedback against the patch.
You should not replace __GLIBC__ with __linux__ because, may, some libc
for Linux have defined gettid().
So, what you should do is to use
 #if defined(__GLIBC__) || defined(__MUSL__)
or some thing similar instead of
 #if defined(__GLIBC__)

Unfortunately, musl libc seems refusing to define __MUSL__ or alike.
So, you need to find a way to detect if it is using musl libc.

Comment 2

[Felix Janda]

Since no libc on linux except bionic has gettid, it seems easier to check whether we are not on Android:
#if defined(__linux__) || !defined(__BIONIC__)

Comment 3

[Jory A. Pratt [:anarchy]]

Attachment: 6009_linux-gettid-wrapper.patch

Please review so we can get this resolved once and for all.

Comment 4

[Nicholas Nethercote [inactive]]

The change seems fine, as long as it passes tests. However, a while back Mozilla switched to using Phabricator for submitting and reviewing patches. Details are here: https://wiki.mozilla.org/Phabricator. So the submission will have to go through there. Sorry for the inconvenience!

Comment 5

[Jory A. Pratt [:anarchy]]

Attachment: gettid wrapper is not provided by any libc in linux

gettid is not supported by any libc in linux so use the wrapper

Comment 6

[Jory A. Pratt [:anarchy]]

Attachment: gettid wrapper is not provided by any libc in linux

BIONIC is only platform that actually supports gettid. Easiest
solution is to check for linux and disable for BIONIC platform. This
includes the change requested by Gerald to keep the two profilers sync'd.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1e2f7d1f21ea
gettid wrapper is not provided by any libc in linux r=njn

Comment 8

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/1e2f7d1f21ea