Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 11 Oct 2022 13:10:14 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        Mark Millard <marklmi@yahoo.com>
Cc:        freebsd-arm <freebsd-arm@freebsd.org>, bob prohaska <fbsd@www.zefox.net>
Subject:   Re: FYI: FreeBSD-14.0-CURRENT-arm-armv7-GENERICSD-20220930-42dc8696df5-258315.img is broken for RPi2 v1.1 (so: armv7)
Message-ID:  <CANCZdfom2TLLHD1SzY8_1TuxTC8GoWwNMGYHvaT-XefHgHmJcw@mail.gmail.com>
In-Reply-To: <CANCZdfrdnxUyBtj0cvAwiX%2BNJZppqrtButU48jiUVgW4op%2B-FQ@mail.gmail.com>
References:  <6B46F46A-2CAF-42C9-9A04-63567D7DB9B2@yahoo.com> <D9B791B7-106A-402E-AD8C-F811EB315560@yahoo.com> <CANCZdfoJ=E=ef86PRaYsvgXWLAu=AdbN%2B_kiv0vPhKVksqPY%2Bg@mail.gmail.com> <FC871551-7C49-4751-8763-2E8F82C1480A@yahoo.com> <CANCZdfoU2Ln2vqymT66Yu6ZdxSrZLAx1%2BA_hstLYSrzBmduHBw@mail.gmail.com> <28D78C89-E195-4EF1-BB7B-E7F75060BDC7@yahoo.com> <5B631C27-E68C-4F38-96B5-B311110A8F86@yahoo.com> <6EF84694-7CD7-4A9F-BF9C-DFFB52F557AD@yahoo.com> <CANCZdfrdnxUyBtj0cvAwiX%2BNJZppqrtButU48jiUVgW4op%2B-FQ@mail.gmail.com>

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

On Tue, Oct 11, 2022 at 1:03 PM Warner Losh <imp@bsdimp.com> wrote:

