Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 6 Jan 2024 19:44:05 GMT
From:      Richard Scheffenegger <rscheff@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 893ed42eca32 - main - tcp: Make use of enum for sack_changed
Message-ID:  <202401061944.406Ji5aE098387@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by rscheff:

URL: https://cgit.FreeBSD.org/src/commit/?id=893ed42eca32f7b5e3a5b8cef3f364be0d9cf3ef

commit 893ed42eca32f7b5e3a5b8cef3f364be0d9cf3ef
Author:     Richard Scheffenegger <rscheff@FreeBSD.org>
AuthorDate: 2024-01-06 19:07:39 +0000
Commit:     Richard Scheffenegger <rscheff@FreeBSD.org>
CommitDate: 2024-01-06 19:23:52 +0000

    tcp: Make use of enum for sack_changed
    
    No functional change.
    
    Reviewed By:           tuexen, #transport
    MFC after:             3 days
    Sponsored by:          NetApp, Inc.
    Differential Revision: https://reviews.freebsd.org/D43346
---
 sys/netinet/tcp_input.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/sys/netinet/tcp_input.c b/sys/netinet/tcp_input.c
index ab8fc4b4a3e4..1d6430f6ef20 100644
--- a/sys/netinet/tcp_input.c
+++ b/sys/netinet/tcp_input.c
@@ -2499,7 +2499,8 @@ tcp_do_segment(struct tcpcb *tp, struct mbuf *m, struct tcphdr *th,
 			goto dropafterack;
 		}
 		if (tcp_is_sack_recovery(tp, &to)) {
-			if (((sack_changed = tcp_sack_doack(tp, &to, th->th_ack)) != 0) &&
+			sack_changed = tcp_sack_doack(tp, &to, th->th_ack);
+			if ((sack_changed != SACK_NOCHANGE) &&
 			    (tp->t_flags & TF_LRD)) {
 				tcp_sack_lost_retransmission(tp, th);
 			}



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202401061944.406Ji5aE098387>