Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 4 Jan 2025 14:27:22 GMT
From:      Michael Tuexen <tuexen@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 305c40dc552f - main - TCP BBR: simplify expression
Message-ID:  <202501041427.504ERMWF007705@gitrepo.freebsd.org>

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

URL: https://cgit.FreeBSD.org/src/commit/?id=305c40dc552f9c150eacce95c181798031368cd9

commit 305c40dc552f9c150eacce95c181798031368cd9
Author:     Michael Tuexen <tuexen@FreeBSD.org>
AuthorDate: 2025-01-04 14:25:01 +0000
Commit:     Michael Tuexen <tuexen@FreeBSD.org>
CommitDate: 2025-01-04 14:25:01 +0000

    TCP BBR: simplify expression
    
    There is no need to check partially for bbr->r_ctl.crte being NULL,
    since this can't be true in this path.
    No functional change intended.
    
    Reviewed by:            rrs
    CID:                    1523810
    MFC after:              1 week
    Sponsored by:           Netflix, Inc.
    Differential Revision:  https://reviews.freebsd.org/D48312
---
 sys/netinet/tcp_stacks/bbr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/netinet/tcp_stacks/bbr.c b/sys/netinet/tcp_stacks/bbr.c
index 9e1f83df4298..ed7e07861ebd 100644
--- a/sys/netinet/tcp_stacks/bbr.c
+++ b/sys/netinet/tcp_stacks/bbr.c
@@ -5553,7 +5553,7 @@ lost_rate:
 		bbr_type_log_hdwr_pacing(bbr,
 					 bbr->r_ctl.crte->ptbl->rs_ifp,
 					 rate,
-					 ((bbr->r_ctl.crte == NULL) ? 0 : bbr->r_ctl.crte->rate),
+					 bbr->r_ctl.crte->rate,
 					 __LINE__,
 					 cts,
 					 error);



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