diff options
author | Ilpo Järvinen <ilpo.jarvinen@helsinki.fi> | 2007-05-31 03:16:47 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-10 19:47:55 -0400 |
commit | 35e8694198ba94b62df8aa35fa6e52a1cfb86df2 (patch) | |
tree | 9f7d2882f4216478853cc1f520982dae37e9a635 /net | |
parent | af610b4ca19f513a50d47ea93ed57241383c8081 (diff) |
[TCP]: Remove num_acked>0 checks from cong.ctrl mods pkts_acked
There is no need for such check in pkts_acked because the
callback is not invoked unless at least one segment got fully
ACKed (i.e., the snd_una moved past skb's end_seq) by the
cumulative ACK's snd_una advancement.
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r-- | net/ipv4/tcp_bic.c | 2 | ||||
-rw-r--r-- | net/ipv4/tcp_cubic.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/net/ipv4/tcp_bic.c b/net/ipv4/tcp_bic.c index 4586211e3757..5dba0fc8f579 100644 --- a/net/ipv4/tcp_bic.c +++ b/net/ipv4/tcp_bic.c | |||
@@ -210,7 +210,7 @@ static void bictcp_acked(struct sock *sk, u32 cnt, s32 rtt) | |||
210 | { | 210 | { |
211 | const struct inet_connection_sock *icsk = inet_csk(sk); | 211 | const struct inet_connection_sock *icsk = inet_csk(sk); |
212 | 212 | ||
213 | if (cnt > 0 && icsk->icsk_ca_state == TCP_CA_Open) { | 213 | if (icsk->icsk_ca_state == TCP_CA_Open) { |
214 | struct bictcp *ca = inet_csk_ca(sk); | 214 | struct bictcp *ca = inet_csk_ca(sk); |
215 | cnt -= ca->delayed_ack >> ACK_RATIO_SHIFT; | 215 | cnt -= ca->delayed_ack >> ACK_RATIO_SHIFT; |
216 | ca->delayed_ack += cnt; | 216 | ca->delayed_ack += cnt; |
diff --git a/net/ipv4/tcp_cubic.c b/net/ipv4/tcp_cubic.c index 485d7ea35f75..80bd084a9f91 100644 --- a/net/ipv4/tcp_cubic.c +++ b/net/ipv4/tcp_cubic.c | |||
@@ -314,7 +314,7 @@ static void bictcp_acked(struct sock *sk, u32 cnt, s32 rtt_us) | |||
314 | struct bictcp *ca = inet_csk_ca(sk); | 314 | struct bictcp *ca = inet_csk_ca(sk); |
315 | u32 delay; | 315 | u32 delay; |
316 | 316 | ||
317 | if (cnt > 0 && icsk->icsk_ca_state == TCP_CA_Open) { | 317 | if (icsk->icsk_ca_state == TCP_CA_Open) { |
318 | cnt -= ca->delayed_ack >> ACK_RATIO_SHIFT; | 318 | cnt -= ca->delayed_ack >> ACK_RATIO_SHIFT; |
319 | ca->delayed_ack += cnt; | 319 | ca->delayed_ack += cnt; |
320 | } | 320 | } |