diff options
| author | Dimitry Andric <dim@FreeBSD.org> | 2022-08-14 19:06:07 +0000 |
|---|---|---|
| committer | Dimitry Andric <dim@FreeBSD.org> | 2022-08-14 19:27:34 +0000 |
| commit | 762489657117037d963aa840374280583acd32c2 (patch) | |
| tree | b4a7676db40daef702bb0a88f608ac09ce59dfce /sys/netinet/tcp_stacks | |
| parent | 9762d48b7f12f7085d3f6ad83b3ca5ebd226b697 (diff) | |
| download | src-762489657117037d963aa840374280583acd32c2.tar.gz src-762489657117037d963aa840374280583acd32c2.zip | |
Fix unused variable warning in tcp_stacks's bbr.c
With clang 15, the following -Werror warning is produced:
sys/netinet/tcp_stacks/bbr.c:11925:11: error: variable 'rtr_cnt' set but not used [-Werror,-Wunused-but-set-variable]
uint32_t rtr_cnt = 0;
^
The 'rtr_cnt' variable was in bbr.c when it was first added, but it
appears to have been a debugging aid that has never been used, so remove
it.
MFC after: 3 days
Diffstat (limited to 'sys/netinet/tcp_stacks')
| -rw-r--r-- | sys/netinet/tcp_stacks/bbr.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/sys/netinet/tcp_stacks/bbr.c b/sys/netinet/tcp_stacks/bbr.c index 9d5a3d52776e..47c7f1ee0eee 100644 --- a/sys/netinet/tcp_stacks/bbr.c +++ b/sys/netinet/tcp_stacks/bbr.c @@ -11922,7 +11922,6 @@ bbr_output_wtime(struct tcpcb *tp, const struct timeval *tv) int32_t prefetch_so_done = 0; int32_t prefetch_rsm = 0; uint32_t tot_len = 0; - uint32_t rtr_cnt = 0; uint32_t maxseg, pace_max_segs, p_maxseg; int32_t csum_flags = 0; int32_t hw_tls; @@ -12242,7 +12241,6 @@ recheck_resend: goto recheck_resend; #endif } - rtr_cnt++; if (rsm->r_flags & BBR_HAS_SYN) { /* Only retransmit a SYN by itself */ len = 0; @@ -12291,7 +12289,6 @@ recheck_resend: bbr->r_ctl.rc_tlp_send = NULL; sack_rxmit = 1; len = rsm->r_end - rsm->r_start; - rtr_cnt++; if ((bbr->rc_resends_use_tso == 0) && (len > maxseg)) len = maxseg; |