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

2020-02-21 Thread GitBox
TaoLv commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-589568416
 
 
   Same issue: https://github.com/apache/incubator-mxnet/issues/17366


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2020-02-24 Thread GitBox
TaoLv commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590645312
 
 
   @leezu I need take time to go through this discussion. But do you think 
changing MKL to static link will solve your question? Please see link adviser 
here: https://software.intel.com/en-us/articles/intel-mkl-link-line-advisor


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2020-02-24 Thread GitBox
TaoLv commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590645896
 
 
   @leezu you may also want to take a look at the discussion around here: 
https://github.com/apache/incubator-mxnet/issues/16891#issuecomment-567051540 


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2020-02-24 Thread GitBox
TaoLv commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590646295
 
 
   > So it seems link order is important?
   
   I think so. I remember @icemelon9 also met the problem that importing 
pytorch before mxnet in a single python script will slow down the execution of 
mxnet.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2020-02-25 Thread GitBox
TaoLv commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590741577
 
 
   @leezu How about also set `-DMKL_USE_STATIC_LIBS=ON`? Then there is no need 
to worry about loading iomp dynamically.
   
   On my machine, I always have another `gomp` as reported 
[here](https://github.com/apache/incubator-mxnet/issues/11417#issuecomment-555413002):
   
   ```
   [lvtao@mlt2-clx016 build]$ ldd libmxnet.so | grep omp
   libomp.so => 
/home/lvtao/Workspace/mxnet-official/build/3rdparty/openmp/runtime/src/libomp.so
 (0x7fea4b117000)
   libgomp.so.1 => /lib64/libgomp.so.1 (0x7fea4a4c8000)
   libXcomposite.so.1 => /lib64/libXcomposite.so.1 (0x7fea43a9e000)
   [lvtao@mlt2-clx016 build]$ ldd libmxnet.so | grep mkl
   [lvtao@mlt2-clx016 build]$
   [lvtao@mlt2-clx016 build]$ readelf -a libmxnet.so| grep NEEDED
0x0001 (NEEDED) Shared library: [librt.so.1]
0x0001 (NEEDED) Shared library: 
[libopencv_imgproc.so.2.4]
0x0001 (NEEDED) Shared library: 
[libopencv_highgui.so.2.4]
0x0001 (NEEDED) Shared library: 
[libopencv_core.so.2.4]
0x0001 (NEEDED) Shared library: [libomp.so]
0x0001 (NEEDED) Shared library: [libdl.so.2]
0x0001 (NEEDED) Shared library: [libpthread.so.0]
0x0001 (NEEDED) Shared library: [libstdc++.so.6]
0x0001 (NEEDED) Shared library: [libm.so.6]
0x0001 (NEEDED) Shared library: [libgomp.so.1]
0x0001 (NEEDED) Shared library: [libgcc_s.so.1]
0x0001 (NEEDED) Shared library: [libc.so.6]
0x0001 (NEEDED) Shared library: 
[ld-linux-x86-64.so.2]
   ```


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2020-02-25 Thread GitBox
TaoLv commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-591199535
 
 
   > @TaoLv which version of gcc?
   
   It's 4.8.5 on centos. Do you want me to try a higher version or exclude 
opencv from the build?
   
   > -DMKL_USE_STATIC_LIBS=ON is currently broken on my system and in 
particular doesn't link iomp statically.
   
   @leezu, with this flag on, I would expect only MKL libraries to be 
statically linked while omp runtime is dynamically linked to mxnet.so. That's 
how we handle the omp linkage of DNNL.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2020-02-25 Thread GitBox
TaoLv commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-591200201
 
 
   @leezu what's the error of statically linking MKL libraries?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2020-02-25 Thread GitBox
TaoLv commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-591207215
 
 
   @leezu, previously we thought intel is not distributing iomp static library: 
https://github.com/apache/incubator-mxnet/issues/8532#issuecomment-341834234. 
But from the linked issue, even we fix the omp runtime conflict inside mxnet, 
we may still encounter conflicts in down stream projects.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2020-02-25 Thread GitBox
TaoLv commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-591208799
 
 
   Thank you @cjolivier01 . That's exactly what I just observed. 


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2020-02-25 Thread GitBox
TaoLv commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-591269085
 
 
   Thanks @leezu! It seems that we have got a consensus to address this issue?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services