Hi
On Mon, Mar 7, 2022 at 11:46 AM Thomas Huth <thuth@redhat.com
<mailto:thuth@redhat.com>> wrote:
On 07/03/2022 08.03, marcandre.lureau@redhat.com
<mailto:marcandre.lureau@redhat.com> wrote:
> From: Marc-André Lureau <marcandre.lureau@redhat.com
<mailto:marcandre.lureau@redhat.com>>
>
> glib provides a convenience helper to measure elapsed time. It isn't
> subject to wall-clock time changes.
>
> Note that this changes the initial OPENED time, which used to print the
> current time.
[...]
> @@ -846,21 +828,20 @@ static void qtest_event(void *opaque,
QEMUChrEvent event)
> for (i = 0; i < ARRAY_SIZE(irq_levels); i++) {
> irq_levels[i] = 0;
> }
> - qemu_gettimeofday(&start_time);
> +
> + g_clear_pointer(&timer, g_timer_destroy);
> + timer = g_timer_new();
> qtest_opened = true;
> if (qtest_log_fp) {
> - fprintf(qtest_log_fp, "[I " FMT_timeval "] OPENED\n",
> - (long) start_time.tv_sec, (long)
start_time.tv_usec);
> + fprintf(qtest_log_fp, "[I " FMT_timeval "] OPENED\n",
g_timer_elapsed(timer, NULL));
> }
> break;
The new timestamp here is quite unuseful now, of course ... could you
replace it with g_get_current_time() instead?
Eventually, but I wonder why this (and only this) particular timestamp
should be the current time.