Commit c8855953 authored by Surjit Reang's avatar Surjit Reang Committed by David S. Miller

S2io: Fix for LRO Bugs

Resubmitting patch from Al Viro <viro@zeniv.linux.org.uk>, with subject -
[PATCH] s2io LRO bugs.
a) initiate_new_session() sets ->tcp_ack to ntohl(...); everything
   else stores and expects to find there the net-endian value.
b) check for monotonic timestamps in verify_l3_l4_lro_capable()
   compares the value sitting in TCP option (right there in the skb->data,
   net-endian 32bit) with the value picked from earlier packet.
   Doing that without ntohl() is an interesting idea and it might even
   work occasionally; unfortunately, it's quite broken.
Signed-off-by: default avatarSurjit Reang <surjit.reang@neterion.com>
Signed-off-by: default avatarRamkrishna Vepa <ram.vepa@neterion.com>
Signed-off-by: default avatarJeff Garzik <jeff@garzik.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 03157ac3
...@@ -8118,7 +8118,7 @@ static void initiate_new_session(struct lro *lro, u8 *l2h, ...@@ -8118,7 +8118,7 @@ static void initiate_new_session(struct lro *lro, u8 *l2h,
lro->iph = ip; lro->iph = ip;
lro->tcph = tcp; lro->tcph = tcp;
lro->tcp_next_seq = tcp_pyld_len + ntohl(tcp->seq); lro->tcp_next_seq = tcp_pyld_len + ntohl(tcp->seq);
lro->tcp_ack = ntohl(tcp->ack_seq); lro->tcp_ack = tcp->ack_seq;
lro->sg_num = 1; lro->sg_num = 1;
lro->total_len = ntohs(ip->tot_len); lro->total_len = ntohs(ip->tot_len);
lro->frags_len = 0; lro->frags_len = 0;
...@@ -8127,10 +8127,10 @@ static void initiate_new_session(struct lro *lro, u8 *l2h, ...@@ -8127,10 +8127,10 @@ static void initiate_new_session(struct lro *lro, u8 *l2h,
* already been done. * already been done.
*/ */
if (tcp->doff == 8) { if (tcp->doff == 8) {
u32 *ptr; __be32 *ptr;
ptr = (u32 *)(tcp+1); ptr = (__be32 *)(tcp+1);
lro->saw_ts = 1; lro->saw_ts = 1;
lro->cur_tsval = *(ptr+1); lro->cur_tsval = ntohl(*(ptr+1));
lro->cur_tsecr = *(ptr+2); lro->cur_tsecr = *(ptr+2);
} }
lro->in_use = 1; lro->in_use = 1;
...@@ -8156,7 +8156,7 @@ static void update_L3L4_header(struct s2io_nic *sp, struct lro *lro) ...@@ -8156,7 +8156,7 @@ static void update_L3L4_header(struct s2io_nic *sp, struct lro *lro)
/* Update tsecr field if this session has timestamps enabled */ /* Update tsecr field if this session has timestamps enabled */
if (lro->saw_ts) { if (lro->saw_ts) {
u32 *ptr = (u32 *)(tcp + 1); __be32 *ptr = (__be32 *)(tcp + 1);
*(ptr+2) = lro->cur_tsecr; *(ptr+2) = lro->cur_tsecr;
} }
...@@ -8181,10 +8181,10 @@ static void aggregate_new_rx(struct lro *lro, struct iphdr *ip, ...@@ -8181,10 +8181,10 @@ static void aggregate_new_rx(struct lro *lro, struct iphdr *ip,
lro->window = tcp->window; lro->window = tcp->window;
if (lro->saw_ts) { if (lro->saw_ts) {
u32 *ptr; __be32 *ptr;
/* Update tsecr and tsval from this packet */ /* Update tsecr and tsval from this packet */
ptr = (u32 *) (tcp + 1); ptr = (__be32 *)(tcp+1);
lro->cur_tsval = *(ptr + 1); lro->cur_tsval = ntohl(*(ptr+1));
lro->cur_tsecr = *(ptr + 2); lro->cur_tsecr = *(ptr + 2);
} }
} }
...@@ -8235,11 +8235,11 @@ static int verify_l3_l4_lro_capable(struct lro *l_lro, struct iphdr *ip, ...@@ -8235,11 +8235,11 @@ static int verify_l3_l4_lro_capable(struct lro *l_lro, struct iphdr *ip,
/* Ensure timestamp value increases monotonically */ /* Ensure timestamp value increases monotonically */
if (l_lro) if (l_lro)
if (l_lro->cur_tsval > *((u32 *)(ptr+2))) if (l_lro->cur_tsval > ntohl(*((__be32 *)(ptr+2))))
return -1; return -1;
/* timestamp echo reply should be non-zero */ /* timestamp echo reply should be non-zero */
if (*((u32 *)(ptr+6)) == 0) if (*((__be32 *)(ptr+6)) == 0)
return -1; return -1;
} }
......
...@@ -809,7 +809,7 @@ struct lro { ...@@ -809,7 +809,7 @@ struct lro {
int in_use; int in_use;
__be16 window; __be16 window;
u32 cur_tsval; u32 cur_tsval;
u32 cur_tsecr; __be32 cur_tsecr;
u8 saw_ts; u8 saw_ts;
}; };
......
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