[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-594100964 yesh just submit the static linkage On Tue, Mar 3, 2020 at 10:18 AM Leonard Lausen wrote: > @cjolivier01

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-591513111 i will post a pr in the next day or two that addresses also clang issue as well as transitive omp dependencies which may also cause the error due to mkl

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-591491999 there’s a lot of stuff in that PR，would prefer a more targeted PR. Also, I think the best fix is link statically to MKL as we discussed, which as far

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-591207663 > @leezu what's the error of statically linking MKL libraries? for me it was some link error on some secondary thing like cpp unit test or

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-591163177 Stopping clang/others from linking to omp seems like the tail wagging the dog. I think we should consider other options, such as making the static mkl

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-591014168 > > @TaoLv > > what version of gcc? > > @cjolivier01 this will happen with any version of gcc @TaoLv which version of gcc?

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590997673 > -DMKL_USE_STATIC_LIBS=ON How broken? When I built with clang, it seems to run ok.

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590996534 mayb e statically linking is ok, I am not sure This is an automated message from the

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590995962 > > @TaoLv > > what version of gcc? > > @cjolivier01 this will happen with any version of gcc it doesn;t happen on my machine, but I am

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590995601 > @TaoLv `-DMKL_USE_STATIC_LIBS=ON` is currently broken on my system and in particular doesn't link `iomp` statically. > > I'll create a PR to

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590990584 @TaoLv what version of gcc? This is an automated message from the Apache Git Service.

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590527708 Another thing to note is that ompenmp with llvm also installs libgomp.so as a symlink to libomp.so, so there's a good chance that libomp.so will be loaded

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590521574 I can reproduce now. However, even if I remove the openmp build in CMakeLists.txt and build with clang, I get that warning, since it pulls in libomp

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590486220 does it occur with opencv turned off? This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590476688 This behavior seems suspicious because the claim would also suggest that the problem would exist for all similar build cases (ie my machine now) as well

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590472526 This is with latest 2020 version of mkl? This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590450010 ```python (pytorch) [chriso@chriso-ripper:~/src/mxnet (master)]python Python 3.6.10 |Anaconda, Inc.| (default, Jan 7 2020, 21:14:29) [GCC 7.3.0]

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590449594 ``` 0x0001 (NEEDED) Shared library: [libmkl_rt.so] 0x0001 (NEEDED) Shared library: [librt.so.1]

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-589853130 can you supply a script to reproduc this error? I am not able to reproduce. This

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-589849228 Clearly it does not: [chriso@chriso-ripper:~/src/mxnet/build (master)]ldd /opt/intel/mkl/lib/intel64/libmkl_rt.so linux-vdso.so.1

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-589848880 Linking in any version of omp statically would probably be a bad idea, since startup order would be important.

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-589836549 Actually, i don;t see this behavior when it oes pull in mkl/pulling in the other omp: ``` [chriso@chriso-ripper:~/src/mxnet/build (master)]ldd

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-589832383 I installed mkl, but it does not appear to pick it up. is there a way to force it? This

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-589832039 openmp is like a 4-5-second build. On my desktop machine it's < 3: real0m2.940s user0m42.446s sys 0m5.442s

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-589817764 > @cjolivier01 you previously vetoed changing the omp configuration in cmake build, due to a race condition that had not been fixed. As that has been

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-589812034 btw, cmake files have min cmake at 3.13, but default 18.04 cmake install is cmake 3.10. Does anyone know what the deal is with 3.13? Ubuntu 18.04 is a

[GitHub] [incubator-mxnet] cjolivier01 commented on issue #17641: OpenMP Error

cjolivier01 commented on issue #17641: OpenMP Error URL: https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-589809022 1) What is pulling in libiomp5.so ? 2) Since when is libomp being linked in statically? I am not aware of this ever being the case.