>
>
> On Tue, Oct 11, 2022 at 12:50 PM Mark Millard <marklmi@yahoo.com> wrote:
>
>> On 2022-Oct-11, at 09:20, Mark Millard <marklmi@yahoo.com> wrote:
>>
>> > [Summary: Both somewhat before and just after your ConOut commit
>> > work. I'll have to do a rough bisect with the available armv7
>> > artifacts that are after those but before October.]
>> >
>> > On 2022-Oct-11, at 08:26, Mark Millard <marklmi@yahoo.com> wrote:
>> >
>> >> On 2022-Oct-11, at 06:17, Warner Losh <imp@bsdimp.com> wrote:
>> >> . . .
>> >>> . . .
>> >>>
>> >>> The change was made in late August that I'm thinking of, so if you
>> could find a
>> >>> snapshot from early August July that would be a useful data point:
>> >>>
>> >>> commit df065f699f1ff819bb9607c44a6754275ab335ed
>> >>> Author: Warner Losh <imp@FreeBSD.org>
>> >>> Date:   Fri Aug 26 15:46:33 2022 -0600
>> >>>
>> >>>   stand: More sensible defaults when ConOut is missing
>> >>
>> >> I'll look at finding and trying some artifact build to extract a
>> >> armv7 EFI loader from. Available snapshot history does not
>> >> go back that far so far as I know.
>> >>
>> >> But I may not be able to look into this immediately.
>> >>
>> >> In main:
>> >>
>> >> commit df065f699f1ff819bb9607c44a6754275ab335ed
>> >> Author:     Warner Losh <imp@FreeBSD.org>
>> >> AuthorDate: 2022-08-26 21:46:33 +0000
>> >> Commit:     Warner Losh <imp@FreeBSD.org>
>> >> CommitDate: 2022-08-27 04:17:56 +0000
>> >>
>> >> It looks like the closest prior artifact is at:
>> >>
>> >>
>> https://artifact.ci.freebsd.org/snapshot/main/a358db5603702d5de5fd75f5bd=
16bbf7c0ab673f/arm/armv7/?C=3DM&O=3DD
>> >> git: a358db560370 - main - socket(2): bring documentation up tp date
>> Gleb Smirnoff
>> >>
>> >> with date/time: 2022-Aug-26 16:26
>> >>
>> >> So I expect to use that.
>> >
>> > Turns out that I had the time.
>> >
>> > Use of the ./boot/loader.efi from a358db560370 substituted
>> > into the failing main snapshot based microsd card worked
>> > fine for both contexts:
>> >
>> > A) Just serial console.
>> > and:
>> > B) Serial console and HDMI console at the same time.
>> >
>> > I also tried the oldest artifact build with the loader change
>> > in place:
>> >
>> >
>> https://artifact.ci.freebsd.org/snapshot/main/7ed3228323ef4f9e3130603ea6=
8c3be9c2ed50ce/arm/armv7/
>> > git: 7ed3228323ef - main - stand: Document that boot0 uses BIOS Warner
>> Losh
>> > Date/time: 2022-Aug-27 05:12
>> >
>> > This also worked for both (A) and (B).
>> >
>> > So it looks like I'll be doing a rough bisect to find a
>> > before/after pair. It might have some elapsed time to
>> > finish.
>>
>> The date/times encoded into the file names below
>> are those shown for base.txz . The overall artifact
>> directory date/time is somewhat later. The -good
>> vs. -bad suffix indicates the boot status.
>>
>> # ls -C1d *
>> boot-2022-08-26-16-26-a358db560370-good
>> boot-2022-08-27-05-12-7ed3228323ef-good
>> boot-2022-09-12-18-30-c198adf39498-good
>> boot-2022-09-15-01-28-145a50dcda7a-good
>> boot-2022-09-16-14-26-b4174079576a-good
>> boot-2022-09-16-15-45-b44869cba1b3-good
>> boot-2022-09-16-18-02-dd2b9c296776-bad
>> boot-2022-09-16-20-51-30cfb3c8ee3d-bad
>> boot-2022-09-18-00-46-c3707bd3d658-bad
>>
>> For:
>>
>> boot-2022-09-16-15-45-b44869cba1b3-good
>> boot-2022-09-16-18-02-dd2b9c296776-bad
>>
>> there are no armv7 artifacts available between.
>>
>> The range is:
>>
>>         =E2=80=A2 git: b44869cba1b3 - main - sound: add patch for Lenovo=
 Legion 5
>> Intel Nuno Teixeira
>>         =E2=80=A2 git: a705c72f2142 - main - stand: use archsw.arch_copy=
in
>> instead of i386_copyin Warner Losh
>>         =E2=80=A2 git: 4c670b53a000 - main - stand: use archsw.arch_copy=
in
>> instead of direct call Warner Losh
>>         =E2=80=A2 git: 8b19d28d68a3 - main - stand: Create MOD_ALIGN mac=
ro and
>> use it everywhere Warner Losh
>>         =E2=80=A2 git: bca9c87b6104 - main - stand: Create common/modinf=
o.h
>> Warner Losh
>>         =E2=80=A2 git: 5d1531d9d4e7 - main - stand: Move md_copymodules =
into
>> modinfo.c and reduce copies Warner Losh
>>         =E2=80=A2 git: 2e6ed47a4609 - main - stand: Move MOD_xxx macros =
from
>> modinfo.h to .c Warner Losh
>>         =E2=80=A2 git: fc352701ff3a - main - stand: collapse all copies =
of
>> *copyenv into md_copyenv Warner Losh
>>         =E2=80=A2 git: e895ab3fbdc1 - main - stand: Remove dead store to
>> bi_kernelname Warner Losh
>>         =E2=80=A2 git: d43bcf62a218 - main - stand: Stop support booting=
 4.x and
>> earlier kernels Warner Losh
>>         =E2=80=A2 git: 59b1d074280d - main - i386: Mark the obsolete fie=
lds in
>> bootinfo with _was_ Warner Losh
>>         =E2=80=A2 git: 4134f677eb39 - main - i386: Make boot loader smal=
ler by
>> reducing size of bootinfo Warner Losh
>>         =E2=80=A2 git: 9758dd3de1cd - main - stand: Allocate bootinfo ra=
ther than
>> have it be static Warner Losh
>>         =E2=80=A2 git: c0ecae78abbe - main - stand/elf: Only support swa=
pping
>> headers on powerpc. Warner Losh
>>         =E2=80=A2 git: dd2b9c296776 - main - stand: fix mismerge Warner =
Losh
>>
>
> Yea, I did a bunch of refactoring. I'm surprised that this produced a
> change at all. Would be nice to
> know which one of these caused the problems.
>

None of the other platforms I tested on when I did the refactor had issue
or saw any kind of change. So
there may be other issues going on that also take out the boot console...


> Side note about /boot/msdos :
>>
>> The Oct-07 snapshot of main [so: 14] has:
>>
>> # ls -Tld /boot/msdos /boot/efi
>> drwxr-xr-x  1 root  wheel  16384 Jan  1 00:00:00 1980 /boot/efi
>> lrwxr-xr-x  1 root  wheel     51 Oct  7 10:36:20 2022 /boot/msdos ->
>> /usr/obj/usr/src/arm.armv7/release/GENERIC/boot/efi
>>
>
> That's likely my fault, it should be a link to a plain 'efi' I'll prep a
> fix.
>

I think this is the right fix:
https://reviews.freebsd.org/D36941
Please comment....

--00000000000055495205eac70704
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 Tue, Oct 11, 2022 at 1:03 PM Warne=
r Losh &lt;<a href=3D"mailto:imp@bsdimp.com">imp@bsdimp.com</a>&gt; wrote:<=
br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8e=
x;border-left:1px solid rgb(204,204,204);padding-left:1ex"><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 Tue, Oct 11, 2022 at 12:50 PM Mark Millard &lt;<a =
href=3D"mailto:marklmi@yahoo.com" target=3D"_blank">marklmi@yahoo.com</a>&g=
t; wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0p=
x 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 202=
2-Oct-11, at 09:20, Mark Millard &lt;<a href=3D"mailto:marklmi@yahoo.com" t=
arget=3D"_blank">marklmi@yahoo.com</a>&gt; wrote:<br>
<br>
&gt; [Summary: Both somewhat before and just after your ConOut commit<br>
&gt; work. I&#39;ll have to do a rough bisect with the available armv7<br>
&gt; artifacts that are after those but before October.]<br>
&gt; <br>
&gt; On 2022-Oct-11, at 08:26, Mark Millard &lt;<a href=3D"mailto:marklmi@y=
ahoo.com" target=3D"_blank">marklmi@yahoo.com</a>&gt; wrote:<br>
&gt; <br>
&gt;&gt; On 2022-Oct-11, at 06:17, Warner Losh &lt;<a href=3D"mailto:imp@bs=
dimp.com" target=3D"_blank">imp@bsdimp.com</a>&gt; wrote:<br>
&gt;&gt; . . .<br>
&gt;&gt;&gt; . . .<br>
&gt;&gt;&gt; <br>
&gt;&gt;&gt; The change was made in late August that I&#39;m thinking of, s=
o if you could find a<br>
&gt;&gt;&gt; snapshot from early August July that would be a useful data po=
int:<br>
&gt;&gt;&gt; <br>
&gt;&gt;&gt; commit df065f699f1ff819bb9607c44a6754275ab335ed<br>
&gt;&gt;&gt; Author: Warner Losh &lt;imp@FreeBSD.org&gt;<br>
&gt;&gt;&gt; Date:=C2=A0 =C2=A0Fri Aug 26 15:46:33 2022 -0600<br>
&gt;&gt;&gt; <br>
&gt;&gt;&gt;=C2=A0 =C2=A0stand: More sensible defaults when ConOut is missi=
ng<br>
&gt;&gt; <br>
&gt;&gt; I&#39;ll look at finding and trying some artifact build to extract=
 a<br>
&gt;&gt; armv7 EFI loader from. Available snapshot history does not<br>
&gt;&gt; go back that far so far as I know.<br>
&gt;&gt; <br>
&gt;&gt; But I may not be able to look into this immediately.<br>
&gt;&gt; <br>
&gt;&gt; In main:<br>
&gt;&gt; <br>
&gt;&gt; commit df065f699f1ff819bb9607c44a6754275ab335ed<br>
&gt;&gt; Author:=C2=A0 =C2=A0 =C2=A0Warner Losh &lt;imp@FreeBSD.org&gt;<br>
&gt;&gt; AuthorDate: 2022-08-26 21:46:33 +0000<br>
&gt;&gt; Commit:=C2=A0 =C2=A0 =C2=A0Warner Losh &lt;imp@FreeBSD.org&gt;<br>
&gt;&gt; CommitDate: 2022-08-27 04:17:56 +0000<br>
&gt;&gt; <br>
&gt;&gt; It looks like the closest prior artifact is at:<br>
&gt;&gt; <br>
&gt;&gt; <a href=3D"https://artifact.ci.freebsd.org/snapshot/main/a358db560=
3702d5de5fd75f5bd16bbf7c0ab673f/arm/armv7/?C=3DM&amp;O=3DD" rel=3D"noreferr=
er" target=3D"_blank">https://artifact.ci.freebsd.org/snapshot/main/a358db5=
603702d5de5fd75f5bd16bbf7c0ab673f/arm/armv7/?C=3DM&amp;O=3DD</a><br>
&gt;&gt; git: a358db560370 - main - socket(2): bring documentation up tp da=
te Gleb Smirnoff<br>
&gt;&gt; <br>
&gt;&gt; with date/time: 2022-Aug-26 16:26<br>
&gt;&gt; <br>
&gt;&gt; So I expect to use that.<br>
&gt; <br>
&gt; Turns out that I had the time.<br>
&gt; <br>
&gt; Use of the ./boot/loader.efi from a358db560370 substituted<br>
&gt; into the failing main snapshot based microsd card worked<br>
&gt; fine for both contexts:<br>
&gt; <br>
&gt; A) Just serial console.<br>
&gt; and:<br>
&gt; B) Serial console and HDMI console at the same time.<br>
&gt; <br>
&gt; I also tried the oldest artifact build with the loader change<br>
&gt; in place:<br>
&gt; <br>
&gt; <a href=3D"https://artifact.ci.freebsd.org/snapshot/main/7ed3228323ef4=
f9e3130603ea68c3be9c2ed50ce/arm/armv7/" rel=3D"noreferrer" target=3D"_blank=
">https://artifact.ci.freebsd.org/snapshot/main/7ed3228323ef4f9e3130603ea68=
c3be9c2ed50ce/arm/armv7/</a><br>
&gt; git: 7ed3228323ef - main - stand: Document that boot0 uses BIOS Warner=
 Losh<br>
