Commit ee2fa8e6 authored by Manish Chopra's avatar Manish Chopra Committed by David S. Miller

qede: Fix single MTU sized packet from firmware GRO flow

In firmware assisted GRO flow there could be a single MTU sized
segment arriving due to firmware aggregation timeout/last segment
in an aggregation flow, which is not expected to be an actual gro
packet. So If a skb has zero frags from the GRO flow then simply
push it in the stack as non gso skb.
Signed-off-by: default avatarManish Chopra <manish.chopra@qlogic.com>
Signed-off-by: default avatarYuval Mintz <yuval.mintz@qlogic.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent aad94c04
...@@ -1069,6 +1069,17 @@ static void qede_gro_receive(struct qede_dev *edev, ...@@ -1069,6 +1069,17 @@ static void qede_gro_receive(struct qede_dev *edev,
struct sk_buff *skb, struct sk_buff *skb,
u16 vlan_tag) u16 vlan_tag)
{ {
/* FW can send a single MTU sized packet from gro flow
* due to aggregation timeout/last segment etc. which
* is not expected to be a gro packet. If a skb has zero
* frags then simply push it in the stack as non gso skb.
*/
if (unlikely(!skb->data_len)) {
skb_shinfo(skb)->gso_type = 0;
skb_shinfo(skb)->gso_size = 0;
goto send_skb;
}
#ifdef CONFIG_INET #ifdef CONFIG_INET
if (skb_shinfo(skb)->gso_size) { if (skb_shinfo(skb)->gso_size) {
skb_set_network_header(skb, 0); skb_set_network_header(skb, 0);
...@@ -1087,6 +1098,8 @@ static void qede_gro_receive(struct qede_dev *edev, ...@@ -1087,6 +1098,8 @@ static void qede_gro_receive(struct qede_dev *edev,
} }
} }
#endif #endif
send_skb:
skb_record_rx_queue(skb, fp->rss_id); skb_record_rx_queue(skb, fp->rss_id);
qede_skb_receive(edev, fp, skb, vlan_tag); qede_skb_receive(edev, fp, skb, vlan_tag);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment