gobes commented on issue #47229:
URL: https://github.com/apache/arrow/issues/47229#issuecomment-3240587130

   @pitrou, no, it does not fix the issue.
   
   Try 1, via arrow cmake  -DMIMALLOC_CMAKE_ARGS="-DMI_NO_OPT_ARCH=ON" \
   result : still crash on the same line
   
   Try 2, hard-coded in    python/cmake_modules/ThirdpartyToolchain.cmake
   
   > set(MIMALLOC_CMAKE_ARGS
         ${EP_COMMON_CMAKE_ARGS}
         "-DCMAKE_C_FLAGS=${MIMALLOC_C_FLAGS}"
         "-DCMAKE_INSTALL_PREFIX=${MIMALLOC_PREFIX}"
         -DMI_OVERRIDE=OFF
         -DMI_LOCAL_DYNAMIC_TLS=ON
         -DMI_BUILD_OBJECT=OFF
         -DMI_BUILD_SHARED=OFF
         -DMI_NO_OPT_ARCH=ON     <---------------
         -DMI_BUILD_TESTS=OFF)
   
   result: still crash on the same line
   
   Try 3, just disable it  -DARROW_MIMALLOC=OFF
   result: **pass** (probably the quicker temporary fix for these cpu; Is there 
any equivalent run time env.  to disable it ? )
   
   Both tries by cross-compile and native.
   
   -DMI_NO_OPT_ARCH it may not be propagated correctly on try 1/2, but even if 
it does, I'm not sure  it will fix it.
   
   Arrow 21.0.0 seems to use ARROW_MIMALLOC_BUILD_VERSION=v2.2.4. 
   
   A quick look into 
https://github.com/microsoft/mimalloc/blob/09a27098aa6e9286518bd9c74e6ffa7199c3f04e/CMakeLists.txt#L146),
 MI_NO_OPT_ARCH=ON doesn't do too much, it just prevent -march=armv8.1-a to be 
passed to compiler, so the -march will be "armv8-a". 
   
   But -march=armv8-a, it was not enough for cortex-a72/a57, that's why I ended 
up in adding -mtune=cortex-a72 -mno-outline-atomics 
   
   
   
   
   
    
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to