netpoll: Add netpoll_rx_processing
authorEric W. Biederman <ebiederm@xmission.com>
Sat, 15 Mar 2014 03:47:49 +0000 (20:47 -0700)
committerDavid S. Miller <davem@davemloft.net>
Mon, 17 Mar 2014 19:47:22 +0000 (15:47 -0400)
Add a helper netpoll_rx_processing that reports when netpoll has
receive side processing to perform.

Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/netpoll.h
net/core/netpoll.c

index fbfdb9d8d3a7f59b4788bdfd1cdb26c86f0a81ca..479d15c9777016ba133723be2c94b1cfc3edfb91 100644 (file)
@@ -82,14 +82,24 @@ static inline void netpoll_send_skb(struct netpoll *np, struct sk_buff *skb)
        local_irq_restore(flags);
 }
 
-
+#ifdef CONFIG_NETPOLL_TRAP
+static inline bool netpoll_rx_processing(struct netpoll_info *npinfo)
+{
+       return !list_empty(&npinfo->rx_np);
+}
+#else
+static inline bool netpoll_rx_processing(struct netpoll_info *npinfo)
+{
+       return false;
+}
+#endif
 
 #ifdef CONFIG_NETPOLL
 static inline bool netpoll_rx_on(struct sk_buff *skb)
 {
        struct netpoll_info *npinfo = rcu_dereference_bh(skb->dev->npinfo);
 
-       return npinfo && (!list_empty(&npinfo->rx_np) || npinfo->rx_flags);
+       return npinfo && (netpoll_rx_processing(npinfo) || npinfo->rx_flags);
 }
 
 static inline bool netpoll_rx(struct sk_buff *skb)
@@ -105,8 +115,8 @@ static inline bool netpoll_rx(struct sk_buff *skb)
 
        npinfo = rcu_dereference_bh(skb->dev->npinfo);
        spin_lock(&npinfo->rx_lock);
-       /* check rx_flags again with the lock held */
-       if (npinfo->rx_flags && __netpoll_rx(skb, npinfo))
+       /* check rx_processing again with the lock held */
+       if (netpoll_rx_processing(npinfo) && __netpoll_rx(skb, npinfo))
                ret = true;
        spin_unlock(&npinfo->rx_lock);
 
index 2ad330e02967132369df3794a5c79e5142f7ebd6..ef83a2530e98bf58573e7865b3aea9871ba62dae 100644 (file)
@@ -538,7 +538,7 @@ static void netpoll_neigh_reply(struct sk_buff *skb, struct netpoll_info *npinfo
        int hlen, tlen;
        int hits = 0, proto;
 
-       if (list_empty(&npinfo->rx_np))
+       if (!netpoll_rx_processing(npinfo))
                return;
 
        /* Before checking the packet, we do some early
@@ -770,7 +770,7 @@ int __netpoll_rx(struct sk_buff *skb, struct netpoll_info *npinfo)
        struct netpoll *np, *tmp;
        uint16_t source;
 
-       if (list_empty(&npinfo->rx_np))
+       if (!netpoll_rx_processing(npinfo))
                goto out;
 
        if (skb->dev->type != ARPHRD_ETHER)