This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v2 4/6] nptl: Use recent additions to libsupport in tst-sem5
LGTM the above change.
On 07/04/2019 16:33, Mike Crowe wrote:
> * nptl/tst-sem5.c(do_test): Use xclock_gettime, timespec_add and
> TEST_TIMESPEC_NOW_OR_AFTER from libsupport.
Ok.
> ---
> ChangeLog | 5 +++++
> nptl/tst-sem5.c | 23 +++++------------------
> 2 files changed, 10 insertions(+), 18 deletions(-)
>
> diff --git a/ChangeLog b/ChangeLog
> index 5aafd69..c78863e 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,5 +1,10 @@
> 2019-04-06 Mike Crowe <mac@mcrowe.com>
>
> + * nptl/tst-sem5.c(do_test): Use xclock_gettime, timespec_add and
> + TEST_TIMESPEC_NOW_OR_AFTER from libsupport.
> +
> +2019-04-06 Mike Crowe <mac@mcrowe.com>
> +
> * nptl/tst-cond11.c: Use libsupport.
>
> 2019-04-06 Mike Crowe <mac@mcrowe.com>
> diff --git a/nptl/tst-sem5.c b/nptl/tst-sem5.c
> index 50ab6f9..7e722bd 100644
> --- a/nptl/tst-sem5.c
> +++ b/nptl/tst-sem5.c
> @@ -22,6 +22,8 @@
> #include <unistd.h>
> #include <sys/time.h>
> #include <support/check.h>
> +#include <support/timespec.h>
> +#include <support/xtime.h>
>
>
> static int
> @@ -29,31 +31,16 @@ do_test (void)
> {
> sem_t s;
> struct timespec ts;
> - struct timeval tv;
>
> TEST_COMPARE (sem_init (&s, 0, 1), 0);
> TEST_COMPARE (TEMP_FAILURE_RETRY (sem_wait (&s)), 0);
> - TEST_COMPARE (gettimeofday (&tv, NULL), 0);
> -
> - TIMEVAL_TO_TIMESPEC (&tv, &ts);
> -
> - /* We wait for half a second. */
> - ts.tv_nsec += 500000000;
> - if (ts.tv_nsec >= 1000000000)
> - {
> - ++ts.tv_sec;
> - ts.tv_nsec -= 1000000000;
> - }
> + xclock_gettime (CLOCK_REALTIME, &ts);
> + ts = timespec_add (ts, make_timespec (0, TIMESPEC_HZ/2));
Don't remove the comment.
>
> errno = 0;
> TEST_COMPARE (TEMP_FAILURE_RETRY (sem_timedwait (&s, &ts)), -1);
> TEST_COMPARE (errno, ETIMEDOUT);
> -
> - struct timespec ts2;
> - TEST_COMPARE (clock_gettime (CLOCK_REALTIME, &ts2), 0);
> -
> - TEST_VERIFY (ts2.tv_sec > ts.tv_sec
> - || (ts2.tv_sec == ts.tv_sec && ts2.tv_nsec > ts.tv_nsec));
> + TEST_TIMESPEC_NOW_OR_AFTER (CLOCK_REALTIME, ts);
>
> return 0;
> }
>
Ok.