mirror of
https://gitee.com/bianbu-linux/linux-6.6
synced 2025-04-24 14:07:52 -04:00
tools/nolibc: sys.h: add a syscall return helper
Most of the library routines share the same syscall return logic: In general, a 0 return value indicates success. A -1 return value indicates an error, and an error number is stored in errno. [1] Let's add a __sysret() helper for the above logic to simplify the coding and shrink the code lines too. Thomas suggested to use inline function instead of macro for __sysret(). Willy suggested to make __sysret() be always inline. [1]: https://man7.org/linux/man-pages/man2/syscall.2.html Suggested-by: Willy Tarreau <w@1wt.eu> Link: https://lore.kernel.org/linux-riscv/ZH1+hkhiA2+ItSvX@1wt.eu/ Suggested-by: Thomas Weißschuh <linux@weissschuh.net> Link: https://lore.kernel.org/linux-riscv/ea4e7442-7223-4211-ba29-70821e907888@t-8ch.de/ Reviewed-by: Thomas Weißschuh <linux@weissschuh.net> Signed-off-by: Zhangjin Wu <falcon@tinylab.org> Signed-off-by: Willy Tarreau <w@1wt.eu>
This commit is contained in:
parent
2f98aca8aa
commit
428905da6e
1 changed files with 10 additions and 0 deletions
|
@ -28,6 +28,16 @@
|
||||||
#include "errno.h"
|
#include "errno.h"
|
||||||
#include "types.h"
|
#include "types.h"
|
||||||
|
|
||||||
|
/* Syscall return helper, set errno as -ret when ret < 0 */
|
||||||
|
static __inline__ __attribute__((unused, always_inline))
|
||||||
|
long __sysret(long ret)
|
||||||
|
{
|
||||||
|
if (ret < 0) {
|
||||||
|
SET_ERRNO(-ret);
|
||||||
|
ret = -1;
|
||||||
|
}
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
|
||||||
/* Functions in this file only describe syscalls. They're declared static so
|
/* Functions in this file only describe syscalls. They're declared static so
|
||||||
* that the compiler usually decides to inline them while still being allowed
|
* that the compiler usually decides to inline them while still being allowed
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue