[LTP] [PATCH] timer_create01: accept ENOTSUP
Cyril Hrubis
chrubis@suse.cz
Fri Sep 27 16:49:02 CEST 2019
Hi!
> CLOCK_REALTIME_ALARM and CLOCK_BOOTTIME_ALARM require an RTC with alarm
> support, which may not be present on a system. In that case, the kernel
> will return EOPNOTSUPP, which is defined as ENOTSUP in userspace.
>
> As these clocks are already possibly unsupported, accept that as an error
> code besides the EINVAL code.
>
> Note that previously the kernel would incorrectly return the ENOTSUPP code,
> which is undefined in userspace. With these unfixed kernels, the test will
> fail when those RTCs are not present.
Can you please include the kernel commit id that fixed the kernel here?
Otherwise this patch is fine.
> Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>
> ---
> testcases/kernel/syscalls/timer_create/timer_create01.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/testcases/kernel/syscalls/timer_create/timer_create01.c b/testcases/kernel/syscalls/timer_create/timer_create01.c
> index 258b6444c362..1ae5b9b2d1c2 100644
> --- a/testcases/kernel/syscalls/timer_create/timer_create01.c
> +++ b/testcases/kernel/syscalls/timer_create/timer_create01.c
> @@ -77,7 +77,8 @@ static void run(unsigned int n)
> &created_timer_id));
>
> if (TST_RET != 0) {
> - if (possibly_unsupported(clock) && TST_ERR == EINVAL) {
> + if (possibly_unsupported(clock) &&
> + (TST_ERR == EINVAL || TST_ERR == ENOTSUP)) {
> tst_res(TPASS | TTERRNO,
> "%s unsupported, failed as expected",
> get_clock_str(clock));
> --
> 2.20.1
>
>
> --
> Mailing list info: https://lists.linux.it/listinfo/ltp
--
Cyril Hrubis
chrubis@suse.cz
More information about the ltp
mailing list