Commit c0c56955 authored by Ron Mercer's avatar Ron Mercer Committed by David S. Miller

qlge: Fix dropping of large non-TCP/UDP frames.

This was found with pings that were large enough to span multiple rx
buffers.  The split frame logic handles this case nicely.
Signed-off-by: default avatarRon Mercer <ron.mercer@qlogic.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 8ea7a37c
...@@ -2066,21 +2066,11 @@ static unsigned long ql_process_mac_rx_intr(struct ql_adapter *qdev, ...@@ -2066,21 +2066,11 @@ static unsigned long ql_process_mac_rx_intr(struct ql_adapter *qdev,
ql_process_mac_rx_page(qdev, rx_ring, ib_mac_rsp, ql_process_mac_rx_page(qdev, rx_ring, ib_mac_rsp,
length, vlan_id); length, vlan_id);
} else { } else {
struct bq_desc *lbq_desc; /* Non-TCP/UDP large frames that span multiple buffers
* can be processed corrrectly by the split frame logic.
/* Free small buffer that holds the IAL */ */
lbq_desc = ql_get_curr_sbuf(rx_ring); ql_process_mac_split_rx_intr(qdev, rx_ring, ib_mac_rsp,
netif_err(qdev, rx_err, qdev->ndev, vlan_id);
"Dropping frame, len %d > mtu %d\n",
length, qdev->ndev->mtu);
/* Unwind the large buffers for this frame. */
while (length > 0) {
lbq_desc = ql_get_curr_lchunk(qdev, rx_ring);
length -= (length < rx_ring->lbq_buf_size) ?
length : rx_ring->lbq_buf_size;
put_page(lbq_desc->p.pg_chunk.page);
}
} }
return (unsigned long)length; return (unsigned long)length;
......
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