[TCP]: Remove unnecessary wrapper tcp_packets_out_dec
Makes caller side more obvious, there's no need to have a wrapper for this oneliner! Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
522400623e
commit
e9144bd8da
@ -626,12 +626,6 @@ static inline void tcp_packets_out_inc(struct sock *sk,
|
|||||||
inet_csk(sk)->icsk_rto, TCP_RTO_MAX);
|
inet_csk(sk)->icsk_rto, TCP_RTO_MAX);
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void tcp_packets_out_dec(struct tcp_sock *tp,
|
|
||||||
const struct sk_buff *skb)
|
|
||||||
{
|
|
||||||
tp->packets_out -= tcp_skb_pcount(skb);
|
|
||||||
}
|
|
||||||
|
|
||||||
/* Events passed to congestion control interface */
|
/* Events passed to congestion control interface */
|
||||||
enum tcp_ca_event {
|
enum tcp_ca_event {
|
||||||
CA_EVENT_TX_START, /* first transmit when no packets in flight */
|
CA_EVENT_TX_START, /* first transmit when no packets in flight */
|
||||||
|
@ -2569,7 +2569,7 @@ static int tcp_clean_rtx_queue(struct sock *sk, __s32 *seq_rtt_p)
|
|||||||
last_ackt = skb->tstamp;
|
last_ackt = skb->tstamp;
|
||||||
}
|
}
|
||||||
tcp_dec_pcount_approx(&tp->fackets_out, skb);
|
tcp_dec_pcount_approx(&tp->fackets_out, skb);
|
||||||
tcp_packets_out_dec(tp, skb);
|
tp->packets_out -= tcp_skb_pcount(skb);
|
||||||
tcp_unlink_write_queue(skb, sk);
|
tcp_unlink_write_queue(skb, sk);
|
||||||
sk_stream_free_skb(sk, skb);
|
sk_stream_free_skb(sk, skb);
|
||||||
clear_all_retrans_hints(tp);
|
clear_all_retrans_hints(tp);
|
||||||
|
@ -1735,7 +1735,7 @@ static void tcp_retrans_try_collapse(struct sock *sk, struct sk_buff *skb, int m
|
|||||||
* it is better to underestimate fackets.
|
* it is better to underestimate fackets.
|
||||||
*/
|
*/
|
||||||
tcp_dec_pcount_approx(&tp->fackets_out, next_skb);
|
tcp_dec_pcount_approx(&tp->fackets_out, next_skb);
|
||||||
tcp_packets_out_dec(tp, next_skb);
|
tp->packets_out -= tcp_skb_pcount(next_skb);
|
||||||
sk_stream_free_skb(sk, next_skb);
|
sk_stream_free_skb(sk, next_skb);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user