xenbaked: properly use time_t in dump_stats()

"int" is not a suitable type to convert time()'s return value to. Avoid
casts and other extra fiddling by using difftime(), on the assumption
that the overhead of using "double" doesn't matter here.

Coverity ID: 1509374
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Anthony PERARD <anthony.perard@citrix.com>
This commit is contained in:
Jan Beulich committed 2022-08-22 12:10:00 +02:00
1 parent f4036c96de
commit 722a61dbb5
1 file changed
+5 -11
+5 -11
View File
@@ -230,11 +230,7 @@ static void check_gotten_sum(void)
static void dump_stats(void)
{
stat_map_t *smt = stat_map;
time_t end_time, run_time;
time(&end_time);
run_time = end_time - start_time;
double run_time = difftime(time(NULL), start_time);
printf("Event counts:\n");
while (smt->text != NULL) {
@@ -242,13 +238,11 @@ static void dump_stats(void)
smt++;
}
printf("processed %d total records in %d seconds (%ld per second)\n",
rec_count, (int)run_time,
run_time ? (long)(rec_count/run_time) : 0L);
printf("processed %d total records in %.0f seconds (%.0f per second)\n",
rec_count, run_time, run_time ? rec_count / run_time : 0);
printf("woke up %d times in %d seconds (%ld per second)\n",
wakeups, (int) run_time,
run_time ? (long)(wakeups/run_time) : 0L);
printf("woke up %d times in %.0f seconds (%.0f per second)\n",
wakeups, run_time, run_time ? wakeups / run_time : 0);
check_gotten_sum();
}
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.