Skip to content

Commit

Permalink
Merge branch 'net-gro-gro_drop-deprecation'
Browse files Browse the repository at this point in the history
Eric Dumazet says:

====================
net-gro: GRO_DROP deprecation

GRO_DROP has no practical use and can be removed,
once ice driver is cleaned up.

This removes one useless conditional test in napi_gro_frags().
====================

Link: https://lore.kernel.org/r/20210108113903.3779510-1-eric.dumazet@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
  • Loading branch information
kuba-moo committed Jan 9, 2021
2 parents efb5b33 + 1d11fa6 commit 38f7b44
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 18 deletions.
7 changes: 1 addition & 6 deletions drivers/net/ethernet/intel/ice/ice_txrx_lib.c
Expand Up @@ -191,12 +191,7 @@ ice_receive_skb(struct ice_ring *rx_ring, struct sk_buff *skb, u16 vlan_tag)
if ((rx_ring->netdev->features & NETIF_F_HW_VLAN_CTAG_RX) &&
(vlan_tag & VLAN_VID_MASK))
__vlan_hwaccel_put_tag(skb, htons(ETH_P_8021Q), vlan_tag);
if (napi_gro_receive(&rx_ring->q_vector->napi, skb) == GRO_DROP) {
/* this is tracked separately to help us debug stack drops */
rx_ring->rx_stats.gro_dropped++;
netdev_dbg(rx_ring->netdev, "Receive Queue %d: Dropped packet from GRO\n",
rx_ring->q_index);
}
napi_gro_receive(&rx_ring->q_vector->napi, skb);
}

/**
Expand Down
1 change: 0 additions & 1 deletion include/linux/netdevice.h
Expand Up @@ -376,7 +376,6 @@ enum gro_result {
GRO_MERGED_FREE,
GRO_HELD,
GRO_NORMAL,
GRO_DROP,
GRO_CONSUMED,
};
typedef enum gro_result gro_result_t;
Expand Down
11 changes: 0 additions & 11 deletions net/core/dev.c
Expand Up @@ -6070,10 +6070,6 @@ static gro_result_t napi_skb_finish(struct napi_struct *napi,
gro_normal_one(napi, skb);
break;

case GRO_DROP:
kfree_skb(skb);
break;

case GRO_MERGED_FREE:
if (NAPI_GRO_CB(skb)->free == NAPI_GRO_FREE_STOLEN_HEAD)
napi_skb_free_stolen_head(skb);
Expand Down Expand Up @@ -6158,10 +6154,6 @@ static gro_result_t napi_frags_finish(struct napi_struct *napi,
gro_normal_one(napi, skb);
break;

case GRO_DROP:
napi_reuse_skb(napi, skb);
break;

case GRO_MERGED_FREE:
if (NAPI_GRO_CB(skb)->free == NAPI_GRO_FREE_STOLEN_HEAD)
napi_skb_free_stolen_head(skb);
Expand Down Expand Up @@ -6223,9 +6215,6 @@ gro_result_t napi_gro_frags(struct napi_struct *napi)
gro_result_t ret;
struct sk_buff *skb = napi_frags_skb(napi);

if (!skb)
return GRO_DROP;

trace_napi_gro_frags_entry(skb);

ret = napi_frags_finish(napi, skb, dev_gro_receive(napi, skb));
Expand Down

0 comments on commit 38f7b44

Please sign in to comment.