net: remove __napi_complete()
authorEric Dumazet <edumazet@google.com>
Sat, 4 Feb 2017 23:25:02 +0000 (15:25 -0800)
committerDavid S. Miller <davem@davemloft.net>
Sun, 5 Feb 2017 21:11:57 +0000 (16:11 -0500)
All __napi_complete() callers have been converted to
use the more standard napi_complete_done(),
we can now remove this NAPI method for good.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/netdevice.h
net/core/dev.c

index 6f18b509fb2f473029561d85646713c149df49ac..014fbe256d55f843b72595e881d646dd5762ac33 100644 (file)
@@ -463,7 +463,6 @@ static inline bool napi_reschedule(struct napi_struct *napi)
        return false;
 }
 
-bool __napi_complete(struct napi_struct *n);
 bool napi_complete_done(struct napi_struct *n, int work_done);
 /**
  *     napi_complete - NAPI processing complete
index 42ba0379575a28525052a36620bfb30d38afc076..404d2e6d5d3233ad95edee18581276841573644c 100644 (file)
@@ -4883,23 +4883,6 @@ void __napi_schedule_irqoff(struct napi_struct *n)
 }
 EXPORT_SYMBOL(__napi_schedule_irqoff);
 
-bool __napi_complete(struct napi_struct *n)
-{
-       BUG_ON(!test_bit(NAPI_STATE_SCHED, &n->state));
-
-       /* Some drivers call us directly, instead of calling
-        * napi_complete_done().
-        */
-       if (unlikely(test_bit(NAPI_STATE_IN_BUSY_POLL, &n->state)))
-               return false;
-
-       list_del_init(&n->poll_list);
-       smp_mb__before_atomic();
-       clear_bit(NAPI_STATE_SCHED, &n->state);
-       return true;
-}
-EXPORT_SYMBOL(__napi_complete);
-
 bool napi_complete_done(struct napi_struct *n, int work_done)
 {
        unsigned long flags;
@@ -4926,14 +4909,13 @@ bool napi_complete_done(struct napi_struct *n, int work_done)
                else
                        napi_gro_flush(n, false);
        }
-       if (likely(list_empty(&n->poll_list))) {
-               WARN_ON_ONCE(!test_and_clear_bit(NAPI_STATE_SCHED, &n->state));
-       } else {
+       if (unlikely(!list_empty(&n->poll_list))) {
                /* If n->poll_list is not empty, we need to mask irqs */
                local_irq_save(flags);
-               __napi_complete(n);
+               list_del_init(&n->poll_list);
                local_irq_restore(flags);
        }
+       WARN_ON_ONCE(!test_and_clear_bit(NAPI_STATE_SCHED, &n->state));
        return true;
 }
 EXPORT_SYMBOL(napi_complete_done);