From: Jan Beulich <jbeulich@suse.com>
Date: Mon, 22 Jul 2019 09:46:31 +0000 (+0200)
Subject: x86/IRQ: drop redundant cpumask_empty() from move_masked_irq()
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=a2070da838048bd865c4ef459d702ba5e6daa5bb;p=people%2Fdariof%2Fxen.git

x86/IRQ: drop redundant cpumask_empty() from move_masked_irq()

The subsequent cpumask_intersects() covers the "empty" case quite fine.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Roger Pau Monné <roger.pau@citrix.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
---

diff --git a/xen/arch/x86/irq.c b/xen/arch/x86/irq.c
index abd773f35d..5e062497ee 100644
--- a/xen/arch/x86/irq.c
+++ b/xen/arch/x86/irq.c
@@ -658,9 +658,6 @@ void move_masked_irq(struct irq_desc *desc)
     
     desc->status &= ~IRQ_MOVE_PENDING;
 
-    if (unlikely(cpumask_empty(pending_mask)))
-        return;
-
     if (!desc->handler->set_affinity)
         return;