[PATCH] tests: Split out delay

Dmitry V. Levin ldv at altlinux.org
Wed Mar 4 01:58:47 UTC 2020


On Tue, Mar 03, 2020 at 05:35:21PM -0800, Alistair Francis wrote:
> On Tue, Mar 3, 2020 at 5:34 PM Alistair Francis
> <alistair.francis at wdc.com> wrote:
> >
> > The delay test uses the gettimeofday. The gettimeofday does not exist on
> > new 32-bit architectures that have been added since the 5.1 kernel
> > (__ARCH_WANT_TIME32_SYSCALLS is not defined).
> >
> > To avoid running tests for syscalls the kernel doesn't support let's
> > make the delay test dependent on supporting the older syscalls.
> >
> > As new architectures are added (and maybe older ones removed) they will
> > need to set kern_5_1_syscalls_only to "xyes".
> >
> > Signed-off-by: Alistair Francis <alistair.francis at wdc.com>
> 
> Argh, I'm pretty sure I can just use SKIP_MAIN_UNDEFINED.

Yes, you could use SKIP_MAIN_UNDEFINED if it was a test of gettimeofday.

The test in question, however, is the test of delay injection, it just
happens to use __NR_gettimeofday because it was the most convenient way to
implement the test.  If clock_gettime is the only universally available
method of time retrieval nowadays, the test should use it instead.


-- 
ldv


More information about the Strace-devel mailing list