diff options
author | Ilpo Järvinen <ilpo.jarvinen@helsinki.fi> | 2009-02-27 23:44:42 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-03-02 06:00:17 -0500 |
commit | 9ce01461028d595a6f1cd724fbd7a0dd70464fe4 (patch) | |
tree | d1cf064853fb84001202a72bc1a03e7f0b69d595 /net/ipv4 | |
parent | 0d6a775e27d975e5f9ea8e2911216d84face50ca (diff) |
tcp: get rid of two unnecessary u16s in TCP skb flags copying
I guess these fields were one day 16-bit in the struct but
nowadays they're just using 8 bits anyway.
This is just a precaution, didn't result any change in my
case but who knows what all those varying gcc versions &
options do. I've been told that 16-bit is not so nice with
some cpus.
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4')
-rw-r--r-- | net/ipv4/tcp_output.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c index 1555bb73b638..920c57b90ded 100644 --- a/net/ipv4/tcp_output.c +++ b/net/ipv4/tcp_output.c | |||
@@ -767,7 +767,7 @@ int tcp_fragment(struct sock *sk, struct sk_buff *skb, u32 len, | |||
767 | struct sk_buff *buff; | 767 | struct sk_buff *buff; |
768 | int nsize, old_factor; | 768 | int nsize, old_factor; |
769 | int nlen; | 769 | int nlen; |
770 | u16 flags; | 770 | u8 flags; |
771 | 771 | ||
772 | BUG_ON(len > skb->len); | 772 | BUG_ON(len > skb->len); |
773 | 773 | ||
@@ -1282,7 +1282,7 @@ static int tso_fragment(struct sock *sk, struct sk_buff *skb, unsigned int len, | |||
1282 | { | 1282 | { |
1283 | struct sk_buff *buff; | 1283 | struct sk_buff *buff; |
1284 | int nlen = skb->len - len; | 1284 | int nlen = skb->len - len; |
1285 | u16 flags; | 1285 | u8 flags; |
1286 | 1286 | ||
1287 | /* All of a TSO frame must be composed of paged data. */ | 1287 | /* All of a TSO frame must be composed of paged data. */ |
1288 | if (skb->len != skb->data_len) | 1288 | if (skb->len != skb->data_len) |