Skip to contentSkip to conversation

compiler.h: Raise minimum version of GCC to 5.1 for arm64

STABLE BACKPORT 4.19.y and 5.14.y Will Deacon 2 participants 2 messages Jan 18, 2021 – Jan 20, 2021 Stable branch

lore
Share & tools
lore
#0 lore raw

commit dca5244d2f5b94f1809f0c02a549edf41ccd5493 upstream.

GCC versions >= 4.9 and < 5.1 have been shown to emit memory references beyond the stack pointer, resulting in memory corruption if an interrupt is taken after the stack pointer has been adjusted but before the reference has been executed. This leads to subtle, infrequent data corruption such as the EXT4 problems reported by Russell King at the link below.

Life is too short for buggy compilers, so raise the minimum GCC version required by arm64 to 5.1.

---
 include/linux/compiler-gcc.h | 6 ++++++
 1 file changed, 6 insertions(+)
include/linux/compiler-gcc.h +6 -0
index 14be09537109..a80d6de3c8ad 100644
@@ -12,6 +12,12 @@
 
 #if GCC_VERSION < 40600
 # error Sorry, your compiler is too old - please upgrade it.
+#elif defined(CONFIG_ARM64) && GCC_VERSION < 50100
+/*
+ * https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63293
+ * https://lore.kernel.org/r/[email protected]
+ */
+# error Sorry, your version of GCC is too old - please use 5.1 or newer.
 #endif
 
 /*
Signature · 1 line
2.30.0.284.gd98b1dd5eaa7-goog
maintainer
#1 lore raw
Quoted 12 lines · Will Deacon

commit dca5244d2f5b94f1809f0c02a549edf41ccd5493 upstream.

GCC versions >= 4.9 and < 5.1 have been shown to emit memory references beyond the stack pointer, resulting in memory corruption if an interrupt is taken after the stack pointer has been adjusted but before the reference has been executed. This leads to subtle, infrequent data corruption such as the EXT4 problems reported by Russell King at the link below.

Life is too short for buggy compilers, so raise the minimum GCC version required by arm64 to 5.1.

Quoted 8 lines

All now queued up, thanks!

greg k-h