[LTP] [PATCH 1/7] getitimer02: Skipped EFAULT tests for libc variant.

Petr Vorel pvorel@suse.cz
Tue Jun 8 22:43:43 CEST 2021


Hi Vinay,

> Tested EFAULT cases only for "__NR_getitimer" syscall.
Well, you added testing raw syscall via test variant.
That's IMHO the most important thing to mention.

In the text after blank line I'd mention EFAULT skipped on raw syscall.
It'd would deserve to mention why.

Also, as test it's quite simple, it'd be nice, if you first convert it to new
API and in second commit added test variants.
During rewrite you should remove #if !defined(UCLINUX) check as getitimer() is
supported on uclibc-ng.

This apply to whole patchset (some tests are already ported to new API).

Kind regards,
Petr


More information about the ltp mailing list