[v2,4/6] nptl: Use recent additions to libsupport in tst-sem5
diff mbox series

Message ID 357cccd801fc058ed6524664e6c22fb4246f9409.1554665560.git-series.mac@mcrowe.com
State New
Headers show
Series
  • Add new helper functions+macros to libsupport and use them in some nptl tests
Related show

Commit Message

Mike Crowe April 7, 2019, 7:33 p.m. UTC
* nptl/tst-sem5.c(do_test): Use xclock_gettime, timespec_add and
	TEST_TIMESPEC_NOW_OR_AFTER from libsupport.
---
 ChangeLog       |  5 +++++
 nptl/tst-sem5.c | 23 +++++------------------
 2 files changed, 10 insertions(+), 18 deletions(-)

Comments

Adhemerval Zanella April 23, 2019, 5:44 p.m. UTC | #1
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.

Patch
diff mbox series

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));
 
   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;
 }