Project

General

Profile

Actions

Bug #17516

closed

aarch64: Compiler-based detection of crc32 extended CPU type is broken

Added by Nathan Cutler over 7 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
% Done:

0%

Source:
Community (dev)
Tags:
Backport:
jewel,luminous,mimic
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Related issues 3 (0 open3 closed)

Copied to devops - Backport #17575: jewel: aarch64: Compiler-based detection of crc32 extended CPU type is brokenResolvedNathan CutlerActions
Copied to devops - Backport #36080: luminous: aarch64: Compiler-based detection of crc32 extended CPU type is brokenResolvedKefu ChaiActions
Copied to devops - Backport #40258: mimic: aarch64: Compiler-based detection of crc32 extended CPU type is brokenResolvedNathan CutlerActions
Actions #2

Updated by Nathan Cutler over 7 years ago

Note that the master PR uses cmake but the jewel backport will need to use autotools - agraf has already prepared an autotools version of the fix.

Actions #3

Updated by Nathan Cutler over 7 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #4

Updated by Loïc Dachary over 7 years ago

  • Copied to Backport #17575: jewel: aarch64: Compiler-based detection of crc32 extended CPU type is broken added
Actions #5

Updated by Loïc Dachary over 7 years ago

  • Status changed from Pending Backport to Resolved
Actions #6

Updated by Kefu Chai over 5 years ago

/usr/bin/aarch64-linux-gnu-g++ -march=armv8-a -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2  -Wall -Wtype-limits -Wignored-qualifiers -Winit-self -Wpointer-arith -Werror=format-security -fno-strict-aliasing -fsigned-char -Wno-unknown-pragmas -rdynamic -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -DHAVE_ARMV8_CRC -fPIE   -o /tmp/foo.o -c /tmp/foo.cxx
/tmp/ccNz2vLJ.s: Assembler messages:
/tmp/ccNz2vLJ.s:23: Error: selected processor does not support `crc32cx w0,w0,x1'

# while +crc compiles

/usr/bin/aarch64-linux-gnu-g++ -march=armv8-a+crc -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2  -Wall -Wtype-limits -Wignored-qualifiers -Winit-self -Wpointer-arith -Werror=format-security -fno-strict-aliasing -fsigned-char -Wno-unknown-pragmas -rdynamic -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -DHAVE_ARMV8_CRC -fPIE   -o /tmp/foo.o -c /tmp/foo.cxx

# /usr/bin/aarch64-linux-gnu-g++ -v
Using built-in specs.
COLLECT_GCC=/usr/bin/aarch64-linux-gnu-g++
COLLECT_LTO_WRAPPER=/usr/lib/gcc/aarch64-linux-gnu/5/lto-wrapper
Target: aarch64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu/Linaro 5.3.1-14ubuntu2' --with-bugurl=file:///usr/share/doc/gcc-5/README.Bugs --enable-languages=c,ada,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-5 --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-libquadmath --enable-plugin --with-system-zlib --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-5-arm64/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-5-arm64 --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-5-arm64 --with-arch-directory=aarch64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-multiarch --enable-fix-cortex-a53-843419 --disable-werror --enable-checking=release --build=aarch64-linux-gnu --host=aarch64-linux-gnu --target=aarch64-linux-gnu
Thread model: posix
gcc version 5.3.1 20160413 (Ubuntu/Linaro 5.3.1-14ubuntu2)

but GCC 5.5 compiles the same source successfully with or without +crc feature specified

/usr/bin/aarch64-linux-gnu-g++-5 -v
Using built-in specs.
COLLECT_GCC=/usr/bin/aarch64-linux-gnu-g++-5
COLLECT_LTO_WRAPPER=/usr/lib/gcc/aarch64-linux-gnu/5/lto-wrapper
Target: aarch64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu/Linaro 5.4.0-6ubuntu1~16.04.10' --with-bugurl=file:///usr/share/doc/gcc-5/README.Bugs --enable-languages=c,ada,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-5 --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-libquadmath --enable-plugin --with-system-zlib --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-5-arm64/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-5-arm64 --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-5-arm64 --with-arch-directory=aarch64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-multiarch --enable-fix-cortex-a53-843419 --disable-werror --enable-checking=release --build=aarch64-linux-gnu --host=aarch64-linux-gnu --target=aarch64-linux-gnu
Thread model: posix
gcc version 5.4.0 20160609 (Ubuntu/Linaro 5.4.0-6ubuntu1~16.04.10)
Actions #7

Updated by Kefu Chai over 5 years ago

  • Status changed from Resolved to 12
  • Backport changed from jewel to luminous
Actions #8

Updated by Kefu Chai over 5 years ago

  • Copied to Backport #36080: luminous: aarch64: Compiler-based detection of crc32 extended CPU type is broken added
Actions #9

Updated by Kefu Chai over 5 years ago

  • Status changed from 12 to Fix Under Review
  • Backport changed from luminous to luminous,mimic
Actions #11

Updated by Nathan Cutler almost 5 years ago

  • Status changed from Fix Under Review to Pending Backport
  • Backport changed from luminous,mimic to jewel,luminous,mimic
Actions #12

Updated by Nathan Cutler almost 5 years ago

  • Copied to Backport #40258: mimic: aarch64: Compiler-based detection of crc32 extended CPU type is broken added
Actions #13

Updated by Nathan Cutler almost 5 years ago

  • Pull request ID set to 24168
Actions #14

Updated by Nathan Cutler over 4 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF