From c22c073d39dee841a3d2f3ae9a1e957404c3af80 Mon Sep 17 00:00:00 2001 From: Jan Beulich Date: Thu, 6 Jun 2019 16:05:27 +0200 Subject: [PATCH] x86/IRQ: ACKTYPE_NONE cannot make it into irq_guest_eoi_timer_fn() MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit action->ack_type is set once before the timer even gets initialized, and is never changed later. The timer gets activated only for EOI and UNMASK types. Hence there's no need to have a respective if() in there. Replace it by an ASSERT(). Signed-off-by: Jan Beulich Reviewed-by: Roger Pau Monné Reviewed-by: Andrew Cooper --- xen/arch/x86/irq.c | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/xen/arch/x86/irq.c b/xen/arch/x86/irq.c index be486a6ce4..4042caaa00 100644 --- a/xen/arch/x86/irq.c +++ b/xen/arch/x86/irq.c @@ -1106,7 +1106,7 @@ static void set_eoi_ready(void *data); static void irq_guest_eoi_timer_fn(void *data) { struct irq_desc *desc = data; - unsigned int irq = desc - irq_desc; + unsigned int i, irq = desc - irq_desc; irq_guest_action_t *action; cpumask_t cpu_eoi_map; @@ -1117,6 +1117,8 @@ static void irq_guest_eoi_timer_fn(void *data) action = (irq_guest_action_t *)desc->action; + ASSERT(action->ack_type != ACKTYPE_NONE); + /* * Is no IRQ in flight at all, or another instance of this timer already * running? Skip everything to avoid forcing an EOI early. @@ -1124,16 +1126,13 @@ static void irq_guest_eoi_timer_fn(void *data) if ( !action->in_flight || timer_is_active(&action->eoi_timer) ) goto out; - if ( action->ack_type != ACKTYPE_NONE ) + for ( i = 0; i < action->nr_guests; i++ ) { - unsigned int i; - for ( i = 0; i < action->nr_guests; i++ ) - { - struct domain *d = action->guest[i]; - unsigned int pirq = domain_irq_to_pirq(d, irq); - if ( test_and_clear_bool(pirq_info(d, pirq)->masked) ) - action->in_flight--; - } + struct domain *d = action->guest[i]; + unsigned int pirq = domain_irq_to_pirq(d, irq); + + if ( test_and_clear_bool(pirq_info(d, pirq)->masked) ) + action->in_flight--; } if ( action->in_flight ) -- 2.39.5