Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 11 Oct 2023 15:44:34 GMT
From:      John Baldwin <jhb@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: cb2cd58dbd56 - stable/14 - cxgbe t4_tls: Don't bother returning RX credits for a protocol receive error
Message-ID:  <202310111544.39BFiYBG024309@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/14 has been updated by jhb:

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

commit cb2cd58dbd56f734f4346ce4db3d0412b2809667
Author:     John Baldwin <jhb@FreeBSD.org>
AuthorDate: 2023-09-08 23:30:52 +0000
Commit:     John Baldwin <jhb@FreeBSD.org>
CommitDate: 2023-10-11 15:10:32 +0000

    cxgbe t4_tls: Don't bother returning RX credits for a protocol receive error
    
    Sponsored by:   Chelsio Communications
    Differential Revision:  https://reviews.freebsd.org/D41689
    
    (cherry picked from commit 75af2d951cce7d51d2033405f96f083c01f39f04)
---
 sys/dev/cxgbe/tom/t4_tls.c | 18 +-----------------
 1 file changed, 1 insertion(+), 17 deletions(-)

diff --git a/sys/dev/cxgbe/tom/t4_tls.c b/sys/dev/cxgbe/tom/t4_tls.c
index b8005e213621..29a840067c62 100644
--- a/sys/dev/cxgbe/tom/t4_tls.c
+++ b/sys/dev/cxgbe/tom/t4_tls.c
@@ -1004,7 +1004,7 @@ do_rx_data_tls(const struct cpl_rx_data *cpl, struct toepcb *toep,
 	struct tcpcb *tp;
 	struct socket *so;
 	struct sockbuf *sb;
-	int len, rx_credits;
+	int len;
 
 	len = m->m_pkthdr.len;
 
@@ -1075,22 +1075,6 @@ do_rx_data_tls(const struct cpl_rx_data *cpl, struct toepcb *toep,
 	so->so_error = EBADMSG;
 
 out:
-	/*
-	 * This connection is going to die anyway, so probably don't
-	 * need to bother with returning credits.
-	 */
-	rx_credits = sbspace(sb) > tp->rcv_wnd ? sbspace(sb) - tp->rcv_wnd : 0;
-#ifdef VERBOSE_TRACES
-	CTR4(KTR_CXGBE, "%s: tid %u rx_credits %u rcv_wnd %u",
-	    __func__, toep->tid, rx_credits, tp->rcv_wnd);
-#endif
-	if (rx_credits > 0 && sbused(sb) + tp->rcv_wnd < sb->sb_lowat) {
-		rx_credits = send_rx_credits(toep->vi->adapter, toep,
-		    rx_credits);
-		tp->rcv_wnd += rx_credits;
-		tp->rcv_adv += rx_credits;
-	}
-
 	sorwakeup_locked(so);
 	SOCKBUF_UNLOCK_ASSERT(sb);
 



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