Commit 6052ae16 authored by Sreenivasa Honnur's avatar Sreenivasa Honnur Committed by David S. Miller

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: default avatarSreenivasa Honnur <sreenivasa.honnur@neterion.com>
Signed-off-by: default avatarRamkrishna Vepa <ram.vepa@neterion.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 3255da41
...@@ -817,7 +817,6 @@ vxge_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -817,7 +817,6 @@ vxge_xmit(struct sk_buff *skb, struct net_device *dev)
u64 dma_pointer; u64 dma_pointer;
struct vxge_tx_priv *txdl_priv = NULL; struct vxge_tx_priv *txdl_priv = NULL;
struct __vxge_hw_fifo *fifo_hw; struct __vxge_hw_fifo *fifo_hw;
u32 max_mss = 0x0;
int offload_type; int offload_type;
unsigned long flags = 0; unsigned long flags = 0;
int vpath_no = 0; int vpath_no = 0;
...@@ -969,10 +968,6 @@ vxge_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -969,10 +968,6 @@ vxge_xmit(struct sk_buff *skb, struct net_device *dev)
int mss = vxge_tcp_mss(skb); int mss = vxge_tcp_mss(skb);
if (mss) { if (mss) {
max_mss = dev->mtu + ETH_HLEN -
VXGE_HW_TCPIP_HEADER_MAX_SIZE;
if (mss > max_mss)
mss = max_mss;
vxge_debug_tx(VXGE_TRACE, vxge_debug_tx(VXGE_TRACE,
"%s: %s:%d mss = %d", "%s: %s:%d mss = %d",
dev->name, __func__, __LINE__, mss); dev->name, __func__, __LINE__, mss);
......
...@@ -35,8 +35,6 @@ ...@@ -35,8 +35,6 @@
VXGE_HW_HEADER_VLAN_SIZE + \ VXGE_HW_HEADER_VLAN_SIZE + \
VXGE_HW_HEADER_SNAP_SIZE) VXGE_HW_HEADER_SNAP_SIZE)
#define VXGE_HW_TCPIP_HEADER_MAX_SIZE (64 + 64)
/* 32bit alignments */ /* 32bit alignments */
#define VXGE_HW_HEADER_ETHERNET_II_802_3_ALIGN 2 #define VXGE_HW_HEADER_ETHERNET_II_802_3_ALIGN 2
#define VXGE_HW_HEADER_802_2_SNAP_ALIGN 2 #define VXGE_HW_HEADER_802_2_SNAP_ALIGN 2
......
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