Using dev_kfree_skb_any for functions called in multiple contexts
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest10 patches 10 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/10 | 8139cp: Call dev_kfree_skby_any instead of kfree_skb.drivers/ | +1 | 0 replies | |
| 1/10 | netpoll: move setting of NETPOLL_RX_DROP into netpoll_poll_devnet/ | +8 | 0 replies | |
| 2/10 | 8139too: Call dev_kfree_skby_any instead of dev_kfree_skb.drivers/ | +2 | 6 replies | |
| 2/10 | netpoll: Pass budget into poll_napinet/ | +3 | 0 replies | |
| 3/10 | r8169: Call dev_kfree_skby_any instead of dev_kfree_skb.drivers/ | +3 | 1 reply | |
| 3/10 | netpoll: Visit all napi handlers in poll_napinet/ | +0 | 0 replies | |
| 4/10 | bonding: Call dev_kfree_skby_any instead of kfree_skb.drivers/ drivers/ drivers/ | +7 | 0 replies | |
| 4/10 | netpoll: Warn if more packets are processed than are budgetednet/ | +1 | 0 replies | |
| 5/10 | bnx2: Call dev_kfree_skby_any instead of dev_kfree_skb.drivers/ | +3 | 0 replies | |
| 5/10 | netpoll: Add netpoll_rx_processinginclude/ net/ | +16 | 0 replies | |
| 6/10 | tg3: Call dev_kfree_skby_any instead of dev_kfree_skb.drivers/ | +7 | 0 replies | |
| 6/10 | netpoll: Don't drop all received packets.include/ net/ | +7 | 0 replies | |
| 7/10 | ixgb: Call dev_kfree_skby_any instead of dev_kfree_skb.drivers/ | +3 | 0 replies | |
| 7/10 | netpoll: Move netpoll_trap under CONFIG_NETPOLL_TRAPinclude/ net/ | +18 | 0 replies | |
| 8/10 | mlx4: Call dev_kfree_skby_any instead of dev_kfree_skb.drivers/ | +1 | 0 replies | |
| 8/10 | netpoll: Consolidate neigh_tx processing in service_neigh_queuenet/ | +16 | 0 replies | |
| 9/10 | benet: Call dev_kfree_skby_any instead of kfree_skb.drivers/ | +1 | 0 replies | |
| 9/10 | netpoll: Move all receive processing under CONFIG_NETPOLL_TRAPinclude/ net/ | +104 | 0 replies | |
| 10/10 | gianfar: Carefully free skbs in functions called by netpoll.drivers/ | +2 | 0 replies | |
| 10/10 | netpoll: Remove dead packet receive code (CONFIG_NETPOLL_TRAP)drivers/ include/ include/ +2 more | +2 | 0 replies |
Reviews trailers per patch
2 of 20 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.
| Patch | Eric Dumazet |
|---|---|
| Series | A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b A-b |
| 1/10 | |
| 1/10 | |
| 2/10 | |
| 2/10 | |
| 3/10 | |
| 3/10 | |
| 4/10 | |
| 4/10 | |
| 5/10 | |
| 5/10 | |
| 6/10 | |
| 6/10 | |
| 7/10 | |
| 7/10 | |
| 8/10 | |
| 8/10 | |
| 9/10 | |
| 9/10 | |
| 10/10 | A-b |
| 10/10 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH next-next 0/10] Using dev_kfree_skb_any for functions called in multiple contexts
This patchset should be an uncontroversial set of changes to change dev_kfree_skb to dev_kfree_skb_any for code paths that are called in hard irq contexts in addition to other contexts. netpoll is the reason this code gets called in multiple contexts.
There is more coming but these changes are a good starting place, and stand on their own.
Since the last round changes to the rx path have been removed netpoll will changed to avoid that.
Eric W. Biederman (10):
8139cp: Call dev_kfree_skby_any instead of kfree_skb. 8139too: Call dev_kfree_skby_any instead of dev_kfree_skb.
r8169: Call dev_kfree_skby_any instead of dev_kfree_skb.
bonding: Call dev_kfree_skby_any instead of kfree_skb.
bnx2: Call dev_kfree_skby_any instead of dev_kfree_skb.
tg3: Call dev_kfree_skby_any instead of dev_kfree_skb.
ixgb: Call dev_kfree_skby_any instead of dev_kfree_skb.
mlx4: Call dev_kfree_skby_any instead of dev_kfree_skb.
benet: Call dev_kfree_skby_any instead of kfree_skb.
gianfar: Carefully free skbs in functions called by netpoll.
drivers/net/bonding/bond_3ad.c | 2 +- drivers/net/bonding/bond_alb.c | 2 +- drivers/net/bonding/bond_main.c | 10 +++++----- drivers/net/ethernet/broadcom/bnx2.c | 6 +++--- drivers/net/ethernet/broadcom/tg3.c | 14 +++++++------- drivers/net/ethernet/emulex/benet/be_main.c | 2 +- drivers/net/ethernet/freescale/gianfar.c | 4 ++-- drivers/net/ethernet/intel/ixgb/ixgb_main.c | 6 +++--- drivers/net/ethernet/mellanox/mlx4/en_tx.c | 2 +- drivers/net/ethernet/realtek/8139cp.c | 2 +- drivers/net/ethernet/realtek/8139too.c | 4 ++-- drivers/net/ethernet/realtek/r8169.c | 6 +++--- 12 files changed, 30 insertions(+), 30 deletions(-)
Eric