mirror of
https://github.com/mii443/qemu.git
synced 2025-09-01 14:49:23 +00:00
Correct alarm deadline computation
When the QEMU_CLOCK_HOST clock was added, computation of its
deadline was added to qemu_next_deadline, which is correct but
incomplete.
I noticed this by reading the very convoluted rules whereby
qemu_next_deadline_dyntick is computed, which miss QEMU_CLOCK_HOST
when use_icount is true. This patch inlines qemu_next_deadline
into qemu_next_deadline_dyntick, and then corrects the logic to skip
only QEMU_CLOCK_VIRTUAL when use_icount is true.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Cc: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
(cherry picked from commit 6ad0a1ed21
)
This commit is contained in:
committed by
Anthony Liguori
parent
d7f88b4bbd
commit
bbd9827cc7
15
qemu-timer.c
15
qemu-timer.c
@ -724,11 +724,18 @@ static uint64_t qemu_next_deadline_dyntick(void)
|
||||
int64_t delta;
|
||||
int64_t rtdelta;
|
||||
|
||||
if (use_icount)
|
||||
if (!use_icount && active_timers[QEMU_CLOCK_VIRTUAL]) {
|
||||
delta = active_timers[QEMU_CLOCK_VIRTUAL]->expire_time -
|
||||
qemu_get_clock(vm_clock);
|
||||
} else {
|
||||
delta = INT32_MAX;
|
||||
else
|
||||
delta = qemu_next_deadline();
|
||||
|
||||
}
|
||||
if (active_timers[QEMU_CLOCK_HOST]) {
|
||||
int64_t hdelta = active_timers[QEMU_CLOCK_HOST]->expire_time -
|
||||
qemu_get_clock_ns(host_clock);
|
||||
if (hdelta < delta)
|
||||
delta = hdelta;
|
||||
}
|
||||
if (active_timers[QEMU_CLOCK_REALTIME]) {
|
||||
rtdelta = (active_timers[QEMU_CLOCK_REALTIME]->expire_time * 1000000 -
|
||||
qemu_get_clock_ns(rt_clock));
|
||||
|
Reference in New Issue
Block a user