Commit bee980d9 authored by Keir Fraser's avatar Keir Fraser Committed by Konrad Rzeszutek Wilk

xen/events: Handle VIRQ_TIMER before any other hardirq in event loop.

This avoids any other hardirq handler seeing a very stale jiffies
value immediately after wakeup from a long idle period. The one
observable symptom of this was a USB keyboard, with software keyboard
repeat, which would always repeat a key immediately that it was
pressed. This is due to the key press waking the guest, the key
handler immediately runs, sees an old jiffies value, and then that
jiffies value significantly updated, before the key is unpressed.
Reviewed-by: default avatarDavid Vrabel <david.vrabel@citrix.com>
Signed-off-by: default avatarKeir Fraser <keir.fraser@citrix.com>
Signed-off-by: default avatarKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
parent c26377e6
...@@ -1316,7 +1316,7 @@ static void __xen_evtchn_do_upcall(void) ...@@ -1316,7 +1316,7 @@ static void __xen_evtchn_do_upcall(void)
{ {
int start_word_idx, start_bit_idx; int start_word_idx, start_bit_idx;
int word_idx, bit_idx; int word_idx, bit_idx;
int i; int i, irq;
int cpu = get_cpu(); int cpu = get_cpu();
struct shared_info *s = HYPERVISOR_shared_info; struct shared_info *s = HYPERVISOR_shared_info;
struct vcpu_info *vcpu_info = __this_cpu_read(xen_vcpu); struct vcpu_info *vcpu_info = __this_cpu_read(xen_vcpu);
...@@ -1324,6 +1324,8 @@ static void __xen_evtchn_do_upcall(void) ...@@ -1324,6 +1324,8 @@ static void __xen_evtchn_do_upcall(void)
do { do {
xen_ulong_t pending_words; xen_ulong_t pending_words;
xen_ulong_t pending_bits;
struct irq_desc *desc;
vcpu_info->evtchn_upcall_pending = 0; vcpu_info->evtchn_upcall_pending = 0;
...@@ -1335,6 +1337,17 @@ static void __xen_evtchn_do_upcall(void) ...@@ -1335,6 +1337,17 @@ static void __xen_evtchn_do_upcall(void)
* selector flag. xchg_xen_ulong must contain an * selector flag. xchg_xen_ulong must contain an
* appropriate barrier. * appropriate barrier.
*/ */
if ((irq = per_cpu(virq_to_irq, cpu)[VIRQ_TIMER]) != -1) {
int evtchn = evtchn_from_irq(irq);
word_idx = evtchn / BITS_PER_LONG;
pending_bits = evtchn % BITS_PER_LONG;
if (active_evtchns(cpu, s, word_idx) & (1ULL << pending_bits)) {
desc = irq_to_desc(irq);
if (desc)
generic_handle_irq_desc(irq, desc);
}
}
pending_words = xchg_xen_ulong(&vcpu_info->evtchn_pending_sel, 0); pending_words = xchg_xen_ulong(&vcpu_info->evtchn_pending_sel, 0);
start_word_idx = __this_cpu_read(current_word_idx); start_word_idx = __this_cpu_read(current_word_idx);
...@@ -1343,7 +1356,6 @@ static void __xen_evtchn_do_upcall(void) ...@@ -1343,7 +1356,6 @@ static void __xen_evtchn_do_upcall(void)
word_idx = start_word_idx; word_idx = start_word_idx;
for (i = 0; pending_words != 0; i++) { for (i = 0; pending_words != 0; i++) {
xen_ulong_t pending_bits;
xen_ulong_t words; xen_ulong_t words;
words = MASK_LSBS(pending_words, word_idx); words = MASK_LSBS(pending_words, word_idx);
...@@ -1372,8 +1384,7 @@ static void __xen_evtchn_do_upcall(void) ...@@ -1372,8 +1384,7 @@ static void __xen_evtchn_do_upcall(void)
do { do {
xen_ulong_t bits; xen_ulong_t bits;
int port, irq; int port;
struct irq_desc *desc;
bits = MASK_LSBS(pending_bits, bit_idx); bits = MASK_LSBS(pending_bits, bit_idx);
......
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