[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 05/15] bsd-user: export get_errno and is_error from syscal
From: |
Kyle Evans |
Subject: |
Re: [PATCH v2 05/15] bsd-user: export get_errno and is_error from syscall.c |
Date: |
Sun, 17 Oct 2021 22:33:23 -0500 |
On Fri, Oct 8, 2021 at 6:15 PM Warner Losh <imp@bsdimp.com> wrote:
>
> Make get_errno and is_error global so files other than syscall.c can use
> them.
>
> Signed-off-by: Warner Losh <imp@bsdimp.com>
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> bsd-user/qemu.h | 4 ++++
> bsd-user/syscall.c | 10 +++++-----
> 2 files changed, 9 insertions(+), 5 deletions(-)
>
> diff --git a/bsd-user/qemu.h b/bsd-user/qemu.h
> index 522d6c4031..3b8475394c 100644
> --- a/bsd-user/qemu.h
> +++ b/bsd-user/qemu.h
> @@ -235,6 +235,10 @@ extern unsigned long target_dflssiz;
> extern unsigned long target_maxssiz;
> extern unsigned long target_sgrowsiz;
>
> +/* syscall.c */
> +abi_long get_errno(abi_long ret);
> +bool is_error(abi_long ret);
> +
> /* user access */
>
> #define VERIFY_READ PAGE_READ
> diff --git a/bsd-user/syscall.c b/bsd-user/syscall.c
> index 372836d44d..2fd2ba8330 100644
> --- a/bsd-user/syscall.c
> +++ b/bsd-user/syscall.c
> @@ -33,18 +33,18 @@
> static abi_ulong target_brk;
> static abi_ulong target_original_brk;
>
> -static inline abi_long get_errno(abi_long ret)
> +abi_long get_errno(abi_long ret)
> {
> - if (ret == -1)
> + if (ret == -1) {
> /* XXX need to translate host -> target errnos here */
> return -(errno);
> - else
> - return ret;
> + }
> + return ret;
> }
>
> #define target_to_host_bitmask(x, tbl) (x)
>
> -static inline int is_error(abi_long ret)
> +bool is_error(abi_long ret)
> {
> return (abi_ulong)ret >= (abi_ulong)(-4096);
> }
> --
> 2.32.0
>
Reviewed-by: Kyle Evans <kevans@FreeBSD.org>
- Re: [PATCH v2 01/15] meson: *-user: only descend into *-user when configured, (continued)
Re: [PATCH v2 01/15] meson: *-user: only descend into *-user when configured, Philippe Mathieu-Daudé, 2021/10/18
[PATCH v2 03/15] bsd-user/strace.list: Remove support for FreeBSD versions older than 12.0, Warner Losh, 2021/10/08
[PATCH v2 02/15] bsd-user/target_os-user.h: Remove support for FreeBSD older than 12.0, Warner Losh, 2021/10/08
[PATCH v2 05/15] bsd-user: export get_errno and is_error from syscall.c, Warner Losh, 2021/10/08
- Re: [PATCH v2 05/15] bsd-user: export get_errno and is_error from syscall.c,
Kyle Evans <=
[PATCH v2 04/15] bsd-user: TARGET_RESET define is unused, remove it, Warner Losh, 2021/10/08
[PATCH v2 09/15] bsd-user/target_os_elf: If ELF_HWCAP2 is defined, publish it, Warner Losh, 2021/10/08
[PATCH v2 06/15] bsd-user/errno_defs.h: Add internal error numbers, Warner Losh, 2021/10/08
[PATCH v2 07/15] bsd-user: move TARGET_MC_GET_CLEAR_RET to target_os_signal.h, Warner Losh, 2021/10/08
[PATCH v2 10/15] bsd-user: Remove used from TaskState, Warner Losh, 2021/10/08