[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [PATCH v4 10/13] x86/IRQ: drop redundant cpumask_empty() from move_masked_irq()


  • To: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Jan Beulich <JBeulich@xxxxxxxx>
  • Date: Tue, 16 Jul 2019 07:43:28 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1;spf=pass smtp.mailfrom=suse.com;dmarc=pass action=none header.from=suse.com;dkim=pass header.d=suse.com;arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ybRJuXWKhFezR7nBnLlUiUoVPERKAmS8rHtqDLOKRK4=; b=dKxh2fIcPs2ZceswfS5HM+sS8MOU0Pinb1dqal0iyXsANyG7rP1xGhl9R1pVP/jjb5YMhO5SEKfI9owAb3LysIJ5qufdUOkXE7+jNsBR6X0dQHgQsQ7SEod7ORYfnHVZ5M6aPiAll71+FiMiyhzHrt947Xg5JCkP9MMKVMpL4tlcuhoZwAkSB1DSd4wecDeXCjhpqBkZNM6KAsyKgbRx8VSo8I7rlr1oH6hNCodk/KJHmzki8/O3KMcTWTpDyTeCeMkM6gimrQhhRT4nLimHuBvCmnZCRTyhuqwA8tfGcQlG9fB5bQrcVVVdexu5KjkreeTT1N7rszLGFcliHdKnYw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=mF/aWkpL/XUTdRoCNwIpIQ4yaoOfhlV5UQ0Y8g7/a7ya+QuRp+MVKhEnOoJVmSNdAt/bXuwq+DY7YFk+DIjD/1qeQpqDBGQgSRv2yuBH96FLlXEimxHlZ5Dw11E2+j/TdKPEEHfarPcrP1v2L8t0tej++q8HYj8XQhPoC8rymkWRtrECaQSAJ9nzQBnw6g/Bjucy8O7qc4VmHGxsfjjYNLBw75C163JSb5EpuJm42D0ZlW2JUMSecBgx1gRhbJX3lzU57irzLveqeAamCSL+49sP3CwFM80Q4eIogcV+AgUoK0P/JMPyS1D6KblINuRFQIROh6J1m2TC4nb3qTNPhA==
  • Authentication-results: spf=none (sender IP is ) smtp.mailfrom=JBeulich@xxxxxxxx;
  • Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Delivery-date: Tue, 16 Jul 2019 07:45:01 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHVO6oom1TvgiYc9Uinigs089keTA==
  • Thread-topic: [PATCH v4 10/13] 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@xxxxxxxx>
Reviewed-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

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

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.