[LTP] [PATCH 2/3] epoll_pwait2: Add test for epoll_pwait201
Cyril Hrubis
chrubis@suse.cz
Mon Jul 12 14:23:54 CEST 2021
Hi!
> +static void run(void)
> +{
> + TEST(tst_syscall(__NR_epoll_pwait2, efd, &e, 1, NULL, NULL, 0));
It would be better to put the prototype for epoll_pwait2 into
include/lapi/epoll.h.
Also given that the difference between epoll_pwait() and epoll_pwait2()
is only in the timeout parameter type it would make much more sense to
write these tests with two test variants so that we would cover both
epoll_pwait() and epoll_wait2().
I would have put these tests into epoll_wait/ directory and added
epoll_var.h that would make a common wrapper for both of these
functions. We do this for example for select in select/select_var.h but
in this case this would be much easier, I guess that the wrapper would
get timespec structure and convert that to miliseconds for
epoll_pwait().
--
Cyril Hrubis
chrubis@suse.cz
More information about the ltp
mailing list