On Fri, Oct 8, 2021 at 6:15 PM Warner Losh <i...@bsdimp.com> wrote: > > Make get_errno and is_error global so files other than syscall.c can use > them. > > Signed-off-by: Warner Losh <i...@bsdimp.com> > Reviewed-by: Richard Henderson <richard.hender...@linaro.org> > Reviewed-by: Philippe Mathieu-Daudé <f4...@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 <kev...@freebsd.org>