Date: Wed, 1 Apr 2020 13:10:13 -0600 From: Warner Losh <imp@bsdimp.com> To: Ed Maste <emaste@freebsd.org> Cc: freebsd-reproducibility@freebsd.org Subject: Re: Nonreproducible kernel modules on stable/12 Message-ID: <CANCZdfpX4pO0ZLx2O7qHocaMBzoB1fKV8tjMPdZHRgY-A4MfoA@mail.gmail.com> In-Reply-To: <CAPyFy2Bzuk3tyrPdkw4L2xD%2BWE6qG1mRDG0hGEx0QOxw%2BXVDvA@mail.gmail.com> References: <CAPyFy2Bzuk3tyrPdkw4L2xD%2BWE6qG1mRDG0hGEx0QOxw%2BXVDvA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Sigh
freebsd-update-build is being naughty:
sed -i "" -E "s,^(BRANCH=3D.*)\",\1$2\","
and editing newvers.sh directly.
This doesn't work.
However, the change was
BRANCH=3D"STABLE"
to
BRANCH=3D${BRANCH_OVERRIDE:-STABLE}
maybe a quick hack is to add back the quotes or to change the above sed to
sed -i "" -E "s,^(BRANCH=3D.*),\1\"$2\","
which would work on all branches because the expression
BRANCH=3D"STABLE""-p2"
is well formed as is
BRANCH=3D${BRANCH_OVERRIDE:-STABLE}"-p2"
Warner
On Tue, Mar 31, 2020 at 8:08 AM Ed Maste <emaste@freebsd.org> wrote:
> Originally reported off-list, during a stable/12 freebsd-update build
> kernel modules are not reproducible and thus get included in the
> update.
>
> diffoscope reports that the .SUNW_ctf section is responsible:
>
> --- cam.0.ko
> +++ cam.1.ko
> =E2=94=9C=E2=94=80=E2=94=80 readelf --wide --sections {}
> ...
> =E2=94=82 [30] .shstrtab STRTAB 0000000000000000 0ea8=
28
> 0001f8 00 0 0 1
> =E2=94=82 - [31] .SUNW_ctf PROGBITS 0000000000000000 0f29=
d8
> 00e2ad 00 32 0 4
> =E2=94=82 + [31] .SUNW_ctf PROGBITS 0000000000000000 0f29=
d8
> 00e2b1 00 32 0 4
> =E2=94=82 [32] .symtab SYMTAB 0000000000000000 0e12=
b8
> 009570 18 33 1001 8
> ...
> =E2=94=9C=E2=94=80=E2=94=80 readelf --wide --decompress --hex-dump=3D.SUN=
W_ctf {}
> =E2=94=82 @@ -1,3630 +1,3631 @@
> =E2=94=82
> =E2=94=82 Hex dump of section '.SUNW_ctf':
> =E2=94=82 0x00000000 f1cf0201 00000000 00000000 00000000 .............=
...
> =E2=94=82 0x00000010 08000000 96040000 fc250000 f4310100 .........%...=
1..
> =E2=94=82 - 0x00000020 6aeb0000 78dabc7d 077c54c5 16f7ccbd j...x..}.|T..=
...
> =E2=94=82 - 0x00000030 774b42ca 26b48524 b0402001 02440d10 wKB.&..$.@ ..=
D..
> ...
> =E2=94=82 + 0x00000020 6feb0000 78dabc7d 077c54c5 16f7ccdc o...x..}.|T..=
...
> =E2=94=82 + 0x00000030 bb252165 13da4242 586a0204 081820d4 .%!e..BBXj...=
. .
>
> (And build-ID and GNU debuglink differ as a result.)
>
> Running ctfdump and comparing the result shows the issue:
>
> --- a 2020-03-31 10:04:39.766567000 -0400
> +++ b 2020-03-31 10:04:44.663292000 -0400
> @@ -11,11 +11,11 @@
> cth_funcoff =3D 1174
> cth_typeoff =3D 9724
> cth_stroff =3D 78324
> - cth_strlen =3D 60266
> + cth_strlen =3D 60271
>
> - Label Table
> ----------------------------------------------------------------
>
> - 2407 FreeBSD 12.1-STABLE amd64 1201512
> + 2407 FreeBSD 12.1-FUTUREBUILD amd64 1201512
>
> - Data Objects
> ---------------------------------------------------------------
>
> Aside, we should add ctfdump to diffoscope.
>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfpX4pO0ZLx2O7qHocaMBzoB1fKV8tjMPdZHRgY-A4MfoA>