Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 20 Nov 2021 00:10:49 -0800
From:      Gleb Smirnoff <glebius@freebsd.org>
To:        src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: ff94500855c1 - main - Add tcp_freecb() - single place to free tcpcb.
Message-ID:  <YZitiVfP%2BrQxy5o4@FreeBSD.org>
In-Reply-To: <202111190428.1AJ4SEAh021121@gitrepo.freebsd.org>
References:  <202111190428.1AJ4SEAh021121@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Nov 19, 2021 at 04:28:14AM +0000, Gleb Smirnoff wrote:
T> The branch main has been updated by glebius:
T> 
T> URL: https://cgit.FreeBSD.org/src/commit/?id=ff94500855c16d0d9cc18aa8b0ba73ea94020c56
T> 
T> commit ff94500855c16d0d9cc18aa8b0ba73ea94020c56
T> Author:     Gleb Smirnoff <glebius@FreeBSD.org>
T> AuthorDate: 2021-11-19 04:26:09 +0000
T> Commit:     Gleb Smirnoff <glebius@FreeBSD.org>
T> CommitDate: 2021-11-19 04:27:45 +0000
T> 
T>     Add tcp_freecb() - single place to free tcpcb.
T>     
T>     Until this change there were two places where we would free tcpcb -
T>     tcp_discardcb() in case if all timers are drained and tcp_timer_discard()
T>     otherwise.  They were pretty much copy-n-paste, except that in the
T>     default case we would run tcp_hc_update().  Merge this into single
T>     function tcp_freecb() and move new short version of tcp_timer_discard()
T>     to tcp_timer.c and make it static.
T>     
T>     Reviewed by:            rrs, hselasky
T>     Differential revision:  https://reviews.freebsd.org/D32965

Should have been https://reviews.freebsd.org/D32966

-- 
Gleb Smirnoff



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YZitiVfP%2BrQxy5o4>