[LTP] [PATCH] Remove ftime and sys/timeb.h

[LTP] [PATCH] Remove ftime and sys/timeb.h

[Yang Xu]

The two case use ftime to get the current milliseconds and it was
used to generate random value. Use gettimeofday() to get the microseconds
can reach the same aim.

This also fixes the travis build error[1] on Fedora:Rawhide because the <sys/timeb.h>
has been deprecated[2].

[1] https://travis-ci.org/github/linux-test-project/ltp/jobs/737698948
[2] https://www.spinics.net/lists/fedora-devel/msg279545.html

Reported-by: Petr Vorel <petr.vorel@gmail.com>
Signed-off-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>
---
 testcases/kernel/mem/hugetlb/lib/hugetlb.c | 12 ++++++------
 testcases/kernel/sched/tool/trace_sched.c  |  8 ++++----
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/testcases/kernel/mem/hugetlb/lib/hugetlb.c b/testcases/kernel/mem/hugetlb/lib/hugetlb.c
index 4bb2d500e..1204f21d2 100644
--- a/testcases/kernel/mem/hugetlb/lib/hugetlb.c
+++ b/testcases/kernel/mem/hugetlb/lib/hugetlb.c
@@ -35,7 +35,7 @@
 #include <sys/types.h>
 #include <sys/ipc.h>
 #include <sys/shm.h>
-#include <sys/timeb.h>
+#include <sys/time.h>
 #include <pwd.h>
 #include "hugetlb.h"
 
@@ -52,7 +52,7 @@ int getipckey(void)
 	char *curdir = NULL;
 	size_t size = 0;
 	key_t ipc_key;
-	struct timeb time_info;
+	struct timeval time_info;
 
 	curdir = getcwd(curdir, size);
 	if (curdir == NULL)
@@ -67,11 +67,11 @@ int getipckey(void)
 	 * project identifier is a "random character" produced by
 	 * generating a random number between 0 and 25 and then adding
 	 * that to the ascii value of 'a'.  The "seed" for the random
-	 * number is the millisecond value that is set in the timeb
-	 * structure after calling ftime().
+	 * number is the microseconds value that is set in the timeval
+	 * structure after calling gettimeofday().
 	 */
-	ftime(&time_info);
-	srandom((unsigned int)time_info.millitm);
+	gettimeofday(&time_info, NULL);
+	srandom((unsigned int)time_info.tv_usec);
 
 	ipc_key = ftok(curdir, ascii_a + random() % 26);
 	if (ipc_key == -1)
diff --git a/testcases/kernel/sched/tool/trace_sched.c b/testcases/kernel/sched/tool/trace_sched.c
index 71caf239a..e23fc7399 100644
--- a/testcases/kernel/sched/tool/trace_sched.c
+++ b/testcases/kernel/sched/tool/trace_sched.c
@@ -52,7 +52,7 @@
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <sys/wait.h>
-#include <sys/timeb.h>
+#include <sys/time.h>
 #include <unistd.h>
 #include <string.h>
 
@@ -199,7 +199,7 @@ void *thread_func(void *args)
 	static int sched_policy;	/* scheduling policy as set by user/default   */
 	struct sched_param ssp;	/* set schedule priority.                     */
 	struct sched_param gsp;	/* gsp schedule priority.                     */
-	struct timeb tptr;	/* tptr.millitm will be used to seed srand.   */
+	struct timeval tptr;	/* tv_usec will be used to seed srand.   */
 	thread_sched_t *locargptr =	/* local ptr to the arguments.                */
 	    (thread_sched_t *) args;
 
