Date: Fri, 14 Jul 2023 13:28:54 +0200 From: tuexen@freebsd.org To: Konstantin Belousov <kostikbel@gmail.com> Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: be78a31188c5 - main - tcp: fix build issue for some cc modules Message-ID: <0C3C0CAD-523A-4A77-8942-4316DB49EF57@freebsd.org> In-Reply-To: <ZLEwLRprzR0gSBmH@kib.kiev.ua> References: <202307132031.36DKVYIK019476@gitrepo.freebsd.org> <ZLEwLRprzR0gSBmH@kib.kiev.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
> On 14. Jul 2023, at 13:23, Konstantin Belousov <kostikbel@gmail.com> = 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 <tuexen@FreeBSD.org> >> AuthorDate: 2023-07-13 16:56:25 +0000 >> Commit: Michael Tuexen <tuexen@FreeBSD.org> >> 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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?0C3C0CAD-523A-4A77-8942-4316DB49EF57>