diff options
author | David L Stevens <david.stevens@oracle.com> | 2015-01-13 12:45:05 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-01-13 17:24:21 -0500 |
commit | 86cfeab6b510a2fe94683bf71f9b96b686e2e0ce (patch) | |
tree | 47158b07586febd9b5ed7acfe9f7e420c3b07fde | |
parent | 3e7a8716e20b759eec0ad88145255bb33174f0c8 (diff) |
sunvnet: fix rx packet length check to allow for TSO
This patch fixes the rx packet length check in the sunvnet driver to allow
for a TSO max packet length greater than the LDC channel negotiated MTU.
These are negotiated separately and there is no requirement that
port->tsolen be less than port->rmtu, but if it isn't, it'll drop packets
with rx length errors.
Signed-off-by: David L Stevens <david.stevens@oracle.com>
Acked-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/sun/sunvnet.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/net/ethernet/sun/sunvnet.c b/drivers/net/ethernet/sun/sunvnet.c index d2835bf7b4fb..b5a1d3d7b0bf 100644 --- a/drivers/net/ethernet/sun/sunvnet.c +++ b/drivers/net/ethernet/sun/sunvnet.c | |||
@@ -351,10 +351,15 @@ static int vnet_rx_one(struct vnet_port *port, struct vio_net_desc *desc) | |||
351 | unsigned int len = desc->size; | 351 | unsigned int len = desc->size; |
352 | unsigned int copy_len; | 352 | unsigned int copy_len; |
353 | struct sk_buff *skb; | 353 | struct sk_buff *skb; |
354 | int maxlen; | ||
354 | int err; | 355 | int err; |
355 | 356 | ||
356 | err = -EMSGSIZE; | 357 | err = -EMSGSIZE; |
357 | if (unlikely(len < ETH_ZLEN || len > port->rmtu)) { | 358 | if (port->tso && port->tsolen > port->rmtu) |
359 | maxlen = port->tsolen; | ||
360 | else | ||
361 | maxlen = port->rmtu; | ||
362 | if (unlikely(len < ETH_ZLEN || len > maxlen)) { | ||
358 | dev->stats.rx_length_errors++; | 363 | dev->stats.rx_length_errors++; |
359 | goto out_dropped; | 364 | goto out_dropped; |
360 | } | 365 | } |