diff mbox series

syscalls/io_pgetevents: Use tst_syscall() instead of syscall()

Message ID 846e4f42d5112c2f945c391644949a14756f53f6.1580295508.git.viresh.kumar@linaro.org
State Accepted
Headers show
Series syscalls/io_pgetevents: Use tst_syscall() instead of syscall() | expand

Commit Message

Viresh Kumar Jan. 29, 2020, 10:58 a.m. UTC
Use the preferred helper tst_syscall() instead of syscall(), as it also
takes care of error handling.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
---
 include/lapi/io_pgetevents.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Xiao Yang Jan. 29, 2020, 2:09 p.m. UTC | #1
On 1/29/20 6:58 PM, Viresh Kumar wrote:
> Use the preferred helper tst_syscall() instead of syscall(), as it also
> takes care of error handling.

Hi,

It is obviously correct.

Reviewed-by: Xiao Yang <ice_yangxiao@163.com>

Best Regards,

Xiao Yang

>
> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
> ---
>   include/lapi/io_pgetevents.h | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/include/lapi/io_pgetevents.h b/include/lapi/io_pgetevents.h
> index 9c7f0ec9d813..3c9d5b2d7618 100644
> --- a/include/lapi/io_pgetevents.h
> +++ b/include/lapi/io_pgetevents.h
> @@ -21,7 +21,8 @@ int io_pgetevents(io_context_t ctx, long min_nr, long max_nr,
>   		 struct io_event *events, struct timespec *timeout,
>   		 sigset_t *sigmask)
>   {
> -	return syscall(__NR_io_pgetevents, ctx, min_nr, max_nr, events, timeout, sigmask);
> +	return tst_syscall(__NR_io_pgetevents, ctx, min_nr, max_nr, events,
> +			   timeout, sigmask);
>   }
>   #endif /* HAVE_IO_PGETEVENTS */
>   #endif /* HAVE_LIBAIO */
Cyril Hrubis Jan. 29, 2020, 2:24 p.m. UTC | #2
Hi!
Pushed, thanks.
diff mbox series

Patch

diff --git a/include/lapi/io_pgetevents.h b/include/lapi/io_pgetevents.h
index 9c7f0ec9d813..3c9d5b2d7618 100644
--- a/include/lapi/io_pgetevents.h
+++ b/include/lapi/io_pgetevents.h
@@ -21,7 +21,8 @@  int io_pgetevents(io_context_t ctx, long min_nr, long max_nr,
 		 struct io_event *events, struct timespec *timeout,
 		 sigset_t *sigmask)
 {
-	return syscall(__NR_io_pgetevents, ctx, min_nr, max_nr, events, timeout, sigmask);
+	return tst_syscall(__NR_io_pgetevents, ctx, min_nr, max_nr, events,
+			   timeout, sigmask);
 }
 #endif /* HAVE_IO_PGETEVENTS */
 #endif /* HAVE_LIBAIO */