Commit 02ce232c authored by Kirill Batuzov's avatar Kirill Batuzov Committed by Stefan Hajnoczi

vl.c: remove init_clocks call from main

Clocks are initialized in qemu_init_main_loop. They are not needed before it.
Initializing them twice is not only unnecessary but is harmful: it results in
memory leak and potentially can lead to a situation where different parts of
QEMU use different sets of timers.

To avoid it remove init_clocks call from main and add an assertion to
qemu_clock_init that corresponding clock has not been initialized yet.
Signed-off-by: default avatarKirill Batuzov <>
Signed-off-by: default avatarStefan Hajnoczi <>
parent b1e6fc08
......@@ -126,6 +126,9 @@ static void qemu_clock_init(QEMUClockType type)
QEMUClock *clock = qemu_clock_ptr(type);
/* Assert that the clock of type TYPE has not been initialized yet. */
assert([type] == NULL);
clock->type = type;
clock->enabled = true;
clock->last = INT64_MIN;
......@@ -3024,7 +3024,6 @@ int main(int argc, char **argv, char **envp)
rtc_clock = QEMU_CLOCK_HOST;
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment