Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove pthreads #3471

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

remove pthreads #3471

wants to merge 1 commit into from

Conversation

OPPO9008
Copy link

POSIX threads (pthreads)
The android libc, bionic, provides built-in support for pthreads, so no
additional linking (-lpthreads) is necessary. It does not implement full
POSIX threads functionality and leaves out support for read/write locks,
pthread_cancel(), process-shared mutexes and condition variables as well as
other more advanced features. Read the bionic OVERVIEW.txt for more
information.

TLS, thread-local storage, is limited to 59 pthread_key_t slots available
to applications, lower than the posix minimum of 128.

https://web.archive.org/web/20180602101341/http://mobilepearls.com/labs/native-android-api/#pthreads

···
2024-04-30T16:07:39.1464475Z 1 warning generated.
2024-04-30T16:07:40.9096644Z ld.lld: error: unable to find library -lpthread
2024-04-30T16:07:40.9097415Z ld.lld: error: unable to find library -lrt
2024-04-30T16:07:40.9134026Z clang++: error: linker command failed with exit code 1 (use -v to see invocation)
2024-04-30T16:07:40.9160076Z make[3]: *** [CMakeFiles/xmrig.dir/build.make:2744: xmrig] Error 1
2024-04-30T16:07:40.9163778Z make[2]: *** [CMakeFiles/Makefile2:138: CMakeFiles/xmrig.dir/all] Error 2
···

The android libc, bionic, provides built-in support for pthreads, so no
additional linking (-lpthreads) is necessary. It does not implement full
POSIX threads functionality and leaves out support for read/write locks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant