Date: Fri, 14 Jul 2023 13:39:32 +0200 From: Kristof Provost <kp@FreeBSD.org> To: tuexen@freebsd.org Cc: Konstantin Belousov <kostikbel@gmail.com>, 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: <00949E97-1098-4FC4-B5FB-9F639E93C81D@FreeBSD.org> In-Reply-To: <0C3C0CAD-523A-4A77-8942-4316DB49EF57@freebsd.org> References: <202307132031.36DKVYIK019476@gitrepo.freebsd.org> <ZLEwLRprzR0gSBmH@kib.kiev.ua> <0C3C0CAD-523A-4A77-8942-4316DB49EF57@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--=_MailMate_10D9FA97-377D-4FCC-A767-2224FD2A307B_= Content-Type: text/plain; format=flowed Content-Transfer-Encoding: quoted-printable On 14 Jul 2023, at 13:28, tuexen@freebsd.org wrote: >> On 14. Jul 2023, at 13:23, Konstantin Belousov <kostikbel@gmail.com> = >> wrote: >> >> On Thu, Jul 13, 2023 at 08:31:34PM +0000, Michael Tuexen wrote: >>> The branch main has been updated by tuexen: >>> >>> URL: = >>> https://cgit.FreeBSD.org/src/commit/?id=3Dbe78a31188c530c93700396ecfd= b5604a8f22fff >>> >>> 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 >>> >>> tcp: fix build issue for some cc modules >>> >>> The TCP_HHOOK option was moved from opt_inet.h to opt_global.h in >>> https://cgit.FreeBSD.org/src/commit/?id=3De68b3792440cac248347afe08b= a5881a00ba6523 >>> The corresponding changes in two Makefiles were missed, which = >>> resulted >>> in not building cc_cdg, cc_chd, cc_hd, and cc_vegas anymore. >>> >>> 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(-) >>> >>> 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 \ >>> >>> # 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 >>> >>> # 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 >>> >> 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? > `make buildkernel KERNCONF=3DMINIMAL` will fail (on amd64). Best regards, Kristof --=_MailMate_10D9FA97-377D-4FCC-A767-2224FD2A307B_= Content-Type: text/html Content-Transfer-Encoding: quoted-printable <!DOCTYPE html> <html> <head> <meta http-equiv=3D"Content-Type" content=3D"text/xhtml; charset=3Dutf-8"= > </head> <body><div style=3D"font-family: sans-serif;"><div class=3D"markdown" sty= le=3D"white-space: normal;"> <p dir=3D"auto">On 14 Jul 2023, at 13:28, <a href=3D"mailto:tuexen@freebs= d.org">tuexen@freebsd.org</a> wrote:</p> </div><div class=3D"plaintext" style=3D"white-space: normal;"><blockquote= style=3D"margin: 0 0 5px; padding-left: 5px; border-left: 2px solid #136= BCE; color: #136BCE;"><blockquote style=3D"margin: 0 0 5px; padding-left:= 5px; border-left: 2px solid #136BCE; border-left-color: #4B89CF; color: = #4B89CF;"><p dir=3D"auto">On 14. Jul 2023, at 13:23, Konstantin Belousov = <kostikbel@gmail.com> wrote:</p> <p dir=3D"auto">On Thu, Jul 13, 2023 at 08:31:34PM +0000, Michael Tuexen = wrote:</p> <blockquote style=3D"margin: 0 0 5px; padding-left: 5px; border-left: 2px= solid #136BCE; border-left-color: #4B89CF; color: #4B89CF;"><p dir=3D"au= to">The branch main has been updated by tuexen:</p> <p dir=3D"auto">URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D= be78a31188c530c93700396ecfdb5604a8f22fff">https://cgit.FreeBSD.org/src/co= mmit/?id=3Dbe78a31188c530c93700396ecfdb5604a8f22fff</a></p> <p dir=3D"auto">commit be78a31188c530c93700396ecfdb5604a8f22fff <br> Author: Michael Tuexen <tuexen@FreeBSD.org> <br> AuthorDate: 2023-07-13 16:56:25 +0000 <br> Commit: Michael Tuexen <tuexen@FreeBSD.org> <br> CommitDate: 2023-07-13 16:56:25 +0000</p> <p dir=3D"auto"> tcp: fix build issue for some cc modules</p> <p dir=3D"auto"> The TCP_HHOOK option was moved from opt_inet.h to opt_g= lobal.h in <br> <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3De68b3792440cac24834= 7afe08ba5881a00ba6523">https://cgit.FreeBSD.org/src/commit/?id=3De68b3792= 440cac248347afe08ba5881a00ba6523</a> <br> The corresponding changes in two Makefiles were missed, which resulted <br> in not building cc_cdg, cc_chd, cc_hd, and cc_vegas anymore.</p> <p dir=3D"auto"> Reported by: void@f-m.fm <br> Reviewed by: rrs, rscheff <br> Sponsored by: Netflix, Inc. <br> Differential Revision: <a href=3D"https://reviews.freebsd.org/D41010">= https://reviews.freebsd.org/D41010</a>; <br> --- <br> sys/modules/cc/Makefile | 6 +++--- <br> sys/modules/khelp/Makefile | 6 +++--- <br> 2 files changed, 6 insertions(+), 6 deletions(-)</p> <p dir=3D"auto">diff --git a/sys/modules/cc/Makefile b/sys/modules/cc/Mak= efile <br> index 3f7110024722..b595cc204481 100644 <br> --- a/sys/modules/cc/Makefile <br> +++ b/sys/modules/cc/Makefile <br> @@ -8,9 +8,9 @@ SUBDIR=3D cc_newreno \</p> <p dir=3D"auto"># Do we have the TCP_HHOOK symbol defined? If not, there = is no point in <br> # building these modules by default. <br> -# We will default to building these modules unless $OPT_INET is defined <br> -# and does not contain the TCP_HHOOK option. <br> -.if defined(ALL_MODULES) || ${OPT_INET:UTCP_HHOOK:MTCP_HHOOK} !=3D "" <br> +# We will default to building these modules if $OPT_GLOBAL does contain <br> +# the TCP_HHOOK option. <br> +.if defined(ALL_MODULES) || ${OPT_GLOBAL:UTCP_HHOOK:MTCP_HHOOK} !=3D "" <br> SUBDIR+=3D \ <br> cc_cdg \ <br> cc_chd \ <br> diff --git a/sys/modules/khelp/Makefile b/sys/modules/khelp/Makefile <br> index 256d8838c573..c01d61541062 100644 <br> --- a/sys/modules/khelp/Makefile <br> +++ b/sys/modules/khelp/Makefile <br> @@ -4,9 +4,9 @@ SUBDIR=3D</p> <p dir=3D"auto"># Do we have the TCP_HHOOK symbol defined? If not, there = is no point in <br> # building this modules by default. <br> -# We will default to building this module unless $OPT_INET is defined <br> -# and does not contain the TCP_HHOOK option. <br> -.if defined(ALL_MODULES) || ${OPT_INET:UTCP_HHOOK:MTCP_HHOOK} !=3D "" <br> +# We will default to building this module if $OPT_GLOBAL does contain <br> +# the TCP_HHOOK option. <br> +.if defined(ALL_MODULES) || ${OPT_GLOBAL:UTCP_HHOOK:MTCP_HHOOK} !=3D "" <br> SUBDIR+=3D h_ertt <br> =2Eendif</p> </blockquote><p dir=3D"auto">It seems modules are actually broken for som= e configurations.</p> </blockquote><p dir=3D"auto">Some problems are known and being worked on.= </p> <p dir=3D"auto">Could you share your kernel conf file and tell us, in whi= ch directory <br> you are running the make command?</p> <br></blockquote></div> <div class=3D"markdown" style=3D"white-space: normal;"> <p dir=3D"auto"><code style=3D"padding: 0 0.25em; background-color: #E4E4= E4;">make buildkernel KERNCONF=3DMINIMAL</code> will fail (on amd64).</p>= <p dir=3D"auto">Best regards,<br> Kristof</p> </div> </div> </body> </html> --=_MailMate_10D9FA97-377D-4FCC-A767-2224FD2A307B_=--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?00949E97-1098-4FC4-B5FB-9F639E93C81D>