svn commit: r358393 - in head/sys: conf riscv/include
Warner Losh
imp at FreeBSD.org
Thu Feb 27 15:34:32 UTC 2020
Author: imp
Date: Thu Feb 27 15:34:30 2020
New Revision: 358393
URL: https://svnweb.freebsd.org/changeset/base/358393
Log:
Better check for floating point type.
Use __riscv_flen instead of __riscv_float_abi_soft. While the latter works for
userland (and one could argue it's more correct), it fails for the kernel. We
compile the kernel with -mabi=lp64 (eg soft float abi) to avoid floating point
instructions in the kernel. We also compile the kernel -march=rv64imafdc for
hard float kernels (eg those with options FPE), but with -march=rv64imac for
softfloat kernels (eg those with FPE). Since we do this, in the kernel (as in
userland) __riscv_flen will be defined for 'riscv64' and not for 'riscv64sf'.
This also removes the -DMACHINE_ARCH hack now that it's no longer needed.
Longer term, we should return the ABI from the sysctl hw.machine_arch like on
amd64 for i386 binaries.
Suggested by: mhorne@
Differential Revision: https://reviews.freebsd.org/D23813
Modified:
head/sys/conf/Makefile.riscv
head/sys/conf/kern.mk
head/sys/riscv/include/param.h
Modified: head/sys/conf/Makefile.riscv
==============================================================================
--- head/sys/conf/Makefile.riscv Thu Feb 27 15:30:13 2020 (r358392)
+++ head/sys/conf/Makefile.riscv Thu Feb 27 15:34:30 2020 (r358393)
@@ -46,10 +46,6 @@ SYSTEM_LD= @${LD} -N -m ${LD_EMULATION} -Bdynamic -T $
CFLAGS += -fno-omit-frame-pointer -fno-optimize-sibling-calls
.endif
-# Currently, the compile flags don't let the kernel know if this is a hard-float
-# ABI build or a soft-float ABI build. We need to pass in this information.
-CFLAGS += -DMACHINE_ARCH=\"${MACHINE_ARCH}\"
-
# hack because genassym.c includes sys/bus.h which includes these.
genassym.o: bus_if.h device_if.h
Modified: head/sys/conf/kern.mk
==============================================================================
--- head/sys/conf/kern.mk Thu Feb 27 15:30:13 2020 (r358392)
+++ head/sys/conf/kern.mk Thu Feb 27 15:34:30 2020 (r358393)
@@ -139,14 +139,23 @@ INLINE_LIMIT?= 8000
#
# For RISC-V we specify the soft-float ABI (lp64) to avoid the use of floating
-# point registers within the kernel. We also specify the "medium" code model,
-# which generates code suitable for a 2GiB addressing range located at any
-# offset, allowing modules to be located anywhere in the 64-bit address space.
-# Note that clang and GCC refer to this code model as "medium" and "medany"
-# respectively.
+# point registers within the kernel. However, for kernels supporting hardware
+# float (FPE), we have to include that in the march so we can have limited
+# floating point support in context switching needed for that. This is different
+# than userland where we use a hard-float ABI (lp64d).
#
+# We also specify the "medium" code model, which generates code suitable for a
+# 2GiB addressing range located at any offset, allowing modules to be located
+# anywhere in the 64-bit address space. Note that clang and GCC refer to this
+# code model as "medium" and "medany" respectively.
+#
.if ${MACHINE_CPUARCH} == "riscv"
-CFLAGS+= -march=rv64imafdc -mabi=lp64
+.if ${MACHINE_ARCH:Mriscv*sf}
+CFLAGS+= -march=rv64imac
+.else
+CFLAGS+= -march=rv64imafdc
+.endif
+CFLAGS+= -mabi=lp64
CFLAGS.clang+= -mcmodel=medium
CFLAGS.gcc+= -mcmodel=medany
INLINE_LIMIT?= 8000
Modified: head/sys/riscv/include/param.h
==============================================================================
--- head/sys/riscv/include/param.h Thu Feb 27 15:30:13 2020 (r358392)
+++ head/sys/riscv/include/param.h Thu Feb 27 15:34:30 2020 (r358393)
@@ -46,10 +46,17 @@
#define MACHINE "riscv"
#endif
#ifndef MACHINE_ARCH
-#ifdef __riscv_float_abi_soft
-#define MACHINE_ARCH "riscv64sf"
-#else
+/*
+ * Check to see if we're building with hardware floating instructions
+ * allowed. We check this instead of hard vs soft float ABI because we build the
+ * kernel with soft float ABI to avoid hard float instruction generation. If
+ * we ever allow a 'soft ABI but with hard floats' userland, then we'll need
+ * to rethink this.
+ */
+#ifdef __riscv_flen
#define MACHINE_ARCH "riscv64"
+#else
+#define MACHINE_ARCH "riscv64sf"
#endif
#endif
More information about the svn-src-all
mailing list