Skip to content

Fix MKL linkage with OpenMP parallel framework#9068

Merged
opencv-pushbot merged 1 commit intoopencv:masterfrom
dkurt:mkl_with_openmp
Jul 3, 2017
Merged

Fix MKL linkage with OpenMP parallel framework#9068
opencv-pushbot merged 1 commit intoopencv:masterfrom
dkurt:mkl_with_openmp

Conversation

@dkurt
Copy link
Copy Markdown
Member

@dkurt dkurt commented Jul 1, 2017

This pullrequest changes

Fix MKL_WITH_OPENMP option.

@alalek
Copy link
Copy Markdown
Member

alalek commented Jul 1, 2017

👍

@dkurt
Copy link
Copy Markdown
Member Author

dkurt commented Jul 1, 2017

BTW, it seems to me that OpenMP is still missed here. Because we link mkl_gnu_thread instead iomp5 as mentioned in output of mkl_link_tool -libs.

@dkurt
Copy link
Copy Markdown
Member Author

dkurt commented Jul 3, 2017

Compared efficiency for different configurations.

OS: Linux 4.8.0-34-generic x86_64
CPU: Intel® Core™ i7-6700K CPU @ 4.00GHz × 8
Note: be sure that LD_PRELOAD has no mkl references.

Linkage with Matrix multiplication, squad 1024, best time
mkl_sequential 17.3ms
pthread?, mkl_gnu_thread 4.9ms
OpenMP, iomp5 4.9ms
TBB, mkl_tbb_thread 9.5ms

MKL build date: 30 Apr 2017

@opencv-pushbot opencv-pushbot merged commit 425e92e into opencv:master Jul 3, 2017
opencv-pushbot pushed a commit that referenced this pull request Jul 3, 2017
@alalek
Merge pull request #9068 from dkurt:mkl_with_openmp
b10bd4b
@dkurt dkurt deleted the mkl_with_openmp branch July 4, 2017 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mshabunin mshabunin mshabunin approved these changes

Assignees

@alalek alalek

Labels

None yet

Projects

None yet

Milestone

3.3-rc

Development

Successfully merging this pull request may close these issues.

4 participants

@dkurt @alalek @mshabunin @opencv-pushbot