[PATCH 0/2] tools/nolibc: riscv: Fix up compile error for rv32

From: Zhangjin Wu
Date: Thu May 18 2023 - 13:00:54 EST


Hi, Willy

nolibc for riscv is only tested for rv64 currently (see
tools/testing/selftests/nolibc/Makefile), this patchset tries to let it
compile for rv32, but still not pass the nolibc selftest:

* The first patch uses lw/sw instead of ld/sd for rv32 and verse-vice for rv64
* This patch may conflict with the stackprotector patch [1], because
both of them changed the _start assembly in arch-riscv.h

* The second patch adds __NR_llseek based sys_lseek implementation for rv32
* There is no __NR_lseek for rv32, see include/uapi/asm-generic/unistd.h
* This code is based on the version from glibc, sysdeps/unix/sysv/linux/lseek.c
* It passed the two lseek tests in nolibc selftest (write a test case manually)

* To let it compile for rv32, we still need to apply one of such actions:
* Revert the kernel commit d4c08b9776b3 ("riscv: Use latest system call ABI"),
but it is not the right direction, that commit has removed all of the time32 syscalls,
and let C lib (e.g. glibc) provide the same C APIs based on the other time64 syscalls

* If not really use any of the time32 syscalls, defining __ARCH_WANT_TIME32_SYSCALLS
macro will let it compile, but this is buggy for the current implmentations are based
on time32 syscalls!

* Really implement the C APIs for rv32, based on the time64 syscalls, just like glibc.
This commit c8ce48f06503 ("asm-generic: Make time32 syscall numbers optional") shows
us which functions should be re-implemented.

So, the work todo for rv32 is:

* Rebasing all of the old time32 syscalls based C APIs on the new time64 syscalls,
but they are not simply mapped one by one, glibc is a good reference.

* Add standalone rv32 test support in tools/testing/selftests/nolibc/

Best Regards,
Zhangjin Wu

[1]: https://lore.kernel.org/linux-riscv/mhng-1ec176a9-ec5d-470b-a278-a4e9cec728a8@palmer-ri-x1c9a/

Zhangjin Wu (2):
tools/nolibc: riscv: Fix up load/store instructions for rv32
tools/nolibc: riscv: Support __NR_llseek for rv32

tools/include/nolibc/arch-riscv.h | 14 +++++++++-----
tools/include/nolibc/std.h | 1 +
tools/include/nolibc/sys.h | 19 +++++++++++++++++++
3 files changed, 29 insertions(+), 5 deletions(-)

--
2.25.1