[RFC,v6,20/23] riscv32: Specify the arch_minimum_kernel as 5.4

Message ID 854f3cf104857caa5180841e9d355c333b6778fc.1578824547.git.alistair.francis@wdc.com
State Committed
Headers

Commit Message

Alistair Francis Jan. 12, 2020, 10:34 a.m. UTC
  ---
 sysdeps/unix/sysv/linux/riscv/configure    | 1 +
 sysdeps/unix/sysv/linux/riscv/configure.ac | 4 ++++
 2 files changed, 5 insertions(+)
  

Comments

Andreas Schwab Jan. 12, 2020, 10:56 a.m. UTC | #1
On Jan 12 2020, Alistair Francis wrote:

> diff --git a/sysdeps/unix/sysv/linux/riscv/configure b/sysdeps/unix/sysv/linux/riscv/configure
> index 2b3c77f18c..db09edad1d 100755
> --- a/sysdeps/unix/sysv/linux/riscv/configure
> +++ b/sysdeps/unix/sysv/linux/riscv/configure
> @@ -155,6 +155,7 @@ _ACEOF
>  if (eval "$ac_cpp conftest.$ac_ext") 2>&5 |
>    $EGREP "4 4 4" >/dev/null 2>&1; then :
>    libc_cv_riscv_int_abi=ilp32
> +  arch_minimum_kernel=5.4.0
>  fi
>  rm -f conftest*
>  
> 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

That doesn't fit together.

Andreas.
  
Alistair Francis Jan. 14, 2020, 7:25 a.m. UTC | #2
On Sun, Jan 12, 2020 at 8:56 PM Andreas Schwab <schwab@linux-m68k.org> wrote:
>
> On Jan 12 2020, Alistair Francis wrote:
>
> > diff --git a/sysdeps/unix/sysv/linux/riscv/configure b/sysdeps/unix/sysv/linux/riscv/configure
> > index 2b3c77f18c..db09edad1d 100755
> > --- a/sysdeps/unix/sysv/linux/riscv/configure
> > +++ b/sysdeps/unix/sysv/linux/riscv/configure
> > @@ -155,6 +155,7 @@ _ACEOF
> >  if (eval "$ac_cpp conftest.$ac_ext") 2>&5 |
> >    $EGREP "4 4 4" >/dev/null 2>&1; then :
> >    libc_cv_riscv_int_abi=ilp32
> > +  arch_minimum_kernel=5.4.0
> >  fi
> >  rm -f conftest*
> >
> > 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
>
> That doesn't fit together.

I'm not clear what you mean. Which part?

Alistair

>
> Andreas.
>
> --
> Andreas Schwab, schwab@linux-m68k.org
> GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510  2552 DF73 E780 A9DA AEC1
> "And now for something completely different."
  
Andreas Schwab Jan. 14, 2020, 9:03 a.m. UTC | #3
On Jan 14 2020, Alistair Francis wrote:

> On Sun, Jan 12, 2020 at 8:56 PM Andreas Schwab <schwab@linux-m68k.org> wrote:
>>
>> On Jan 12 2020, Alistair Francis wrote:
>>
>> > diff --git a/sysdeps/unix/sysv/linux/riscv/configure b/sysdeps/unix/sysv/linux/riscv/configure
>> > index 2b3c77f18c..db09edad1d 100755
>> > --- a/sysdeps/unix/sysv/linux/riscv/configure
>> > +++ b/sysdeps/unix/sysv/linux/riscv/configure
>> > @@ -155,6 +155,7 @@ _ACEOF
>> >  if (eval "$ac_cpp conftest.$ac_ext") 2>&5 |
>> >    $EGREP "4 4 4" >/dev/null 2>&1; then :
>> >    libc_cv_riscv_int_abi=ilp32
>> > +  arch_minimum_kernel=5.4.0
>> >  fi
>> >  rm -f conftest*
>> >
>> > 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
>>
>> That doesn't fit together.
>
> I'm not clear what you mean. Which part?

Both parts.

Andreas.
  

Patch

diff --git a/sysdeps/unix/sysv/linux/riscv/configure b/sysdeps/unix/sysv/linux/riscv/configure
index 2b3c77f18c..db09edad1d 100755
--- a/sysdeps/unix/sysv/linux/riscv/configure
+++ b/sysdeps/unix/sysv/linux/riscv/configure
@@ -155,6 +155,7 @@  _ACEOF
 if (eval "$ac_cpp conftest.$ac_ext") 2>&5 |
   $EGREP "4 4 4" >/dev/null 2>&1; then :
   libc_cv_riscv_int_abi=ilp32
+  arch_minimum_kernel=5.4.0
 fi
 rm -f conftest*
 
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