Skip to content

Commit 67ff326

Browse files
Wei LiuNanlinXie
authored andcommitted
hv: retain the timer irq
Timer IRQ was static mapped IRQ. It should not be cleaned up when one of the cpu dead. The patch will retain the timer irq. Tracked-On: #1359 Signed-off-by: Wei Liu <weix.w.liu@intel.com> Signed-off-by: Jason Chen CJ <jason.cj.chen@intel.com> Acked-by: Eddie Dong <eddie.dong@intel.com>
1 parent 07e7121 commit 67ff326

File tree

3 files changed

+0
-11
lines changed

3 files changed

+0
-11
lines changed

hypervisor/arch/x86/cpu.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -701,7 +701,6 @@ void cpu_dead(uint16_t pcpu_id)
701701
}
702702

703703
/* clean up native stuff */
704-
timer_cleanup();
705704
vmx_off(pcpu_id);
706705
cache_flush_invalidate_all();
707706

hypervisor/arch/x86/timer.c

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -188,15 +188,6 @@ void timer_init(void)
188188
init_tsc_deadline_timer();
189189
}
190190

191-
void timer_cleanup(void)
192-
{
193-
uint16_t pcpu_id = get_cpu_id();
194-
195-
if (pcpu_id == BOOT_CPU_ID) {
196-
free_irq(TIMER_IRQ);
197-
}
198-
}
199-
200191
void check_tsc(void)
201192
{
202193
uint64_t temp64;

hypervisor/include/arch/x86/timer.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@ int add_timer(struct hv_timer *timer);
6060
void del_timer(struct hv_timer *timer);
6161

6262
void timer_init(void);
63-
void timer_cleanup(void);
6463
void check_tsc(void);
6564
void calibrate_tsc(void);
6665

0 commit comments

Comments
 (0)