From nobody Sun Apr 12 21:38:22 2026 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4fv3ng635Cz6Z0bP; Sun, 12 Apr 2026 21:38:47 +0000 (UTC) (envelope-from herbert@gojira.at) Received: from fhigh-a4-smtp.messagingengine.com (fhigh-a4-smtp.messagingengine.com [103.168.172.155]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4fv3ng44y7z461Y; Sun, 12 Apr 2026 21:38:47 +0000 (UTC) (envelope-from herbert@gojira.at) Authentication-Results: mx1.freebsd.org; none Received: from phl-compute-02.internal (phl-compute-02.internal [10.202.2.42]) by mailfhigh.phl.internal (Postfix) with ESMTP id 82A8A14000F7; Sun, 12 Apr 2026 17:38:45 -0400 (EDT) Received: from phl-frontend-04 ([10.202.2.163]) by phl-compute-02.internal (MEProxy); Sun, 12 Apr 2026 17:38:45 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gojira.at; h=cc :cc:content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:subject :subject:to:to; s=fm2; t=1776029925; x=1776116325; bh=0yjJKk+xU4 Gj7Mt9uqHDSy3SIUyDmIPvLIz5vRhVfK4=; b=d7YfBX7qekCSECVwNCRXAFOjjn SEsOqRqD+UYvCS8MBrc/1ThLqo0MheKGVyQi5qMge3QDt+1CUCYkucHYKF6oTazU dbe8sPsChj9BZCCPH+ML8kC48zPFooVz6paUXzFlmZNStJw8CJ1uSW/MchKgUaWL noMZ42FS2IV4VwJHRrgyxtbEk2GHmg8B4/hBIA3xB1KmskG85mHdJibkWvqrC5yo P2nPLe03iig1t+8WKc5dATE2KmaCI4fbdiRFh5yUxIvpZgoyRcV3Fz+07OWx/Rvw 8IrajDHHUkIJWMdafFFtrxl91wQieXanavKGR8P8XBCCXumjSm6y0jiHxewQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:subject:subject:to :to:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm2; t= 1776029925; x=1776116325; bh=0yjJKk+xU4Gj7Mt9uqHDSy3SIUyDmIPvLIz 5vRhVfK4=; b=n+yaGx//ew3auO26uPVSbwsZmpgT3amC1wBN3p4DYIIVaY3tMRr xJ8IA8TkQ84MuvGvYp9fy1fTqjxu59+GTmz/aLCR4G+n7jhtW5IlrNBsm/YuQ2RB Hg2oZHW1A4rpesyH6mWc74bET51LCPr/4xyqxlcdsXm6cF552yMx6AAeAAygIlKk /CnZgKENdewUPpy+df5F0/B0pOjH0ymPp7t5jaZmTL/kG+RRaQ1j+9Klsi8gdOby 9o+NOAqHrqXlG433BPf993R9b9quN++LQudbxqRfRN3fpJR7MOXfGF5aJFX/4UEX vcSplLZAxnYG88SoMWzxLMKXYEpvODy0kIw== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeefhedrtddtgdefieegfecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpuffrtefokffrpgfnqfghnecuuegr ihhlohhuthemuceftddtnecunecujfgurhepfffkhffvvefujghffgggtgesthdtredttd ervdenucfhrhhomhepfdfjvghrsggvrhhtucflrdcuufhkuhhhrhgrfdcuoehhvghrsggv rhhtsehgohhjihhrrgdrrghtqeenucggtffrrghtthgvrhhnpeefhfehffegvdehuedtue fgkeefieffieejhfeggeekgeevjeegfeffgeeglefhtdenucffohhmrghinhepfhhrvggv sghsugdrohhrghenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfh hrohhmpehhvghrsggvrhhtsehgohhjihhrrgdrrghtpdhnsggprhgtphhtthhopeegpdhm ohguvgepshhmthhpohhuthdprhgtphhtthhopehglhgvsghiuhhssehfrhgvvggsshgurd horhhgpdhrtghpthhtohepshhrtgdqtghomhhmihhtthgvrhhssehfrhgvvggsshgurdho rhhgpdhrtghpthhtohepuggvvhdqtghomhhmihhtshdqshhrtgdqrghllhesfhhrvggvsg hsugdrohhrghdprhgtphhtthhopeguvghvqdgtohhmmhhithhsqdhsrhgtqdhmrghinhes fhhrvggvsghsugdrohhrgh X-ME-Proxy: Feedback-ID: i64fe486c:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Sun, 12 Apr 2026 17:38:44 -0400 (EDT) Date: Sun, 12 Apr 2026 23:38:22 +0200 Message-ID: <87h5pfvoz5.wl-herbert@gojira.at> From: "Herbert J. Skuhra" To: Gleb Smirnoff Cc: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: Re: git: 40dbb06fa73c - main - inpcb: retire INP_DROPPED and in_pcbdrop() In-Reply-To: <69dbe5f7.3af83.6ab643b3@gitrepo.freebsd.org> References: <69dbe5f7.3af83.6ab643b3@gitrepo.freebsd.org> User-Agent: Wanderlust/2.15.9 (Almost Unreal) Emacs/31.0 Mule/6.0 List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:151847, ipnet:103.168.172.0/24, country:AU] X-Rspamd-Queue-Id: 4fv3ng44y7z461Y X-Spamd-Bar: ---- On Sun, 12 Apr 2026 20:35:35 +0200, Gleb Smirnoff wrote: > > The branch main has been updated by glebius: > > URL: https://cgit.FreeBSD.org/src/commit/?id=40dbb06fa73cac37d57563c07e55efd0cabbd488 > > commit 40dbb06fa73cac37d57563c07e55efd0cabbd488 > Author: Gleb Smirnoff > AuthorDate: 2026-04-12 18:33:07 +0000 > Commit: Gleb Smirnoff > CommitDate: 2026-04-12 18:33:07 +0000 > > inpcb: retire INP_DROPPED and in_pcbdrop() > > The inpcb flag INP_DROPPED served two purposes. > > It was used by TCP and subsystems running on top of TCP as a flag that > marks a connection that is now in TCPS_CLOSED, but was in some other state > before (not a new-born connection). Create a new TCP flag TF_DISCONNECTED > for this purpose. > > The in_pcbdrop() was a TCP's version of in_pcbdisconnect() that also sets > INP_DROPPED. Use in_pcbdisconnect() instead. > > Second purpose of INP_DROPPED was a negative lookup mask in > inp_smr_lock(), as SMR-protected lookup may see inpcbs that had been > removed from the hash. We already have had INP_INHASHLIST that marks > inpcb that is in hash. Convert it into INP_UNCONNECTED with the opposite > meaning. This allows to combine it with INP_FREED for the negative lookup > mask. > > The Chelsio/ToE and kTLS changes are done with some style refactoring, > like moving inp/tp assignments up and using macros for that. However, no > deep thinking was taken to check if those checks are really needed, it > could be that some are not. > > Reviewed by: rrs > Differential Revision: https://reviews.freebsd.org/D56186 > --- > sys/dev/cxgbe/crypto/t6_kern_tls.c | 6 +- > sys/dev/cxgbe/crypto/t7_kern_tls.c | 2 +- > sys/dev/cxgbe/cxgbei/cxgbei.c | 17 ++-- > sys/dev/cxgbe/cxgbei/icl_cxgbei.c | 13 ++- > sys/dev/cxgbe/iw_cxgbe/qp.c | 24 +++-- > sys/dev/cxgbe/nvmf/nvmf_che.c | 23 +++-- > sys/dev/cxgbe/tom/t4_connect.c | 3 +- > sys/dev/cxgbe/tom/t4_cpl_io.c | 44 ++++----- > sys/dev/cxgbe/tom/t4_ddp.c | 18 ++-- > sys/dev/cxgbe/tom/t4_listen.c | 14 ++- > sys/dev/cxgbe/tom/t4_tls.c | 18 ++-- > sys/dev/cxgbe/tom/t4_tom.c | 10 +- > sys/kern/uipc_ktls.c | 71 ++++--------- > sys/netinet/in_pcb.c | 74 +++++--------- > sys/netinet/in_pcb.h | 9 +- > sys/netinet/in_pcb_var.h | 2 + > sys/netinet/tcp_hpts.c | 12 ++- > sys/netinet/tcp_hpts_test.c | 3 +- > sys/netinet/tcp_log_buf.c | 17 ++-- > sys/netinet/tcp_output.c | 1 + > sys/netinet/tcp_stacks/bbr.c | 2 +- > sys/netinet/tcp_stacks/rack.c | 4 +- > sys/netinet/tcp_subr.c | 3 +- > sys/netinet/tcp_timewait.c | 5 +- > sys/netinet/tcp_usrreq.c | 198 ++++++++++++++----------------------- > sys/netinet/tcp_var.h | 2 +- > sys/netinet/toecore.c | 10 +- > sys/netinet6/in6_pcb.c | 8 +- > sys/netipsec/xform_tcp.c | 8 +- > 29 files changed, 258 insertions(+), 363 deletions(-) This commit seems to break GENERIC-NODEBUG: cc -target x86_64-unknown-freebsd16.0 --sysroot=/usr/obj/home/herbert/sources/freebsd/main/src/amd64.amd64/tmp -B/usr/obj/home/herbert/sources/freebsd/main/src/amd64.amd64/tmp/usr/bin -c -O2 -pipe -fno-strict-aliasing -g -nostdinc -I. -I/home/herbert/sources/freebsd/main/src/sys -I/home/herbert/sources/freebsd/main/src/sys/contrib/ck/include -I/home/herbert/sources/freebsd/main/src/sys/contrib/libfdt -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -MD -MF.depend.tcp_usrreq.o -MTtcp_usrreq.o -fdebug-prefix-map=./machine=/home/herbert/sources/freebsd/main/src/sys/amd64/include -fdebug-prefix-map=./x86=/home/herbert/sources/freebsd/main/src/sys/x86/include -fdebug-prefix-map=./i386=/home/herbert/sources/freebsd/main/src/sys/i386/include -mcmodel=kernel -mno-red-zone -mno-mmx -mno-sse -msoft-float -fno-asynchronous-unwind-tables -ffreestanding -fwrapv -fstack-protector -mretpoline -gdwarf-4 -Wall -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Wcast-qual -Wundef -Wno-pointer-sign -D__printf__=__freebsd_kprintf__ -Wmissing-include-dirs -fdiagnostics-show-option -Wno-unknown-pragmas -Wswitch -Wno-error=tautological-compare -Wno-error=empty-body -Wno-error=parentheses-equality -Wno-error=unused-function -Wno-error=pointer-sign -Wno-error=shift-negative-value -Wno-address-of-packed-member -Wno-format-zero-length -mno-aes -mno-avx -std=gnu17 -Werror /home/herbert/sources/freebsd/main/src/sys/netinet/tcp_usrreq.c /home/herbert/sources/freebsd/main/src/sys/netinet/tcp_usrreq.c:2589:16: error: unused variable 'inp' [-Werror,-Wunused-variable] 2589 | struct inpcb *inp = tptoinpcb(tp); | ^~~