From nobody Fri Jul 14 11:28:54 2023 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 4R2Tky6Gskz2tlhG; Fri, 14 Jul 2023 11:28:58 +0000 (UTC) (envelope-from tuexen@freebsd.org) Received: from drew.franken.de (mail-n.franken.de [193.175.24.27]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "*.franken.de", Issuer "Sectigo RSA Domain Validation Secure Server CA" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4R2Tky3SsXz3P67; Fri, 14 Jul 2023 11:28:58 +0000 (UTC) (envelope-from tuexen@freebsd.org) Authentication-Results: mx1.freebsd.org; none Received: from smtpclient.apple (unknown [IPv6:2a02:8109:1140:c3d:85a6:7e27:ddf8:905b]) (Authenticated sender: micmac) by mail-n.franken.de (Postfix) with ESMTPSA id 9AF4380573EAA; Fri, 14 Jul 2023 13:28:55 +0200 (CEST) Content-Type: text/plain; charset=us-ascii 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: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3731.600.7\)) Subject: Re: git: be78a31188c5 - main - tcp: fix build issue for some cc modules From: tuexen@freebsd.org In-Reply-To: Date: Fri, 14 Jul 2023 13:28:54 +0200 Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Content-Transfer-Encoding: quoted-printable Message-Id: <0C3C0CAD-523A-4A77-8942-4316DB49EF57@freebsd.org> References: <202307132031.36DKVYIK019476@gitrepo.freebsd.org> To: Konstantin Belousov X-Mailer: Apple Mail (2.3731.600.7) X-Spam-Status: No, score=-2.9 required=5.0 tests=ALL_TRUSTED,BAYES_00, T_SCC_BODY_TEXT_LINE autolearn=disabled version=3.4.1 X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on mail-n.franken.de X-Rspamd-Queue-Id: 4R2Tky3SsXz3P67 X-Spamd-Bar: ---- X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:680, ipnet:193.174.0.0/15, country:DE] X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-ThisMailContainsUnwantedMimeParts: N > On 14. Jul 2023, at 13:23, Konstantin Belousov = wrote: >=20 > On Thu, Jul 13, 2023 at 08:31:34PM +0000, Michael Tuexen wrote: >> The branch main has been updated by tuexen: >>=20 >> URL: = https://cgit.FreeBSD.org/src/commit/?id=3Dbe78a31188c530c93700396ecfdb5604= a8f22fff >>=20 >> commit be78a31188c530c93700396ecfdb5604a8f22fff >> Author: Michael Tuexen >> AuthorDate: 2023-07-13 16:56:25 +0000 >> Commit: Michael Tuexen >> CommitDate: 2023-07-13 16:56:25 +0000 >>=20 >> tcp: fix build issue for some cc modules >>=20 >> The TCP_HHOOK option was moved from opt_inet.h to opt_global.h in >> = https://cgit.FreeBSD.org/src/commit/?id=3De68b3792440cac248347afe08ba5881a= 00ba6523 >> The corresponding changes in two Makefiles were missed, which = resulted >> in not building cc_cdg, cc_chd, cc_hd, and cc_vegas anymore. >>=20 >> Reported by: void@f-m.fm >> Reviewed by: rrs, rscheff >> Sponsored by: Netflix, Inc. >> Differential Revision: https://reviews.freebsd.org/D41010 >> --- >> sys/modules/cc/Makefile | 6 +++--- >> sys/modules/khelp/Makefile | 6 +++--- >> 2 files changed, 6 insertions(+), 6 deletions(-) >>=20 >> diff --git a/sys/modules/cc/Makefile b/sys/modules/cc/Makefile >> index 3f7110024722..b595cc204481 100644 >> --- a/sys/modules/cc/Makefile >> +++ b/sys/modules/cc/Makefile >> @@ -8,9 +8,9 @@ SUBDIR=3D cc_newreno \ >>=20 >> # Do we have the TCP_HHOOK symbol defined? If not, there is no point = in >> # building these modules by default. >> -# We will default to building these modules unless $OPT_INET is = defined >> -# and does not contain the TCP_HHOOK option. >> -.if defined(ALL_MODULES) || ${OPT_INET:UTCP_HHOOK:MTCP_HHOOK} !=3D = "" >> +# We will default to building these modules if $OPT_GLOBAL does = contain >> +# the TCP_HHOOK option. >> +.if defined(ALL_MODULES) || ${OPT_GLOBAL:UTCP_HHOOK:MTCP_HHOOK} !=3D = "" >> SUBDIR+=3D \ >> cc_cdg \ >> cc_chd \ >> diff --git a/sys/modules/khelp/Makefile b/sys/modules/khelp/Makefile >> index 256d8838c573..c01d61541062 100644 >> --- a/sys/modules/khelp/Makefile >> +++ b/sys/modules/khelp/Makefile >> @@ -4,9 +4,9 @@ SUBDIR=3D >>=20 >> # Do we have the TCP_HHOOK symbol defined? If not, there is no point = in >> # building this modules by default. >> -# We will default to building this module unless $OPT_INET is = defined >> -# and does not contain the TCP_HHOOK option. >> -.if defined(ALL_MODULES) || ${OPT_INET:UTCP_HHOOK:MTCP_HHOOK} !=3D = "" >> +# We will default to building this module if $OPT_GLOBAL does = contain >> +# the TCP_HHOOK option. >> +.if defined(ALL_MODULES) || ${OPT_GLOBAL:UTCP_HHOOK:MTCP_HHOOK} !=3D = "" >> SUBDIR+=3D h_ertt >> .endif >>=20 > It seems modules are actually broken for some configurations. Some problems are known and being worked on. Could you share your kernel conf file and tell us, in which directory you are running the make command? Best regards Michael > I get >=20 > =3D=3D=3D> cc/cc_cdg (all) > /usr/home/kostik/work/DEV/src/sys/netinet/cc/cc_cdg.c:585:47: error: = no member named 't_osd' in 'struct tcpcb' > e_t =3D (struct ertt *)khelp_get_osd(&CCV(ccv, t_osd), ertt_id); > ~~~~~~~~~^~~~~~ > /usr/home/kostik/work/DEV/src/sys/netinet/cc/cc_module.h:59:41: note: = expanded from macro 'CCV' > #define CCV(ccv, what) (ccv)->ccvc.tcp->what > ~~~~~~~~~~~~~~~ ^ > 1 error generated. > --- cc_cdg.o --- > *** [cc_cdg.o] Error code 1 >=20