summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorRon Mercer <ron.mercer@qlogic.com>2010-02-17 06:41:21 +0000
committerDavid S. Miller <davem@davemloft.net>2010-02-17 13:35:37 -0800
commitc0c56955e7325ba2f910c760111457adc21ef0d4 (patch)
tree30aeef6b13f83e030508a0dd1564e49fe3e7ed82 /drivers
parent8ea7a37c5a312bfee51ff7f12f78efe4fbc901cc (diff)
downloadop-kernel-dev-c0c56955e7325ba2f910c760111457adc21ef0d4.zip
op-kernel-dev-c0c56955e7325ba2f910c760111457adc21ef0d4.tar.gz
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: Ron Mercer <ron.mercer@qlogic.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/net/qlge/qlge_main.c20
1 files changed, 5 insertions, 15 deletions
diff --git a/drivers/net/qlge/qlge_main.c b/drivers/net/qlge/qlge_main.c
index 2c052ca..0541e77 100644
--- a/drivers/net/qlge/qlge_main.c
+++ b/drivers/net/qlge/qlge_main.c
@@ -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,
length, vlan_id);
} else {
- struct bq_desc *lbq_desc;
-
- /* Free small buffer that holds the IAL */
- lbq_desc = ql_get_curr_sbuf(rx_ring);
- netif_err(qdev, rx_err, qdev->ndev,
- "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);
- }
+ /* Non-TCP/UDP large frames that span multiple buffers
+ * can be processed corrrectly by the split frame logic.
+ */
+ ql_process_mac_split_rx_intr(qdev, rx_ring, ib_mac_rsp,
+ vlan_id);
}
return (unsigned long)length;
OpenPOWER on IntegriCloud