&gt; Date/time: 2022-Aug-27 05:12<br>
&gt; <br>
&gt; This also worked for both (A) and (B).<br>
&gt; <br>
&gt; So it looks like I&#39;ll be doing a rough bisect to find a<br>
&gt; before/after pair. It might have some elapsed time to<br>
&gt; finish.<br>
<br>
The date/times encoded into the file names below<br>
are those shown for base.txz . The overall artifact<br>
directory date/time is somewhat later. The -good<br>
vs. -bad suffix indicates the boot status.<br>
<br>
# ls -C1d *<br>
boot-2022-08-26-16-26-a358db560370-good<br>
boot-2022-08-27-05-12-7ed3228323ef-good<br>
boot-2022-09-12-18-30-c198adf39498-good<br>
boot-2022-09-15-01-28-145a50dcda7a-good<br>
boot-2022-09-16-14-26-b4174079576a-good<br>
boot-2022-09-16-15-45-b44869cba1b3-good<br>
boot-2022-09-16-18-02-dd2b9c296776-bad<br>
boot-2022-09-16-20-51-30cfb3c8ee3d-bad<br>
boot-2022-09-18-00-46-c3707bd3d658-bad<br>
<br>
For:<br>
<br>
boot-2022-09-16-15-45-b44869cba1b3-good<br>
boot-2022-09-16-18-02-dd2b9c296776-bad<br>
<br>
there are no armv7 artifacts available between.<br>
<br>
The range is:<br>
<br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: b44869cba1b3 - main - sound: add=
 patch for Lenovo Legion 5 Intel Nuno Teixeira <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: a705c72f2142 - main - stand: use=
 archsw.arch_copyin instead of i386_copyin Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: 4c670b53a000 - main - stand: use=
 archsw.arch_copyin instead of direct call Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: 8b19d28d68a3 - main - stand: Cre=
ate MOD_ALIGN macro and use it everywhere Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: bca9c87b6104 - main - stand: Cre=
ate common/modinfo.h Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: 5d1531d9d4e7 - main - stand: Mov=
e md_copymodules into modinfo.c and reduce copies Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: 2e6ed47a4609 - main - stand: Mov=
e MOD_xxx macros from modinfo.h to .c Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: fc352701ff3a - main - stand: col=
lapse all copies of *copyenv into md_copyenv Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: e895ab3fbdc1 - main - stand: Rem=
ove dead store to bi_kernelname Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: d43bcf62a218 - main - stand: Sto=
p support booting 4.x and earlier kernels Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: 59b1d074280d - main - i386: Mark=
 the obsolete fields in bootinfo with _was_ Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: 4134f677eb39 - main - i386: Make=
 boot loader smaller by reducing size of bootinfo Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: 9758dd3de1cd - main - stand: All=
ocate bootinfo rather than have it be static Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: c0ecae78abbe - main - stand/elf:=
 Only support swapping headers on powerpc. Warner Losh <br>
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: dd2b9c296776 - main - stand: fix=
 mismerge Warner Losh<br></blockquote><div><br></div><div>Yea, I did a bunc=
h of refactoring. I&#39;m surprised that this produced a change at all. Wou=
ld be nice to</div><div>know which one of these caused the problems.</div><=
/div></div></blockquote><div><br></div><div>None of the other platforms I t=
ested on when I did the refactor had issue or saw any kind of change. So</d=
iv><div>there may be other issues going on that also take out the boot cons=
ole...</div><div>=C2=A0</div><blockquote class=3D"gmail_quote" style=3D"mar=
gin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1=
ex"><div dir=3D"ltr"><div class=3D"gmail_quote"><blockquote class=3D"gmail_=
quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,=
204);padding-left:1ex">
Side note about /boot/msdos :<br>
<br>
The Oct-07 snapshot of main [so: 14] has:<br>
<br>
# ls -Tld /boot/msdos /boot/efi<br>
drwxr-xr-x=C2=A0 1 root=C2=A0 wheel=C2=A0 16384 Jan=C2=A0 1 00:00:00 1980 /=
boot/efi<br>
lrwxr-xr-x=C2=A0 1 root=C2=A0 wheel=C2=A0 =C2=A0 =C2=A051 Oct=C2=A0 7 10:36=
:20 2022 /boot/msdos -&gt; /usr/obj/usr/src/arm.armv7/release/GENERIC/boot/=
efi<br></blockquote><div><br></div><div>That&#39;s likely my fault, it shou=
ld be a link to a plain &#39;efi&#39; I&#39;ll prep a fix.</div></div></div=
></blockquote><div><br></div><div>I think this is the right fix:</div><div>=
<a href=3D"https://reviews.freebsd.org/D36941">https://reviews.freebsd.org/=
D36941</a></div><div>Please comment....=C2=A0</div></div></div>

--00000000000055495205eac70704--



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