Matt Fleming b05d5c
From: Mike Galbraith <mgalbraith@suse.de>
Matt Fleming b05d5c
Date: Wed Aug  2 10:55:41 CEST 2017
Matt Fleming b05d5c
Subject: net: net dont disable irqs in netpoll_send_skb()
Matt Fleming b05d5c
Patch-mainline: Never, RT specific
Matt Fleming b05d5c
References: SLE Realtime Extension
Matt Fleming b05d5c
Matt Fleming b05d5c
RT takes sleeping locks, we can't disable irqs.
Matt Fleming b05d5c
Matt Fleming b05d5c
Signed-off-by: Mike Galbraith <mgalbraith@suse.de>
Matt Fleming b05d5c
---
Matt Fleming b05d5c
 include/linux/netpoll.h |    8 ++++++--
Matt Fleming b05d5c
 net/core/netpoll.c      |    6 +++---
Matt Fleming b05d5c
 2 files changed, 9 insertions(+), 5 deletions(-)
Matt Fleming b05d5c
Matt Fleming b05d5c
--- a/include/linux/netpoll.h
Matt Fleming b05d5c
+++ b/include/linux/netpoll.h
Matt Fleming b05d5c
@@ -67,9 +67,13 @@ void netpoll_send_skb_on_dev(struct netp
Matt Fleming b05d5c
 static inline void netpoll_send_skb(struct netpoll *np, struct sk_buff *skb)
Matt Fleming b05d5c
 {
Matt Fleming b05d5c
 	unsigned long flags;
Matt Fleming b05d5c
-	local_irq_save(flags);
Matt Fleming b05d5c
+	local_irq_save_nort(flags);
Daniel Wagner fdf974
+	if (IS_ENABLED(CONFIG_PREEMPT_RT))
Matt Fleming b05d5c
+		rcu_read_lock();
Matt Fleming b05d5c
 	netpoll_send_skb_on_dev(np, skb, np->dev);
Matt Fleming b05d5c
-	local_irq_restore(flags);
Daniel Wagner fdf974
+	if (IS_ENABLED(CONFIG_PREEMPT_RT))
Matt Fleming b05d5c
+		rcu_read_unlock();
Matt Fleming b05d5c
+	local_irq_restore_nort(flags);
Matt Fleming b05d5c
 }
Matt Fleming b05d5c
 
Matt Fleming b05d5c
 #ifdef CONFIG_NETPOLL
Matt Fleming b05d5c
--- a/net/core/netpoll.c
Matt Fleming b05d5c
+++ b/net/core/netpoll.c
Matt Fleming b05d5c
@@ -334,7 +334,7 @@ void netpoll_send_skb_on_dev(struct netp
Matt Fleming b05d5c
 	/* It is up to the caller to keep npinfo alive. */
Matt Fleming b05d5c
 	struct netpoll_info *npinfo;
Matt Fleming b05d5c
 
Matt Fleming b05d5c
-	lockdep_assert_irqs_disabled();
Matt Fleming b05d5c
+	lockdep_assert_irqs_disabled_nonrt();
Matt Fleming b05d5c
 
Matt Fleming b05d5c
 	npinfo = rcu_dereference_bh(np->dev->npinfo);
Matt Fleming b05d5c
 	if (!npinfo || !netif_running(dev) || !netif_device_present(dev)) {
Matt Fleming b05d5c
@@ -368,7 +368,7 @@ void netpoll_send_skb_on_dev(struct netp
Matt Fleming b05d5c
 			udelay(USEC_PER_POLL);
Matt Fleming b05d5c
 		}
Matt Fleming b05d5c
 
Matt Fleming b05d5c
-		WARN_ONCE(!irqs_disabled(),
Daniel Wagner fdf974
+		WARN_ONCE(!IS_ENABLED(CONFIG_PREEMPT_RT) && !irqs_disabled(),
Matt Fleming b05d5c
 			"netpoll_send_skb_on_dev(): %s enabled interrupts in poll (%pF)\n",
Matt Fleming b05d5c
 			dev->name, dev->netdev_ops->ndo_start_xmit);
Matt Fleming b05d5c
 
Matt Fleming b05d5c
@@ -391,7 +391,7 @@ void netpoll_send_udp(struct netpoll *np
Matt Fleming b05d5c
 	static atomic_t ip_ident;
Matt Fleming b05d5c
 	struct ipv6hdr *ip6h;
Matt Fleming b05d5c
 
Matt Fleming b05d5c
-	WARN_ON_ONCE(!irqs_disabled());
Matt Fleming b05d5c
+	WARN_ON_ONCE_NONRT(!irqs_disabled());
Matt Fleming b05d5c
 
Matt Fleming b05d5c
 	udp_len = len + sizeof(*udph);
Matt Fleming b05d5c
 	if (np->ipv6)