Zha0q1 opened a new pull request #20364:
URL: https://github.com/apache/incubator-mxnet/pull/20364


   In the cd arm pipeline it looks like cmake is finding the wrong header for 
amrpl (integrated with mkldnn)
   
https://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/restricted-mxnet-cd%2Fmxnet-cd-release-job-1.x/detail/mxnet-cd-release-job-1.x/2035/pipeline/88
   ```
   [2021-06-17T20:00:16.022Z] -- Looking for sgemm_ - found
   [2021-06-17T20:00:16.022Z] -- Found BLAS: 
/opt/arm/armpl_21.0_gcc-8.2/lib/libarmpl_lp64_mp.so;/usr/local/gcc-8.5.0/lib64/libgfortran.so;/opt/arm/armpl_21.0_gcc-8.2/lib/libamath.so;/usr/lib/aarch64-linux-gnu/libm.so
  
   [2021-06-17T20:00:16.022Z] -- Looking for cblas_sgemm
   [2021-06-17T20:00:16.022Z] -- Looking for cblas_sgemm - found
   [2021-06-17T20:00:16.022Z] -- Found CBLAS: 
/opt/arm/armpl_21.0_gcc-8.2/lib/libarmpl_lp64_mp.so;/usr/local/gcc-8.5.0/lib64/libgfortran.so;/opt/arm/armpl_21.0_gcc-8.2/lib/libamath.so;/usr/lib/aarch64-linux-gnu/libm.so
   [2021-06-17T20:00:16.022Z] -- CBLAS include path: 
/work/mxnet/staticdeps/include
   ```
   
   So it appears this line in the mkldnn repo caused the problem 
   
https://github.com/oneapi-src/oneDNN/blob/83ebc40d86bc54f0f23e947235e53570eeacf254/cmake/blas.cmake#L75
   After changing 
   ```
   find_path(BLAS_INCLUDE_DIR ${CBLAS_HEADERS} $ENV{CPATH} 
${BLAS_LIB_DIR}/../include ${BLAS_LIB_DIR}/../../include)
   ```
   to 
   ```
   find_path(BLAS_INCLUDE_DIR ${CBLAS_HEADERS} HINTS $ENV{CPATH} 
${BLAS_LIB_DIR}/../include ${BLAS_LIB_DIR}/../../include)
   ```
   CMake can find the correct header file. But without changing MKLDNN code we 
can also set `BLAS_INCLUDE_DIR` on our side just for the cd armpl build


-- 
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:
[email protected]


Reply via email to