All of lore.kernel.org
 help / color / mirror / Atom feed
* [LTP] [PATCH] syscalls/io_pgetevents: Use tst_syscall() instead of syscall()
@ 2020-01-29 10:58 Viresh Kumar
  2020-01-29 14:09 ` Xiao Yang
  2020-01-29 14:24 ` Cyril Hrubis
  0 siblings, 2 replies; 3+ messages in thread
From: Viresh Kumar @ 2020-01-29 10:58 UTC (permalink / raw)
  To: ltp

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(-)

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 */
-- 
2.21.0.rc0.269.g1a574e7a288b


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [LTP] [PATCH] syscalls/io_pgetevents: Use tst_syscall() instead of syscall()
  2020-01-29 10:58 [LTP] [PATCH] syscalls/io_pgetevents: Use tst_syscall() instead of syscall() Viresh Kumar
@ 2020-01-29 14:09 ` Xiao Yang
  2020-01-29 14:24 ` Cyril Hrubis
  1 sibling, 0 replies; 3+ messages in thread
From: Xiao Yang @ 2020-01-29 14:09 UTC (permalink / raw)
  To: ltp

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 */


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [LTP] [PATCH] syscalls/io_pgetevents: Use tst_syscall() instead of syscall()
  2020-01-29 10:58 [LTP] [PATCH] syscalls/io_pgetevents: Use tst_syscall() instead of syscall() Viresh Kumar
  2020-01-29 14:09 ` Xiao Yang
@ 2020-01-29 14:24 ` Cyril Hrubis
  1 sibling, 0 replies; 3+ messages in thread
From: Cyril Hrubis @ 2020-01-29 14:24 UTC (permalink / raw)
  To: ltp

Hi!
Pushed, thanks.

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-01-29 14:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-29 10:58 [LTP] [PATCH] syscalls/io_pgetevents: Use tst_syscall() instead of syscall() Viresh Kumar
2020-01-29 14:09 ` Xiao Yang
2020-01-29 14:24 ` Cyril Hrubis

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.