diff options
author | Sreenivasa Honnur <Sreenivasa.Honnur@neterion.com> | 2009-07-01 17:15:41 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-07-05 22:16:32 -0400 |
commit | 6052ae1676b705c67133542e663ec3380f032e2c (patch) | |
tree | 5e1dbdb57ebb66441da3443ab04928a37ccecbdf /drivers/net/vxge/vxge-main.c | |
parent | 3255da416de6da8b3bac6d30b8eed61b54953480 (diff) |
vxge: Removed the code to bounds check the mss value
- Removed the code to bounds check the mss value. The hardware does bounds checking and
will not allow an oversized mss to lockup the transmit path.
Signed-off-by: Sreenivasa Honnur <sreenivasa.honnur@neterion.com>
Signed-off-by: Ramkrishna Vepa <ram.vepa@neterion.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/vxge/vxge-main.c')
-rw-r--r-- | drivers/net/vxge/vxge-main.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/net/vxge/vxge-main.c b/drivers/net/vxge/vxge-main.c index dd2add1d7566..cda18f815c3a 100644 --- a/drivers/net/vxge/vxge-main.c +++ b/drivers/net/vxge/vxge-main.c | |||
@@ -817,7 +817,6 @@ vxge_xmit(struct sk_buff *skb, struct net_device *dev) | |||
817 | u64 dma_pointer; | 817 | u64 dma_pointer; |
818 | struct vxge_tx_priv *txdl_priv = NULL; | 818 | struct vxge_tx_priv *txdl_priv = NULL; |
819 | struct __vxge_hw_fifo *fifo_hw; | 819 | struct __vxge_hw_fifo *fifo_hw; |
820 | u32 max_mss = 0x0; | ||
821 | int offload_type; | 820 | int offload_type; |
822 | unsigned long flags = 0; | 821 | unsigned long flags = 0; |
823 | int vpath_no = 0; | 822 | int vpath_no = 0; |
@@ -969,10 +968,6 @@ vxge_xmit(struct sk_buff *skb, struct net_device *dev) | |||
969 | 968 | ||
970 | int mss = vxge_tcp_mss(skb); | 969 | int mss = vxge_tcp_mss(skb); |
971 | if (mss) { | 970 | if (mss) { |
972 | max_mss = dev->mtu + ETH_HLEN - | ||
973 | VXGE_HW_TCPIP_HEADER_MAX_SIZE; | ||
974 | if (mss > max_mss) | ||
975 | mss = max_mss; | ||
976 | vxge_debug_tx(VXGE_TRACE, | 971 | vxge_debug_tx(VXGE_TRACE, |
977 | "%s: %s:%d mss = %d", | 972 | "%s: %s:%d mss = %d", |
978 | dev->name, __func__, __LINE__, mss); | 973 | dev->name, __func__, __LINE__, mss); |