Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 17 Oct 2024 12:20:04 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        Wolfram Schneider <wosch@freebsd.org>
Cc:        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:  <CANCZdfrVapV2VBvLFxAhMpbLeYPJRvV3BT1D3UxBP-nR2zPEiQ@mail.gmail.com>
In-Reply-To: <CAMWY7CDGAM_F3cn86DZuUgghF7g8BDnSgTbwr4s_JQPE8uUxKg@mail.gmail.com>
References:  <202410122058.49CKwkrP038188@gitrepo.freebsd.org> <CAMWY7CDGAM_F3cn86DZuUgghF7g8BDnSgTbwr4s_JQPE8uUxKg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000049b280624b03d26
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

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 "amd=
64"
> >  SRCS+=3D linux_dummy_x86.c
> >  VDSODEPS=3Dlinux_vdso_gettc_x86.inc
> > diff --git a/sys/modules/linux64/Makefile b/sys/modules/linux64/Makefil=
e
> > 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?

Warner


> -Wolfram
>
> --
> Wolfram Schneider <wosch@FreeBSD.org> https://wolfram.schneider.org
>

--000000000000049b280624b03d26
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr" =
class=3D"gmail_attr">On Thu, Oct 17, 2024, 11:29=E2=80=AFAM Wolfram Schneid=
er &lt;<a href=3D"mailto:wosch@freebsd.org">wosch@freebsd.org</a>&gt; wrote=
:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;bor=
der-left:1px #ccc solid;padding-left:1ex">On Sat, 12 Oct 2024 at 22:58, War=
ner Losh &lt;<a href=3D"mailto:imp@freebsd.org" target=3D"_blank" rel=3D"no=
referrer">imp@freebsd.org</a>&gt; wrote:<br>
&gt;<br>
&gt; The branch main has been updated by imp:<br>
&gt;<br>
&gt; URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D1a37caeb076b=
9d31e13c54691d7f1eeb589798bb" rel=3D"noreferrer noreferrer" target=3D"_blan=
k">https://cgit.FreeBSD.org/src/commit/?id=3D1a37caeb076b9d31e13c54691d7f1e=
eb589798bb</a><br>
&gt;<br>
&gt; commit 1a37caeb076b9d31e13c54691d7f1eeb589798bb<br>
&gt; Author:=C2=A0 =C2=A0 =C2=A0Warner Losh &lt;imp@FreeBSD.org&gt;<br>
&gt; AuthorDate: 2024-10-12 20:40:41 +0000<br>
&gt; Commit:=C2=A0 =C2=A0 =C2=A0Warner Losh &lt;imp@FreeBSD.org&gt;<br>
&gt; CommitDate: 2024-10-12 20:40:41 +0000<br>
&gt;<br>
&gt;=C2=A0 =C2=A0 =C2=A0modules: Add missing opt_*.h files for stand-alone =
compile<br>
&gt;<br>
&gt;=C2=A0 =C2=A0 =C2=A0Standalone compile that we at least create these op=
t_*.h files.<br>
&gt;<br>
&gt;=C2=A0 =C2=A0 =C2=A0Sponsored by:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0Netflix<br>
&gt; ---<br>
&gt;=C2=A0 sys/modules/linux/Makefile=C2=A0 =C2=A0| 2 ++<br>
&gt;=C2=A0 sys/modules/linux64/Makefile | 5 ++++-<br>
&gt;=C2=A0 2 files changed, 6 insertions(+), 1 deletion(-)<br>
&gt;<br>
&gt; diff --git a/sys/modules/linux/Makefile b/sys/modules/linux/Makefile<b=
r>
&gt; index eb4e17ec706b..36dfbbe2192f 100644<br>
&gt; --- a/sys/modules/linux/Makefile<br>
&gt; +++ b/sys/modules/linux/Makefile<br>
&gt; @@ -33,6 +33,7 @@ SRCS=3D linux${SFX}_dummy_machdep.c \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_uid16.c \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_vdso.c \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_xattr.c \<br>
&gt; +=C2=A0 =C2=A0 =C2=A0 =C2=A0opt_compat.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0opt_inet.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0opt_inet6.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0opt_ktrace.h \<br>
&gt; @@ -41,6 +42,7 @@ SRCS=3D linux${SFX}_dummy_machdep.c \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0bus_if.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0device_if.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0vnode_if.h<br>
&gt; +<br>
&gt;=C2=A0 .if ${MACHINE_CPUARCH} =3D=3D &quot;i386&quot; || ${MACHINE_CPUA=
RCH} =3D=3D &quot;amd64&quot;<br>
&gt;=C2=A0 SRCS+=3D linux_dummy_x86.c<br>
&gt;=C2=A0 VDSODEPS=3Dlinux_vdso_gettc_x86.inc<br>
&gt; diff --git a/sys/modules/linux64/Makefile b/sys/modules/linux64/Makefi=
le<br>
&gt; index b23891a65a4f..119d90126ab3 100644<br>
&gt; --- a/sys/modules/linux64/Makefile<br>
&gt; +++ b/sys/modules/linux64/Makefile<br>
&gt; @@ -28,14 +28,17 @@ SRCS=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0linux_dummy_mach=
dep.c \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_vdso.c \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_timer.c \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_xattr.c \<br>
&gt; -=C2=A0 =C2=A0 =C2=A0 =C2=A0opt_ktrace.h \<br>
&gt; +=C2=A0 =C2=A0 =C2=A0 =C2=A0opt_compat.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0opt_inet6.h \<br>
&gt; +=C2=A0 =C2=A0 =C2=A0 =C2=A0opt_ktrace.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0opt_posix.h \<br>
&gt; +=C2=A0 =C2=A0 =C2=A0 =C2=A0opt_usb.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0bus_if.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0device_if.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0vnode_if.h \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_support.S \<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0linux_vdso_inc.S<br>
&gt; +<br>
&gt;=C2=A0 .if ${MACHINE_CPUARCH} =3D=3D &quot;amd64&quot;<br>
&gt;=C2=A0 SRCS+=3D linux_dummy_x86.c<br>
&gt;=C2=A0 .endif<br>
<br>
After this commit I get stale symlinks in /usr/obj<br>
<br>
make -j$(nproc) buildworld=C2=A0 &gt; log.buildworld 2&gt;&amp;1; make -j$(=
nproc)<br>
buildkernel &gt;&amp; 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 17:00<br>
/scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/amd64.amd64/sys/=
GENERIC/modules/home/wosch/projects/freebsd-src/sys/modules/linux/opt_compa=
t.h<br>
-&gt; /scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/amd64.amd6=
4/sys/GENERIC/opt_compat.h<br>
<br>
stale symlink detected: lrwxrwxr-x=C2=A0 1 wosch wheel 95 Oct 17 17:00<br>
/scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/amd64.amd64/sys/=
GENERIC/modules/home/wosch/projects/freebsd-src/sys/modules/linux64/opt_com=
pat.h<br>
-&gt; /scratch/tmp/wosch/freebsd/home/wosch/projects/freebsd-src/amd64.amd6=
4/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/modules/home/p=
rojects/freebsd-src/sys/modules/linux64/opt_compat.h<br>
-&gt; /usr/obj/home/projects/freebsd-src/arm64.aarch64/sys/GENERIC/opt_comp=
at.h<br>
<br>
see also PR: 276235<br></blockquote></div></div><div dir=3D"auto"><br></div=
><div dir=3D"auto">The symlinks look right to me. What makes them stale?</d=
iv><div dir=3D"auto"><br></div><div dir=3D"auto">Warner=C2=A0</div><div dir=
=3D"auto"><br></div><div dir=3D"auto"><div class=3D"gmail_quote"><blockquot=
e class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc sol=
id;padding-left:1ex">
<br>
-Wolfram<br>
<br>
-- <br>
Wolfram Schneider &lt;wosch@FreeBSD.org&gt; <a href=3D"https://wolfram.schn=
eider.org" rel=3D"noreferrer noreferrer" target=3D"_blank">https://wolfram.=
schneider.org</a><br>
</blockquote></div></div></div>

--000000000000049b280624b03d26--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrVapV2VBvLFxAhMpbLeYPJRvV3BT1D3UxBP-nR2zPEiQ>