[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