Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/ebiederm/net...
authorDavid S. Miller <davem@davemloft.net>
Tue, 25 Mar 2014 22:54:36 +0000 (18:54 -0400)
committerDavid S. Miller <davem@davemloft.net>
Tue, 25 Mar 2014 22:54:36 +0000 (18:54 -0400)
commit8779772c93286fcfe8ab428dc83434c783225535
tree2e259136fe391a5068bec0ea7f1e8d75e0cf7800
parent35d499ee3a63b645f355d98dc407b22751896a43
parentfb7c03dfc48eb81bda368561f4ca162749d127c9
Merge branch 'master' of git://git./linux/kernel/git/ebiederm/net-next

Eric W. Biederman says:

====================
Using dev_kfree/consume_skb_any for functions called in multiple contexts

These changes are a result of walking through the network drivers
supporting netpoll and verifying the code paths that netpoll can cause
to be called in hard irq context use an appropriate flavor of
kfree_skb.  Either dev_kfree_skb_any or dev_consume_skb_any.

Since my last pass at this I have become aware of the small differences
between dev_kfree_skb_any and dev_consume_skb_any.
net/core/drop_monitor.c reports the dev_kfree_skb_any as a drop and
while being quite about the second.  With the weird twist that
dev_kfree_skb is unintuitively consume_skb.

As netpoll now calls the napi poll function with budget == 0, pieces of
a drivers the napi poll function that don't run when budget == 0 have
been ignored.

The most interesting change is to the atl1c which tried unsuccesfully to
tell one of it's functions which context it is called in so that it
could call dev_kfree_skb_irq or dev_kfree_skb as appropriate.  I have
just removed the extra parameter and called dev_consume_skb_any.

At 54 separate changes I will post each change as a separate patch (so
they can be reviewed) but for general sanity sake I have gathered them
all into a git branch for easy acces.
====================

Acked-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>