Seperate real and user timer interfaces
authorjca <jca@openbsd.org>
Tue, 5 Dec 2017 15:02:06 +0000 (15:02 +0000)
committerjca <jca@openbsd.org>
Tue, 5 Dec 2017 15:02:06 +0000 (15:02 +0000)
Use more descriptive names, and make it clearer that real and user
timers work on different static storage.  The end goal is to be able to
reuse those timer functions, instead of inlining other timer
implementations subject to clock jumps.

Discussed with Scott Cheloha

usr.bin/openssl/apps.h
usr.bin/openssl/apps_posix.c
usr.bin/openssl/s_time.c
usr.bin/openssl/speed.c

index 4276e53..d02169b 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: apps.h,v 1.19 2016/08/30 14:34:59 deraadt Exp $ */
+/* $OpenBSD: apps.h,v 1.20 2017/12/05 15:02:06 jca Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -279,7 +279,8 @@ int app_isdir(const char *);
 
 #define TM_START       0
 #define TM_STOP                1
-double app_tminterval (int stop, int usertime);
+double app_timer_real(int stop);
+double app_timer_user(int stop);
 
 #define OPENSSL_NO_SSL_INTERN
 
index 94c6d35..502919c 100644 (file)
 
 #include "apps.h"
 
-static double
-real_interval(int stop)
+double
+app_timer_real(int stop)
 {
        static struct timespec start;
        struct timespec elapsed, now;
@@ -138,8 +138,8 @@ real_interval(int stop)
        return 0.0;
 }
 
-static double
-user_interval(int stop)
+double
+app_timer_user(int stop)
 {
        static struct timeval start;
        struct timeval elapsed;
@@ -154,12 +154,6 @@ user_interval(int stop)
        return 0.0;
 }
 
-double
-app_tminterval(int stop, int usertime)
-{
-       return (usertime) ? user_interval(stop) : real_interval(stop);
-}
-
 int
 setup_ui(void)
 {
index e7a1ef7..75009f8 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: s_time.c,v 1.18 2017/11/02 00:31:49 mestre Exp $ */
+/* $OpenBSD: s_time.c,v 1.19 2017/12/05 15:02:06 jca Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -233,9 +233,9 @@ s_time_usage(void)
 #define STOP   1
 
 static double
-tm_Time_F(int s)
+tm_Time_F(int op)
 {
-       return app_tminterval(s, 1);
+       return app_timer_user(op);
 }
 
 /***********************************************************************
index 90cf6de..4238b15 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: speed.c,v 1.20 2017/10/07 06:16:54 guenther Exp $ */
+/* $OpenBSD: speed.c,v 1.21 2017/12/05 15:02:06 jca Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -202,7 +202,10 @@ sig_done(int sig)
 static double
 Time_F(int s)
 {
-       return app_tminterval(s, usertime);
+       if (usertime)
+               return app_timer_user(s);
+       else
+               return app_timer_real(s);
 }