diff options
author | Jan Beulich <jbeulich@suse.com> | 2012-09-28 09:22:14 +0200 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2012-09-28 09:22:14 +0200 |
commit | e0ffcf808f850ec175895cb7509c7ce06dedbda0 (patch) | |
tree | a3986f6db8fa3223e3bc816026f7afc0cb70261a | |
parent | 4394401748e2892710d641ec3e73a63aecf6d6d9 (diff) | |
download | xen-e0ffcf808f850ec175895cb7509c7ce06dedbda0.tar.gz xen-e0ffcf808f850ec175895cb7509c7ce06dedbda0.tar.bz2 xen-e0ffcf808f850ec175895cb7509c7ce06dedbda0.zip |
x86/HPET: don't disable interrupt delivery right after setting it up
We shouldn't clear HPET_TN_FSB right after we (indirectly, via
request_irq()) enabled it for the channels we intend to use for
broadcasts.
This fixes a regression introduced by c/s 25103:0b0e42dc4f0a.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Keir Fraser <keir@xen.org>
-rw-r--r-- | xen/arch/x86/hpet.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xen/arch/x86/hpet.c b/xen/arch/x86/hpet.c index 45f7abae7d..4626c29077 100644 --- a/xen/arch/x86/hpet.c +++ b/xen/arch/x86/hpet.c @@ -533,7 +533,7 @@ void __init hpet_broadcast_init(void) { /* set HPET Tn as oneshot */ cfg = hpet_read32(HPET_Tn_CFG(hpet_events[i].idx)); - cfg &= ~(HPET_TN_LEVEL | HPET_TN_PERIODIC | HPET_TN_FSB); + cfg &= ~(HPET_TN_LEVEL | HPET_TN_PERIODIC); cfg |= HPET_TN_ENABLE | HPET_TN_32BIT; hpet_write32(cfg, HPET_Tn_CFG(hpet_events[i].idx)); @@ -590,7 +590,7 @@ void hpet_broadcast_resume(void) /* set HPET Tn as oneshot */ cfg = hpet_read32(HPET_Tn_CFG(hpet_events[i].idx)); - cfg &= ~(HPET_TN_LEVEL | HPET_TN_PERIODIC | HPET_TN_FSB); + cfg &= ~(HPET_TN_LEVEL | HPET_TN_PERIODIC); cfg |= HPET_TN_ENABLE | HPET_TN_32BIT; hpet_write32(cfg, HPET_Tn_CFG(hpet_events[i].idx)); |