On 24/03/20 07:32, Wanpeng Li wrote: > hrtimer_cancel(&apic->lapic_timer.timer); > + preempt_disable(); > + if (apic->lapic_timer.hv_timer_in_use) > + cancel_hv_timer(apic); > + preempt_enable(); > kvm_lapic_set_reg(apic, APIC_TMICT, 0); > apic->lapic_timer.period = 0; > apic->lapic_timer.tscdeadline = 0; There are a few other occurrences of hrtimer_cancel, and all of them probably have a similar issue. What about adding a cancel_apic_timer function that contains the combination of hrtimer_cancel/preempt_disable/cancel_hv_timer/preempt_enable? Thanks, Paolo
