[v3,16/19] riscv32: Specify the arch_minimum_kernel as 5.4

Message ID f2b402b31d9aa8cbd5bcbe568c7f5f59cd00aeaa.1594568655.git.alistair.francis@wdc.com
State Committed
Headers
Series glibc port for 32-bit RISC-V (RV32) |

Commit Message

Alistair Francis July 12, 2020, 3:48 p.m. UTC
  ---
 sysdeps/unix/sysv/linux/riscv/configure    | 4 ++++
 sysdeps/unix/sysv/linux/riscv/configure.ac | 4 ++++
 2 files changed, 8 insertions(+)
  

Comments

Maciej W. Rozycki July 15, 2020, 12:06 a.m. UTC | #1
Alistair,

 Is there a specific Linux kernel feature required for RV32 support that 
version 5.4 has added worth mentioning in the change description?

 Otherwise OK.

  Maciej
  
Maciej W. Rozycki July 16, 2020, 1:34 a.m. UTC | #2
On Wed, 15 Jul 2020, Maciej W. Rozycki wrote:

>  Otherwise OK.

 Also I suggest reordering this change and the next ahead of 15/19, so 
that the tree is consistent at the point RV32 builds are enabled with the 
removal of the trap from `sysdeps/riscv/preconfigure'.

  Maciej
  

Patch

diff --git a/sysdeps/unix/sysv/linux/riscv/configure b/sysdeps/unix/sysv/linux/riscv/configure
index 2b3c77f18c..33139618e7 100755
--- a/sysdeps/unix/sysv/linux/riscv/configure
+++ b/sysdeps/unix/sysv/linux/riscv/configure
@@ -162,6 +162,10 @@  if test $libc_cv_riscv_int_abi = no; then
   as_fn_error $? "Unable to determine integer ABI" "$LINENO" 5
 fi
 
+if test $libc_cv_riscv_int_abi = ilp32; then
+  arch_minimum_kernel=5.4.0
+fi
+
 libc_cv_riscv_float_abi=no
 cat confdefs.h - <<_ACEOF >conftest.$ac_ext
 /* end confdefs.h.  */
diff --git a/sysdeps/unix/sysv/linux/riscv/configure.ac b/sysdeps/unix/sysv/linux/riscv/configure.ac
index 710d46afcd..9c736415f7 100644
--- a/sysdeps/unix/sysv/linux/riscv/configure.ac
+++ b/sysdeps/unix/sysv/linux/riscv/configure.ac
@@ -13,6 +13,10 @@  if test $libc_cv_riscv_int_abi = no; then
   AC_MSG_ERROR([Unable to determine integer ABI])
 fi
 
+if test $libc_cv_riscv_int_abi = ilp32; then
+  arch_minimum_kernel=5.4.0
+fi
+
 libc_cv_riscv_float_abi=no
 AC_EGREP_CPP(yes, [#ifdef __riscv_float_abi_double
 		   yes