@@ -215,8 +215,8 @@ void *thread_func(void *args)
 		ssp.sched_priority = 0;
 	else {
 		/* Set a random value between max_priority and min_priority */
-		ftime(&tptr);
-		srand((tptr.millitm) % 1000);
+		gettimeofday(&tptr, NULL);
+		srand((tptr.tv_usec) % 1000000);
 		set_priority = (min_priority + (int)((float)max_priority
 						     * rand() / (RAND_MAX +
 								 1.0)));
-- 
2.23.0

[LTP] [PATCH] Remove ftime and sys/timeb.h

[Li Wang]

Yang Xu <xuyang2018.jy@cn.fujitsu.com> wrote:

> The two case use ftime to get the current milliseconds and it was
> used to generate random value. Use gettimeofday() to get the microseconds
> can reach the same aim.
>
> This also fixes the travis build error[1] on Fedora:Rawhide because the <sys/timeb.h>
> has been deprecated[2].
>
> [1] https://travis-ci.org/github/linux-test-project/ltp/jobs/737698948
> [2] https://www.spinics.net/lists/fedora-devel/msg279545.html
>
> Reported-by: Petr Vorel <petr.vorel@gmail.com>
> Signed-off-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>

Reviewed-by: Li Wang <liwang@redhat.com>

Travis CI Passed: https://travis-ci.org/github/wangli5665/ltp/builds/737927003

-- 
Regards,
Li Wang

[LTP] [PATCH] Remove ftime and sys/timeb.h

[Jan Stancek]

----- Original Message -----
> The two case use ftime to get the current milliseconds and it was
> used to generate random value. Use gettimeofday() to get the microseconds
> can reach the same aim.
> 
> This also fixes the travis build error[1] on Fedora:Rawhide because the
> <sys/timeb.h>
> has been deprecated[2].
> 
> [1] https://travis-ci.org/github/linux-test-project/ltp/jobs/737698948
> [2] https://www.spinics.net/lists/fedora-devel/msg279545.html
> 
> Reported-by: Petr Vorel <petr.vorel@gmail.com>
> Signed-off-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>

Acked-by: Jan Stancek <jstancek@redhat.com>

[LTP] [PATCH] Remove ftime and sys/timeb.h

[Xiao Yang]

On 2020/10/22 11:57, Yang Xu wrote:
> The two case use ftime to get the current milliseconds and it was
> used to generate random value. Use gettimeofday() to get the microseconds
> can reach the same aim.
>
> This also fixes the travis build error[1] on Fedora:Rawhide because the<sys/timeb.h>
> has been deprecated[2].
>
> [1] https://travis-ci.org/github/linux-test-project/ltp/jobs/737698948
> [2] https://www.spinics.net/lists/fedora-devel/msg279545.html
>
> Reported-by: Petr Vorel<petr.vorel@gmail.com>
> Signed-off-by: Yang Xu<xuyang2018.jy@cn.fujitsu.com>
> ---
>   testcases/kernel/mem/hugetlb/lib/hugetlb.c | 12 ++++++------
>   testcases/kernel/sched/tool/trace_sched.c  |  8 ++++----
>   2 files changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/testcases/kernel/mem/hugetlb/lib/hugetlb.c b/testcases/kernel/mem/hugetlb/lib/hugetlb.c
> index 4bb2d500e..1204f21d2 100644
> --- a/testcases/kernel/mem/hugetlb/lib/hugetlb.c
> +++ b/testcases/kernel/mem/hugetlb/lib/hugetlb.c
> @@ -35,7 +35,7 @@
>   #include<sys/types.h>
>   #include<sys/ipc.h>
>   #include<sys/shm.h>
> -#include<sys/timeb.h>
> +#include<sys/time.h>
>   #include<pwd.h>
>   #include "hugetlb.h"
>
> @@ -52,7 +52,7 @@ int getipckey(void)
>   	char *curdir = NULL;
>   	size_t size = 0;
>   	key_t ipc_key;
> -	struct timeb time_info;
> +	struct timeval time_info;
>
>   	curdir = getcwd(curdir, size);
>   	if (curdir == NULL)
> @@ -67,11 +67,11 @@ int getipckey(void)
>   	 * project identifier is a "random character" produced by
>   	 * generating a random number between 0 and 25 and then adding
>   	 * that to the ascii value of 'a'.  The "seed" for the random
> -	 * number is the millisecond value that is set in the timeb
> -	 * structure after calling ftime().
> +	 * number is the microseconds value that is set in the timeval
> +	 * structure after calling gettimeofday().
>   	 */
> -	ftime(&time_info);
> -	srandom((unsigned int)time_info.millitm);
> +	gettimeofday(&time_info, NULL);
> +	srandom((unsigned int)time_info.tv_usec);
>
>   	ipc_key = ftok(curdir, ascii_a + random() % 26);
>   	if (ipc_key == -1)
> diff --git a/testcases/kernel/sched/tool/trace_sched.c b/testcases/kernel/sched/tool/trace_sched.c
> index 71caf239a..e23fc7399 100644
> --- a/testcases/kernel/sched/tool/trace_sched.c
> +++ b/testcases/kernel/sched/tool/trace_sched.c
> @@ -52,7 +52,7 @@
>   #include<sys/types.h>
>   #include<sys/stat.h>
>   #include<sys/wait.h>
> -#include<sys/timeb.h>
> +#include<sys/time.h>
>   #include<unistd.h>
>   #include<string.h>
>
> @@ -199,7 +199,7 @@ void *thread_func(void *args)
>   	static int sched_policy;	/* scheduling policy as set by user/default   */
>   	struct sched_param ssp;	/* set schedule priority.                     */
>   	struct sched_param gsp;	/* gsp schedule priority.                     */
> -	struct timeb tptr;	/* tptr.millitm will be used to seed srand.   */
> +	struct timeval tptr;	/* tv_usec will be used to seed srand.   */
>   	thread_sched_t *locargptr =	/* local ptr to the arguments.                */
>   	    (thread_sched_t *) args;
>
> @@ -215,8 +215,8 @@ void *thread_func(void *args)
>   		ssp.sched_priority = 0;
>   	else {
>   		/* Set a random value between max_priority and min_priority */
> -		ftime(&tptr);
> -		srand((tptr.millitm) % 1000);
> +		gettimeofday(&tptr, NULL);
> +		srand((tptr.tv_usec) % 1000000);
Hi Yang,

One minor issue:
'% 1000000' seems useless because the range of tv_usec is 0~999999.

Other than that, it's good to me.

Best Regards,
Xiao Yang
>   		set_priority = (min_priority + (int)((float)max_priority
>   						     * rand() / (RAND_MAX +
>   								 1.0)));

[LTP] [PATCH] Remove ftime and sys/timeb.h

[Yang Xu]

Hi Xiao
> On 2020/10/22 11:57, Yang Xu wrote:
>> The two case use ftime to get the current milliseconds and it was
>> used to generate random value. Use gettimeofday() to get the microseconds
>> can reach the same aim.
>>
>> This also fixes the travis build error[1] on Fedora:Rawhide because
>> the<sys/timeb.h>
>> has been deprecated[2].
>>
>> [1] https://travis-ci.org/github/linux-test-project/ltp/jobs/737698948
>> [2] https://www.spinics.net/lists/fedora-devel/msg279545.html
>>
>> Reported-by: Petr Vorel<petr.vorel@gmail.com>
>> Signed-off-by: Yang Xu<xuyang2018.jy@cn.fujitsu.com>
>> ---
>> testcases/kernel/mem/hugetlb/lib/hugetlb.c | 12 ++++++------
>> testcases/kernel/sched/tool/trace_sched.c | 8 ++++----
>> 2 files changed, 10 insertions(+), 10 deletions(-)
>>
>> diff --git a/testcases/kernel/mem/hugetlb/lib/hugetlb.c
>> b/testcases/kernel/mem/hugetlb/lib/hugetlb.c
>> index 4bb2d500e..1204f21d2 100644
>> --- a/testcases/kernel/mem/hugetlb/lib/hugetlb.c
>> +++ b/testcases/kernel/mem/hugetlb/lib/hugetlb.c
>> @@ -35,7 +35,7 @@
>> #include<sys/types.h>
>> #include<sys/ipc.h>
>> #include<sys/shm.h>
>> -#include<sys/timeb.h>
>> +#include<sys/time.h>
>> #include<pwd.h>
>> #include "hugetlb.h"
>>
>> @@ -52,7 +52,7 @@ int getipckey(void)
>> char *curdir = NULL;
>> size_t size = 0;
>> key_t ipc_key;
>> - struct timeb time_info;
>> + struct timeval time_info;
>>
>> curdir = getcwd(curdir, size);
>> if (curdir == NULL)
>> @@ -67,11 +67,11 @@ int getipckey(void)
>> * project identifier is a "random character" produced by
>> * generating a random number between 0 and 25 and then adding
>> * that to the ascii value of 'a'. The "seed" for the random
>> - * number is the millisecond value that is set in the timeb
>> - * structure after calling ftime().
>> + * number is the microseconds value that is set in the timeval
>> + * structure after calling gettimeofday().
>> */
>> - ftime(&time_info);
>> - srandom((unsigned int)time_info.millitm);
>> + gettimeofday(&time_info, NULL);
>> + srandom((unsigned int)time_info.tv_usec);
>>
>> ipc_key = ftok(curdir, ascii_a + random() % 26);
>> if (ipc_key == -1)
>> diff --git a/testcases/kernel/sched/tool/trace_sched.c
>> b/testcases/kernel/sched/tool/trace_sched.c
>> index 71caf239a..e23fc7399 100644
>> --- a/testcases/kernel/sched/tool/trace_sched.c
>> +++ b/testcases/kernel/sched/tool/trace_sched.c
>> @@ -52,7 +52,7 @@
>> #include<sys/types.h>
>> #include<sys/stat.h>
>> #include<sys/wait.h>
>> -#include<sys/timeb.h>
>> +#include<sys/time.h>
>> #include<unistd.h>
>> #include<string.h>
>>
>> @@ -199,7 +199,7 @@ void *thread_func(void *args)
>> static int sched_policy; /* scheduling policy as set by user/default */
>> struct sched_param ssp; /* set schedule priority. */
>> struct sched_param gsp; /* gsp schedule priority. */
>> - struct timeb tptr; /* tptr.millitm will be used to seed srand. */
>> + struct timeval tptr; /* tv_usec will be used to seed srand. */
>> thread_sched_t *locargptr = /* local ptr to the arguments. */
>> (thread_sched_t *) args;
>>
>> @@ -215,8 +215,8 @@ void *thread_func(void *args)
>> ssp.sched_priority = 0;
>> else {
>> /* Set a random value between max_priority and min_priority */
>> - ftime(&tptr);
>> - srand((tptr.millitm) % 1000);
>> + gettimeofday(&tptr, NULL);
>> + srand((tptr.tv_usec) % 1000000);
> Hi Yang,
>
> One minor issue:
> '% 1000000' seems useless because the range of tv_usec is 0~999999.
Agree. Will fix it in v2 patch.

Best Regards
Yang Xu
>
> Other than that, it's good to me.
>
> Best Regards,
> Xiao Yang
>> set_priority = (min_priority + (int)((float)max_priority
>> * rand() / (RAND_MAX +
>> 1.0)));
>
> .
>

[LTP] [PATCH v2] Remove ftime and sys/timeb.h

[Yang Xu]

The two case uses ftime to get the current milliseconds and it was
used to generate random value. Using gettimeofday() to get the microseconds
can reach the same aim.

This also fixes the travis build error[1] on Fedora:Rawhide because the <sys/timeb.h>
has been deprecated[2].

[1] https://travis-ci.org/github/linux-test-project/ltp/jobs/737698948
[2] https://www.spinics.net/lists/fedora-devel/msg279545.html

Reviewed-by: Li Wang <liwang@redhat.com>
Acked-by: Jan Stancek <jstancek@redhat.com>
Acked-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
Signed-off-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>
---
v1->v2
1.Remove useless "% 1000000" since the range of tv_usec is 0~999999.
2.case suseconds_t to unsigned int 
 testcases/kernel/mem/hugetlb/lib/hugetlb.c | 12 ++++++------
 testcases/kernel/sched/tool/trace_sched.c  |  8 ++++----
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/testcases/kernel/mem/hugetlb/lib/hugetlb.c b/testcases/kernel/mem/hugetlb/lib/hugetlb.c
index 4bb2d500e..1204f21d2 100644
--- a/testcases/kernel/mem/hugetlb/lib/hugetlb.c
+++ b/testcases/kernel/mem/hugetlb/lib/hugetlb.c
@@ -35,7 +35,7 @@
 #include <sys/types.h>
 #include <sys/ipc.h>
 #include <sys/shm.h>
-#include <sys/timeb.h>
+#include <sys/time.h>
 #include <pwd.h>
 #include "hugetlb.h"
 
@@ -52,7 +52,7 @@ int getipckey(void)
 	char *curdir = NULL;
 	size_t size = 0;
 	key_t ipc_key;
-	struct timeb time_info;
+	struct timeval time_info;
 
 	curdir = getcwd(curdir, size);
 	if (curdir == NULL)
@@ -67,11 +67,11 @@ int getipckey(void)
 	 * project identifier is a "random character" produced by
 	 * generating a random number between 0 and 25 and then adding
 	 * that to the ascii value of 'a'.  The "seed" for the random
-	 * number is the millisecond value that is set in the timeb
-	 * structure after calling ftime().
+	 * number is the microseconds value that is set in the timeval
+	 * structure after calling gettimeofday().
 	 */
-	ftime(&time_info);
-	srandom((unsigned int)time_info.millitm);
+	gettimeofday(&time_info, NULL);
+	srandom((unsigned int)time_info.tv_usec);
 
 	ipc_key = ftok(curdir, ascii_a + random() % 26);
 	if (ipc_key == -1)
diff --git a/testcases/kernel/sched/tool/trace_sched.c b/testcases/kernel/sched/tool/trace_sched.c
index 71caf239a..85c4b23ad 100644
--- a/testcases/kernel/sched/tool/trace_sched.c
+++ b/testcases/kernel/sched/tool/trace_sched.c
@@ -52,7 +52,7 @@
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <sys/wait.h>
-#include <sys/timeb.h>
+#include <sys/time.h>
 #include <unistd.h>
 #include <string.h>
 
@@ -199,7 +199,7 @@ void *thread_func(void *args)
 	static int sched_policy;	/* scheduling policy as set by user/default   */
 	struct sched_param ssp;	/* set schedule priority.                     */
 	struct sched_param gsp;	/* gsp schedule priority.                     */
-	struct timeb tptr;	/* tptr.millitm will be used to seed srand.   */
+	struct timeval tptr;	/* tptr.tv_usec will be used to seed srand.   */
 	thread_sched_t *locargptr =	/* local ptr to the arguments.                */
 	    (thread_sched_t *) args;
 
@@ -215,8 +215,8 @@ void *thread_func(void *args)
 		ssp.sched_priority = 0;
 	else {
 		/* Set a random value between max_priority and min_priority */
-		ftime(&tptr);
-		srand((tptr.millitm) % 1000);
+		gettimeofday(&tptr, NULL);
+		srand((unsigned int)tptr.tv_usec);
 		set_priority = (min_priority + (int)((float)max_priority
 						     * rand() / (RAND_MAX +
 								 1.0)));
-- 
2.23.0

[LTP] [PATCH v2] Remove ftime and sys/timeb.h

[Li Wang]

Yang Xu <xuyang2018.jy@cn.fujitsu.com> wrote:

> The two case uses ftime to get the current milliseconds and it was
> used to generate random value. Using gettimeofday() to get the microseconds
> can reach the same aim.
>
> This also fixes the travis build error[1] on Fedora:Rawhide because the <sys/timeb.h>
> has been deprecated[2].
>
> [1] https://travis-ci.org/github/linux-test-project/ltp/jobs/737698948
> [2] https://www.spinics.net/lists/fedora-devel/msg279545.html
>
> Reviewed-by: Li Wang <liwang@redhat.com>
> Acked-by: Jan Stancek <jstancek@redhat.com>
> Acked-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
> Signed-off-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>

Pushed, thanks!

-- 
Regards,
Li Wang

[LTP] [PATCH v2] Remove ftime and sys/timeb.h

[Petr Vorel]

Hi all,

> Yang Xu <xuyang2018.jy@cn.fujitsu.com> wrote:

> > The two case uses ftime to get the current milliseconds and it was
> > used to generate random value. Using gettimeofday() to get the microseconds
> > can reach the same aim.

> > This also fixes the travis build error[1] on Fedora:Rawhide because the <sys/timeb.h>
> > has been deprecated[2].

> > [1] https://travis-ci.org/github/linux-test-project/ltp/jobs/737698948
> > [2] https://www.spinics.net/lists/fedora-devel/msg279545.html

> > Reviewed-by: Li Wang <liwang@redhat.com>
> > Acked-by: Jan Stancek <jstancek@redhat.com>
> > Acked-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
> > Signed-off-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>

> Pushed, thanks!
Thanks for handling these!

Kind regards,
Petr