Date: Thu, 17 Oct 2024 12:56:00 -0600 From: Warner Losh <imp@bsdimp.com> To: Jessica Clarke <jrtc27@freebsd.org> Cc: Wolfram Schneider <wosch@freebsd.org>, Warner Losh <imp@freebsd.org>, src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, "<dev-commits-src-main@freebsd.org>" <dev-commits-src-main@freebsd.org> Subject: Re: git: 1a37caeb076b - main - modules: Add missing opt_*.h files for stand-alone compile Message-ID: <CANCZdfoDVhZ8x2LLh0tGhhzUpHc%2BphUdpeWOD66JzQj34OOZ5Q@mail.gmail.com> In-Reply-To: <8549EB6C-DB57-4ECC-8020-C799F9789D5F@freebsd.org> References: <202410122058.49CKwkrP038188@gitrepo.freebsd.org> <CAMWY7CDGAM_F3cn86DZuUgghF7g8BDnSgTbwr4s_JQPE8uUxKg@mail.gmail.com> <CANCZdfrVapV2VBvLFxAhMpbLeYPJRvV3BT1D3UxBP-nR2zPEiQ@mail.gmail.com> <8549EB6C-DB57-4ECC-8020-C799F9789D5F@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--00000000000080cc240624b0bdcc Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, Oct 17, 2024 at 12:26=E2=80=AFPM Jessica Clarke <jrtc27@freebsd.org= > wrote: > On 17 Oct 2024, at 19:20, Warner Losh <imp@bsdimp.com> wrote: > > > > > > > > On Thu, Oct 17, 2024, 11:29=E2=80=AFAM Wolfram Schneider <wosch@freebsd= .org> > wrote: > >> On Sat, 12 Oct 2024 at 22:58, Warner Losh <imp@freebsd.org> wrote: > >> > > >> > The branch main has been updated by imp: > >> > > >> > URL: > https://cgit.FreeBSD.org/src/commit/?id=3D1a37caeb076b9d31e13c54691d7f1ee= b589798bb > >> > > >> > commit 1a37caeb076b9d31e13c54691d7f1eeb589798bb > >> > Author: Warner Losh <imp@FreeBSD.org> > >> > AuthorDate: 2024-10-12 20:40:41 +0000 > >> > Commit: Warner Losh <imp@FreeBSD.org> > >> > CommitDate: 2024-10-12 20:40:41 +0000 > >> > > >> > modules: Add missing opt_*.h files for stand-alone compile > >> > > >> > Standalone compile that we at least create these opt_*.h files. > >> > > >> > Sponsored by: Netflix > >> > --- > >> > sys/modules/linux/Makefile | 2 ++ > >> > sys/modules/linux64/Makefile | 5 ++++- > >> > 2 files changed, 6 insertions(+), 1 deletion(-) > >> > > >> > diff --git a/sys/modules/linux/Makefile b/sys/modules/linux/Makefile > >> > index eb4e17ec706b..36dfbbe2192f 100644 > >> > --- a/sys/modules/linux/Makefile > >> > +++ b/sys/modules/linux/Makefile > >> > @@ -33,6 +33,7 @@ SRCS=3D linux${SFX}_dummy_machdep.c \ > >> > linux_uid16.c \ > >> > linux_vdso.c \ > >> > linux_xattr.c \ > >> > + opt_compat.h \ > >> > opt_inet.h \ > >> > opt_inet6.h \ > >> > opt_ktrace.h \ > >> > @@ -41,6 +42,7 @@ SRCS=3D linux${SFX}_dummy_machdep.c \ > >> > bus_if.h \ > >> > device_if.h \ > >> > vnode_if.h > >> > + > >> > .if ${MACHINE_CPUARCH} =3D=3D "i386" || ${MACHINE_CPUARCH} =3D=3D "= amd64" > >> > SRCS+=3D linux_dummy_x86.c > >> > VDSODEPS=3Dlinux_vdso_gettc_x86.inc > >> > diff --git a/sys/modules/linux64/Makefile > b/sys/modules/linux64/Makefile > >> > index b23891a65a4f..119d90126ab3 100644 > >> > --- a/sys/modules/linux64/Makefile > >> > +++ b/sys/modules/linux64/Makefile > >> > @@ -28,14 +28,17 @@ SRCS=3D linux_dummy_machdep.c \ > >> > linux_vdso.c \ > >> > linux_timer.c \ > >> > linux_xattr.c \ > >> > - opt_ktrace.h \ > >> > + opt_compat.h \ > >> > opt_inet6.h \ > >> > + opt_ktrace.h \ > >> > opt_posix.h \ > >> > + opt_usb.h \ > >> > bus_if.h \ > >> > device_if.h \ > >> > vnode_if.h \ > >> > linux_support.S \ > >> > linux_vdso_inc.S > >> > + > >> > .if ${MACHINE_CPUARCH} =3D=3D "amd64" > >> > SRCS+=3D linux_dummy_x86.c > >> > .endif > >> > >> After this commit I get stale symlinks in /usr/obj > >> > >> make -j$(nproc) buildworld > log.buildworld 2>&1; make -j$(nproc) > >> buildkernel >& log.buildkernel; > >> ./tools/build/stale-symlink-buildworld.sh > >> > >> amd64: > >> stale symlink detected: lrwxrwxr-x 1 wosch wheel 95 Oct 17 17:00 > >> > /scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/amd64.amd64/sy= s/GENERIC/modules/home/wosch/projects/freebsd-src/sys/modules/linux/opt_com= pat.h > >> -> > /scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/amd64.amd64/sy= s/GENERIC/opt_compat.h > >> > >> stale symlink detected: lrwxrwxr-x 1 wosch wheel 95 Oct 17 17:00 > >> > /scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/amd64.amd64/sy= s/GENERIC/modules/home/wosch/projects/freebsd-src/sys/modules/linux64/opt_c= ompat.h > >> -> > /scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/amd64.amd64/sy= s/GENERIC/opt_compat.h > >> > >> arm64: > >> stale symlink detected: lrwxr-xr-x 1 root wheel 73 Oct 17 17:18 > >> > /usr/obj/home/projects/freebsd-src/arm64.aarch64/sys/GENERIC/modules/home= /projects/freebsd-src/sys/modules/linux64/opt_compat.h > >> -> > /usr/obj/home/projects/freebsd-src/arm64.aarch64/sys/GENERIC/opt_compat.h > >> > >> see also PR: 276235 > > > > The symlinks look right to me. What makes them stale? > > Given the script does find -H -type l, it=E2=80=99ll be that they point t= o > non-existent files. I don=E2=80=99t know what opt_compat.h is for Netflix= but > there=E2=80=99s no such thing in-tree, these are the only references to i= t. > Ah, but there use to be... I rebased a tree that had a number of similar cleanups across its removal and didn't notice. And my checks of my tree turned that file up, but I didn't notice it was in a tree I'd not built out of since before the change: commit 06c07e1203324bfe8ab4526c82dbb750864272d5 Author: Dmitry Chagin <dchagin@FreeBSD.org> Date: Mon Feb 13 19:07:38 2023 +0300 Complete removal of opt_compat.h Since Linux emulation layer build options was removed there is no reaso= n to keep opt_compat.h. Reviewed by: emaste Differential Revision: https://reviews.freebsd.org/D38548 MFC after: 2 weeks So I've reverted since opt_usb.h was also removed from linuxulator. Warner --00000000000080cc240624b0bdcc Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><br></div><br><div class=3D"gmail_quote">= <div dir=3D"ltr" class=3D"gmail_attr">On Thu, Oct 17, 2024 at 12:26=E2=80= =AFPM Jessica Clarke <<a href=3D"mailto:jrtc27@freebsd.org">jrtc27@freeb= sd.org</a>> wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"m= argin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left= :1ex">On 17 Oct 2024, at 19:20, Warner Losh <<a href=3D"mailto:imp@bsdim= p.com" target=3D"_blank">imp@bsdimp.com</a>> wrote:<br> > <br> > <br> > <br> > On Thu, Oct 17, 2024, 11:29=E2=80=AFAM Wolfram Schneider <<a href= =3D"mailto:wosch@freebsd.org" target=3D"_blank">wosch@freebsd.org</a>> w= rote:<br> >> On Sat, 12 Oct 2024 at 22:58, Warner Losh <<a href=3D"mailto:im= p@freebsd.org" target=3D"_blank">imp@freebsd.org</a>> wrote:<br> >> ><br> >> > The branch main has been updated by imp:<br> >> ><br> >> > URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D1a3= 7caeb076b9d31e13c54691d7f1eeb589798bb" rel=3D"noreferrer" target=3D"_blank"= >https://cgit.FreeBSD.org/src/commit/?id=3D1a37caeb076b9d31e13c54691d7f1eeb= 589798bb</a><br> >> ><br> >> > commit 1a37caeb076b9d31e13c54691d7f1eeb589798bb<br> >> > Author:=C2=A0 =C2=A0 =C2=A0Warner Losh <imp@FreeBSD.org>= ;<br> >> > AuthorDate: 2024-10-12 20:40:41 +0000<br> >> > Commit:=C2=A0 =C2=A0 =C2=A0Warner Losh <imp@FreeBSD.org>= ;<br> >> > CommitDate: 2024-10-12 20:40:41 +0000<br> >> ><br> >> >=C2=A0 =C2=A0 =C2=A0modules: Add missing opt_*.h files for sta= nd-alone compile<br> >> ><br> >> >=C2=A0 =C2=A0 =C2=A0Standalone compile that we at least create= these opt_*.h files.<br> >> ><br> >> >=C2=A0 =C2=A0 =C2=A0Sponsored by:=C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0Netflix<br> >> > ---<br> >> >=C2=A0 sys/modules/linux/Makefile=C2=A0 =C2=A0| 2 ++<br> >> >=C2=A0 sys/modules/linux64/Makefile | 5 ++++-<br> >> >=C2=A0 2 files changed, 6 insertions(+), 1 deletion(-)<br> >> ><br> >> > diff --git a/sys/modules/linux/Makefile b/sys/modules/linux/M= akefile<br> >> > index eb4e17ec706b..36dfbbe2192f 100644<br> >> > --- a/sys/modules/linux/Makefile<br> >> > +++ b/sys/modules/linux/Makefile<br> >> > @@ -33,6 +33,7 @@ SRCS=3D linux${SFX}_dummy_machdep.c \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_uid16.c \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_vdso.c \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_xattr.c \<br> >> > +=C2=A0 =C2=A0 =C2=A0 =C2=A0opt_compat.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0opt_inet.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0opt_inet6.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0opt_ktrace.h \<br> >> > @@ -41,6 +42,7 @@ SRCS=3D linux${SFX}_dummy_machdep.c \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0bus_if.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0device_if.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0vnode_if.h<br> >> > +<br> >> >=C2=A0 .if ${MACHINE_CPUARCH} =3D=3D "i386" || ${MAC= HINE_CPUARCH} =3D=3D "amd64"<br> >> >=C2=A0 SRCS+=3D linux_dummy_x86.c<br> >> >=C2=A0 VDSODEPS=3Dlinux_vdso_gettc_x86.inc<br> >> > diff --git a/sys/modules/linux64/Makefile b/sys/modules/linux= 64/Makefile<br> >> > index b23891a65a4f..119d90126ab3 100644<br> >> > --- a/sys/modules/linux64/Makefile<br> >> > +++ b/sys/modules/linux64/Makefile<br> >> > @@ -28,14 +28,17 @@ SRCS=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0linux_d= ummy_machdep.c \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_vdso.c \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_timer.c \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_xattr.c \<br> >> > -=C2=A0 =C2=A0 =C2=A0 =C2=A0opt_ktrace.h \<br> >> > +=C2=A0 =C2=A0 =C2=A0 =C2=A0opt_compat.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0opt_inet6.h \<br> >> > +=C2=A0 =C2=A0 =C2=A0 =C2=A0opt_ktrace.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0opt_posix.h \<br> >> > +=C2=A0 =C2=A0 =C2=A0 =C2=A0opt_usb.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0bus_if.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0device_if.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0vnode_if.h \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_support.S \<br> >> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_vdso_inc.S<br> >> > +<br> >> >=C2=A0 .if ${MACHINE_CPUARCH} =3D=3D "amd64"<br> >> >=C2=A0 SRCS+=3D linux_dummy_x86.c<br> >> >=C2=A0 .endif<br> >> <br> >> After this commit I get stale symlinks in /usr/obj<br> >> <br> >> make -j$(nproc) buildworld=C2=A0 > log.buildworld 2>&1; = make -j$(nproc)<br> >> buildkernel >& log.buildkernel;<br> >> ./tools/build/stale-symlink-buildworld.sh<br> >> <br> >> amd64:<br> >> stale symlink detected: lrwxrwxr-x=C2=A0 1 wosch wheel 95 Oct 17 1= 7:00<br> >> /scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/amd64.a= md64/sys/GENERIC/modules/home/wosch/projects/freebsd-src/sys/modules/linux/= opt_compat.h<br> >> -> /scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/a= md64.amd64/sys/GENERIC/opt_compat.h<br> >> <br> >> stale symlink detected: lrwxrwxr-x=C2=A0 1 wosch wheel 95 Oct 17 1= 7:00<br> >> /scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/amd64.a= md64/sys/GENERIC/modules/home/wosch/projects/freebsd-src/sys/modules/linux6= 4/opt_compat.h<br> >> -> /scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/a= md64.amd64/sys/GENERIC/opt_compat.h<br> >> <br> >> arm64:<br> >> stale symlink detected: lrwxr-xr-x=C2=A0 1 root wheel 73 Oct 17 17= :18<br> >> /usr/obj/home/projects/freebsd-src/arm64.aarch64/sys/GENERIC/modul= es/home/projects/freebsd-src/sys/modules/linux64/opt_compat.h<br> >> -> /usr/obj/home/projects/freebsd-src/arm64.aarch64/sys/GENERIC= /opt_compat.h<br> >> <br> >> see also PR: 276235<br> > <br> > The symlinks look right to me. What makes them stale?<br> <br> Given the script does find -H -type l, it=E2=80=99ll be that they point to<= br> non-existent files. I don=E2=80=99t know what opt_compat.h is for Netflix b= ut<br> there=E2=80=99s no such thing in-tree, these are the only references to it.= <br></blockquote><div><br></div><div>Ah, but there use to be...=C2=A0 I reb= ased a tree that had a number of similar cleanups across</div><div>its remo= val and didn't notice. And my checks of my tree turned that file up, bu= t I didn't</div><div>notice it was in a tree I'd not built out of s= ince before the change:</div><div><br></div><div>commit 06c07e1203324bfe8ab= 4526c82dbb750864272d5<br>Author: Dmitry Chagin <dchagin@FreeBSD.org><= br>Date: =C2=A0 Mon Feb 13 19:07:38 2023 +0300<br><br>=C2=A0 =C2=A0 Complet= e removal of opt_compat.h<br><br>=C2=A0 =C2=A0 Since Linux emulation layer = build options was removed there is no reason<br>=C2=A0 =C2=A0 to keep opt_c= ompat.h.<br><br>=C2=A0 =C2=A0 Reviewed by: =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0emaste<br>=C2=A0 =C2=A0 Differential Revision: =C2=A0<a href=3D"h= ttps://reviews.freebsd.org/D38548">https://reviews.freebsd.org/D38548</a><b= r>=C2=A0 =C2=A0 MFC after: =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= 2 weeks<br></div><div><br></div><div>So I've reverted since opt_usb.h w= as also removed from linuxulator.</div><div><br></div><div>Warner</div></di= v></div> --00000000000080cc240624b0bdcc--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfoDVhZ8x2LLh0tGhhzUpHc%2BphUdpeWOD66JzQj34OOZ5Q>