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

TBB: enable RML, do not use text based .so files #2988

Open
wants to merge 2 commits into
base: master
from

Conversation

@extrowerk
Copy link
Member

extrowerk commented Sep 1, 2018

The LIBRARY_PATH needs some love in TEST().
Can somebody help here?

+ifeq (haiku,$(tbb_os))
+ ln -s $(RML.DLL) $(RML_NO_VERSION.DLL)
+else
echo "INPUT ($(RML.DLL))" > $(RML_NO_VERSION.DLL)

This comment has been minimized.

Copy link
@alaviss

alaviss Sep 7, 2018

Contributor

This shouldn't be a concern to us. The text files are ld scripts, and only used during compilation with other softwares. Our dynamic loader should have used the SONAME instead, if not, chances are SONAME weren't set for the library when building on Haiku.

This comment has been minimized.

Copy link
@extrowerk

extrowerk Sep 8, 2018

Author Member

The tests fails with runtime_loader complaining about the contents, thats why i did this.
Do you mean a non-set SONAME could be the culprit?

This comment has been minimized.

Copy link
@alaviss

alaviss Sep 9, 2018

Contributor

The tests run fine without that patch. Got a compiler segmentation fault however:

g++ -o test_eh_algorithms.o -c -MMD -DDO_ITT_NOTIFY -g -O2 -DUSE_PTHREAD -m64 -mrtm   -DTEST_USES_TBB=1 -Wall -Wshadow -Wcast-qual -Woverloaded-virtual -Wnon-virtual-dtor -Wextra    -I../../src -I../../src/rml/include -I../../include -I. -include ../../src/test/harness_preload.h ../../src/test/test_eh_algorithms.cpp
../../src/test/test_eh_algorithms.cpp: In function 'void TestParallelLoopAux() [with ParForBody = SimpleParForBody; Partitioner = const tbb::simple_partitioner]':
../../src/test/test_eh_algorithms.cpp:162:6: internal compiler error: Segmentation violation
 void TestParallelLoopAux() {
      ^~~~~~~~~~~~~~~~~~~
libbacktrace could not find executable to open
Please submit a full bug report,
with preprocessed source if appropriate.
See <https://gcc.gnu.org/bugs/> for instructions.
../../build/common_rules.inc:86: recipe for target 'test_eh_algorithms.o' failed
make[1]: *** [test_eh_algorithms.o] Error 1
make[1]: Leaving directory '/sources/tbb-2018_U5/build/haiku_intel64_gcc_cc7.3.0_libc_kernel1_release'
Makefile:44: recipe for target 'test' failed
@alaviss

This comment has been minimized.

Copy link
Contributor

alaviss commented Sep 8, 2018

@extrowerk

This comment has been minimized.

Copy link
Member Author

extrowerk commented Sep 8, 2018

We have literally the same part in haiku_gcc.inc (it is based on linux_gcc.inc, but trimmed as the TBB guys wanted).
So i still doesn't really understand, whats going on.

@alaviss

This comment has been minimized.

Copy link
Contributor

alaviss commented Sep 9, 2018

Got this lurking in the logs while building:

hostname: invalid option -- 's'
Try 'hostname --help' for more information.
../../build/version_info_linux.sh: line 29: getconf: command not found

This might be the culprit. Scrap that, the SONAME is set correctly.

@extrowerk

This comment has been minimized.

Copy link
Member Author

extrowerk commented Dec 18, 2018

tls fixed, let's continue this...

@extrowerk extrowerk force-pushed the extrowerk:tbb_rml branch from 4dabde4 to 9baa048 Apr 10, 2019
extrowerk added 2 commits Apr 10, 2019
@korli

This comment has been minimized.

Copy link
Contributor

korli commented Nov 22, 2019

Why enabling RML?

@extrowerk

This comment has been minimized.

Copy link
Member Author

extrowerk commented Nov 22, 2019

@korli I can't answer it anymore, it was long time ago, but it is easy to disable it.
It is the Intel Resource Management Layer library. It is a work dispatcher used by Threading Building Blocks (TBB).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.