From nobody Tue Oct 11 19:10:14 2022 X-Original-To: freebsd-arm@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4Mn52r2jqKz4fDXm for ; Tue, 11 Oct 2022 19:10:28 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-ej1-x62c.google.com (mail-ej1-x62c.google.com [IPv6:2a00:1450:4864:20::62c]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1D4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4Mn52q2YtDz3Mr1 for ; Tue, 11 Oct 2022 19:10:27 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-ej1-x62c.google.com with SMTP id 13so33526398ejn.3 for ; Tue, 11 Oct 2022 12:10:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20210112.gappssmtp.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=66gfnv+gKf2M+jF2rHgW1xvVsL8TgGf7C/iY9n7y2hM=; b=p2gpSWaffDqKIc1A5NyPPGewgdIygG0xTUjNHS3xCnnCkDUgSqDIWsv4EpiaIWugAV 8y9MwDvzVMrjS19ESgUM5QVdT+ETgEupJG5SfmY528yh0rM+xWsdx+8x4EnpUBLeJgLk YUek92ZzP0zLEsacBGXxuud7VhXjR0kE2Enly9hvt+3kkztD4xJsuj3Js9S4e6EYkGT1 zBiB9G/BcHQW42zDJpv/F+4QoXxR5cIhsObeAHUqOlVnvc+xnK0fBIR1r2JMsDMV0CYY eKj8rxplyE0K8urZOm4RkYlPXDTT6Vu+fcobmdGdZp8pMrT94V014XhccDoyXFYHZl26 XIcA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=66gfnv+gKf2M+jF2rHgW1xvVsL8TgGf7C/iY9n7y2hM=; b=vvjnNrFjN7Ya/NKUPbtZgtepXcV+VuP0rfnXHtFzgmUrlyJ6btUapOoA28b5kjNNe0 Kj5dshH9ZsREapAj6RRoxMlxXzdKRF9L8xRwyqMIm+dDSD6UIw3t03DMjfA8CoELODs1 avgjkX6LrjrfYK8jnDdVDQHE6TUWQCotHmhkIyE19dEe4+E3wGtf4dR+txoTOY6FtX1S yh0JOzRSDoPt50X5EiB/hiAt0q0S4JaUQgVO1NlbH/XxFUW5GZb0OWjtP+h2yAfx1ck1 WD826LD+Re1CWINCYa7rBV5kPEHsU1Mo1m1YuR0o05VMVLXSw7GbMF7b7D3ZJoFOh3Ij o/sg== X-Gm-Message-State: ACrzQf3RFuUkAzl0hANXtaw8EOFLP+/mGQ1SrF6+UVQB8di9FG9NBLW5 gfAlkIFenoGXhUB7ZLg6Mm+ZBCG3QvJeoc1TQeNPfQ== X-Google-Smtp-Source: AMsMyM4eQoNfUm1YGR5djYZwHt/G06J8uTTMmgnZIvUel1hdk/VOvaLA5sWzBHApCkzUSAxps5iAoK18ueY8roCYDhM= X-Received: by 2002:a17:906:328c:b0:780:7574:ced2 with SMTP id 12-20020a170906328c00b007807574ced2mr20010730ejw.634.1665515425967; Tue, 11 Oct 2022 12:10:25 -0700 (PDT) List-Id: Porting FreeBSD to ARM processors List-Archive: https://lists.freebsd.org/archives/freebsd-arm List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-freebsd-arm@freebsd.org MIME-Version: 1.0 References: <6B46F46A-2CAF-42C9-9A04-63567D7DB9B2@yahoo.com> <28D78C89-E195-4EF1-BB7B-E7F75060BDC7@yahoo.com> <5B631C27-E68C-4F38-96B5-B311110A8F86@yahoo.com> <6EF84694-7CD7-4A9F-BF9C-DFFB52F557AD@yahoo.com> In-Reply-To: From: Warner Losh Date: Tue, 11 Oct 2022 13:10:14 -0600 Message-ID: Subject: Re: FYI: FreeBSD-14.0-CURRENT-arm-armv7-GENERICSD-20220930-42dc8696df5-258315.img is broken for RPi2 v1.1 (so: armv7) To: Mark Millard Cc: freebsd-arm , bob prohaska Content-Type: multipart/alternative; boundary="00000000000055495205eac70704" X-Rspamd-Queue-Id: 4Mn52q2YtDz3Mr1 X-Spamd-Bar: -- Authentication-Results: mx1.freebsd.org; dkim=pass header.d=bsdimp-com.20210112.gappssmtp.com header.s=20210112 header.b=p2gpSWaf; dmarc=none; spf=none (mx1.freebsd.org: domain of wlosh@bsdimp.com has no SPF policy when checking 2a00:1450:4864:20::62c) smtp.mailfrom=wlosh@bsdimp.com X-Spamd-Result: default: False [-3.00 / 15.00]; NEURAL_HAM_LONG(-1.00)[-1.000]; NEURAL_HAM_MEDIUM(-1.00)[-1.000]; NEURAL_HAM_SHORT(-1.00)[-1.000]; FORGED_SENDER(0.30)[imp@bsdimp.com,wlosh@bsdimp.com]; R_DKIM_ALLOW(-0.20)[bsdimp-com.20210112.gappssmtp.com:s=20210112]; MIME_GOOD(-0.10)[multipart/alternative,text/plain]; FREEMAIL_TO(0.00)[yahoo.com]; MLMMJ_DEST(0.00)[freebsd-arm@freebsd.org]; R_SPF_NA(0.00)[no SPF record]; ARC_NA(0.00)[]; RCVD_TLS_LAST(0.00)[]; MIME_TRACE(0.00)[0:+,1:+,2:~]; ASN(0.00)[asn:15169, ipnet:2a00:1450::/32, country:US]; RCVD_IN_DNSWL_NONE(0.00)[2a00:1450:4864:20::62c:from]; DKIM_TRACE(0.00)[bsdimp-com.20210112.gappssmtp.com:+]; TO_DN_ALL(0.00)[]; RCPT_COUNT_THREE(0.00)[3]; FROM_HAS_DN(0.00)[]; FROM_NEQ_ENVFROM(0.00)[imp@bsdimp.com,wlosh@bsdimp.com]; PREVIOUSLY_DELIVERED(0.00)[freebsd-arm@freebsd.org]; TO_MATCH_ENVRCPT_SOME(0.00)[]; DMARC_NA(0.00)[bsdimp.com]; RCVD_COUNT_TWO(0.00)[2] X-ThisMailContainsUnwantedMimeParts: N --00000000000055495205eac70704 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, Oct 11, 2022 at 1:03 PM Warner Losh wrote: > > > On Tue, Oct 11, 2022 at 12:50 PM Mark Millard wrote: > >> On 2022-Oct-11, at 09:20, Mark Millard 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 wrote: >> > >> >> On 2022-Oct-11, at 06:17, Warner Losh 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 >> >>> 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 >> >> AuthorDate: 2022-08-26 21:46:33 +0000 >> >> Commit: Warner Losh >> >> 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


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


On Tue, Oct 11, 2022 at 12:50 PM Mark Millard <marklmi@yahoo.com&g= t; wrote:
On 202= 2-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, s= o if you could find a
>>> snapshot from early August July that would be a useful data po= int:
>>>
>>> commit df065f699f1ff819bb9607c44a6754275ab335ed
>>> Author: Warner Losh <imp@FreeBSD.org>
>>> Date:=C2=A0 =C2=A0Fri Aug 26 15:46:33 2022 -0600
>>>
>>>=C2=A0 =C2=A0stand: More sensible defaults when ConOut is missi= ng
>>
>> 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:=C2=A0 =C2=A0 =C2=A0Warner Losh <imp@FreeBSD.org>
>> AuthorDate: 2022-08-26 21:46:33 +0000
>> Commit:=C2=A0 =C2=A0 =C2=A0Warner 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/a358db5= 603702d5de5fd75f5bd16bbf7c0ab673f/arm/armv7/?C=3DM&O=3DD
>> git: a358db560370 - main - socket(2): bring documentation up tp da= te 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/7ed3228323ef4f9e3130603ea68= c3be9c2ed50ce/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:

=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: b44869cba1b3 - main - sound: add= patch for Lenovo Legion 5 Intel Nuno Teixeira
=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
=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
=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
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: bca9c87b6104 - main - stand: Cre= ate common/modinfo.h Warner Losh
=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
=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
=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
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: e895ab3fbdc1 - main - stand: Rem= ove dead store to bi_kernelname Warner Losh
=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
=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
=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
=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
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: c0ecae78abbe - main - stand/elf:= Only support swapping headers on powerpc. Warner Losh
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =E2=80=A2 git: dd2b9c296776 - main - stand: fix= mismerge Warner Losh

Yea, I did a bunc= h of refactoring. I'm surprised that this produced a change at all. Wou= ld be nice to
know which one of these caused the problems.
<= /div>

None of the other platforms I t= ested 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 cons= ole...
=C2=A0
Side note about /boot/msdos :

The Oct-07 snapshot of main [so: 14] has:

# ls -Tld /boot/msdos /boot/efi
drwxr-xr-x=C2=A0 1 root=C2=A0 wheel=C2=A0 16384 Jan=C2=A0 1 00:00:00 1980 /= boot/efi
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 -> /usr/obj/usr/src/arm.armv7/release/GENERIC/boot/= efi

That's likely my fault, it shou= ld be a link to a plain 'efi' I'll prep a fix.

I think this is the right fix:
Please comment....=C2=A0
--00000000000055495205eac70704--