LoongArch: vDSO: Correctly use asm parameters in syscall wrappers
authorThomas Weißschuh <thomas.weissschuh@linutronix.de>
Thu, 5 Jun 2025 12:34:18 +0000 (20:34 +0800)
committerHuacai Chen <chenhuacai@loongson.cn>
Fri, 6 Jun 2025 10:51:16 +0000 (18:51 +0800)
commite242bbbb6d7ac7556aa1e358294dc7e3c82cc902
tree085e06d75972a133c3d1cb2bc6b83416d52a3c8a
parentee084fa96123ede8b0563a1b5a9b23adc43cd50d
LoongArch: vDSO: Correctly use asm parameters in syscall wrappers

The syscall wrappers use the "a0" register for two different register
variables, both the first argument and the return value. Here the "ret"
variable is used as both input and output while the argument register is
only used as input. Clang treats the conflicting input parameters as an
undefined behaviour and optimizes away the argument assignment.

The code seems to work by chance for the most part today but that may
change in the future. Specifically clock_gettime_fallback() fails with
clockids from 16 to 23, as implemented by the upcoming auxiliary clocks.

Switch the "ret" register variable to a pure output, similar to the
other architectures' vDSO code. This works in both clang and GCC.

Link: https://lore.kernel.org/lkml/20250602102825-42aa84f0-23f1-4d10-89fc-e8bbaffd291a@linutronix.de/
Link: https://lore.kernel.org/lkml/20250519082042.742926976@linutronix.de/
Fixes: c6b99bed6b8f ("LoongArch: Add VDSO and VSYSCALL support")
Fixes: 18efd0b10e0f ("LoongArch: vDSO: Wire up getrandom() vDSO implementation")
Cc: stable@vger.kernel.org
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
Reviewed-by: Yanteng Si <si.yanteng@linux.dev>
Reviewed-by: WANG Xuerui <git@xen0n.name>
Reviewed-by: Xi Ruoyao <xry111@xry111.site>
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
Signed-off-by: Huacai Chen <chenhuacai@loongson.cn>
arch/loongarch/include/asm/vdso/getrandom.h
arch/loongarch/include/asm/vdso/gettimeofday.h