Date: Tue, 3 Apr 2018 15:17:23 -0500 From: Kyle Evans <kevans@freebsd.org> To: Zaphod Beeblebrox <zbeeble@gmail.com> Cc: freebsd-current <freebsd-current@freebsd.org> Subject: Re: Call for Testing: UEFI Changes Message-ID: <CACNAnaHSpicqRzH=YAZWxVqSh6HnPSa=FBeTJxoZZOXRrS5NyQ@mail.gmail.com> In-Reply-To: <CACpH0McmjRUm7hRoydCtSwYqhHuHvMvjbSk7yQ8drp823b3WvA@mail.gmail.com> References: <CACNAnaFmu2rFF1w4ar4xHUbN5vHitTLi0Ui6aCjL1MuTj3iJsQ@mail.gmail.com> <5f663141-433c-951d-a350-7369b004415f@alvermark.net> <CACNAnaE1UaqzS4bjQY9uqnhH2mX7XfUKT%2BVA%2Bn0v1=xF2FWXRg@mail.gmail.com> <20180322225644.1480251c4547ce30f3d88de9@dec.sakura.ne.jp> <dd6790b2-dfd2-d851-e851-99dfc489c5e5@ieee.org> <CACNAnaEfw2vzgnqrRkjgzqqtEyOYCeCR%2BP%2BVexPGzbOvNxsJ5A@mail.gmail.com> <CACNAnaE237%2Bb9-ZD1AWpnsSeE5EnDU539QDMDDcdZ5KWJjQwQA@mail.gmail.com> <20180401213831.1535c7511c1d95a258318bc7@dec.sakura.ne.jp> <CA%2B_TYQEFtc0LPSJBuggP1__gPNfaoBTf7qPadbmOXB_-CnHh5Q@mail.gmail.com> <CACpH0Md=ky4gs8pgvFa2nhbiSwdeKU1ZW6c3JdxfERKTOaSUTw@mail.gmail.com> <CACNAnaGgZy5k6hV2VG4PhSvzut9Uzno1SYpk0fmHcQD5S5k8Ag@mail.gmail.com> <CACpH0McmjRUm7hRoydCtSwYqhHuHvMvjbSk7yQ8drp823b3WvA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi, Right- so, `gop set 0` should've immediately cleared your screen and put it into 1920x1080, full stop. If it did not, I think that's indicative of some kind of interestingly broken firmware... Regardless! We're clearly bad at trying to set a mode before the kernel starts, so r331904 sets the default max resolution in such a way that we just don't change the resolution by default anymore and interested parties can bump that up if they want to try it. This Thursday's snapshots should have this included. I think if we're going to change modes as a default, we should have some way for vt/efifb to use EFIRT or be notified by EFIRT of supported resolutions so that vt/efifb can hopefully just handle it all and do it properly. This approach would be more similar I guess to how KMS drivers operate, and less fragile than what we're seeing with these different approaches we've taken. On Tue, Apr 3, 2018 at 3:05 PM, Zaphod Beeblebrox <zbeeble@gmail.com> wrote: > I did as you asked. You can see the result at: > https://owncloud.towernet.ca/s/6K3pGknCyGTi7du > > ... but the long and short of it is that even though the loader is printing > in the center of the screen (text mode?), it sees the 1920x1080 mode as the > "mode 0" ... I even tried "gop set 0" ... which it accepted, but then > continuing to boot produced what you see in the video. > > I think, since it's in the cetner of the screen, that we're looking at 80x25 > text... which is ... 800 x 600 -ish? The kernel definately wants graphics > again ... but ... I dunno ... is it getting it? Is the 80x25 text mode > emulated on a bitmapped screen? > > > On Mon, Apr 2, 2018 at 9:07 PM, Kyle Evans <kevans@freebsd.org> wrote: >> >> On Mon, Apr 2, 2018 at 5:51 PM, Zaphod Beeblebrox <zbeeble@gmail.com> >> wrote: >> > On Sun, Apr 1, 2018 at 7:41 PM, David NewHamlet <wheelcomplex@gmail.com> >> > wrote: >> >> >> >> Fixed in: FreeBSD-12.0-CURRENT-amd64-20180329-r331740-disc1.iso >> >> >> >> >> >> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=176694 >> >> >> >> On Mon, Apr 2, 2018 at 12:38 AM, Tomoaki AOKI >> >> <junchoon@dec.sakura.ne.jp> >> >> wrote: >> >> >> >> > Confirmed both loader (with boot1) part and efirt.ko part. >> >> > Working OK on my ThinkPad420 (with nvidia GPU) at r331864. >> >> > >> >> > No benefit (VGA resolution) but no new harm (no panic nor silent >> >> > reboot). >> >> > >> >> > *Maybe gracefully falling back to mode 0. >> >> > >> >> > As I'm on x11/nvidia-driver, completely no test is done with >> >> > drm-next. >> >> > >> >> > One more to mention. >> >> > I've cherry-picked r330868, r331241 and r331361 on stable/11 after >> >> > r331114, amd64 and works OK as on head. >> >> > Additional cherry-picking of r331365 is OK, too. >> >> > >> >> > Without r330868, my ThinkPad silently reboots within about 10-60 >> >> > minutes range, maybe by actual access to UEFI RS. >> >> > With r331241 without r331361 causes instant panic on loading >> >> > efirt.ko. >> >> > So all 3 (4) revs should be MFC'ed together. >> >> > >> >> > Sorry for delay. >> >> > >> >> > >> >> > On Thu, 22 Mar 2018 10:34:33 -0500 >> >> > Kyle Evans <kevans@freebsd.org> wrote: >> >> > >> >> > > On Thu, Mar 22, 2018 at 10:18 AM, Kyle Evans <kevans@freebsd.org> >> >> > > wrote: >> >> > > > On Thu, Mar 22, 2018 at 10:16 AM, Peter Lei <peter.lei@ieee.org> >> >> > wrote: >> >> > > >> On 3/22/18 8:56 AM, Tomoaki AOKI wrote: >> >> > > >>> Hi. >> >> > > >>> For problem 2, try reverting r331241 alone. >> >> > > >>> This worked for my ThinkPad T420. >> >> > > >> >> >> > > >> >> >> > > >> I also hit this after updating to latest and was about to post >> >> > > >> when >> >> > > >> I >> >> > > >> saw this thread - >> >> > > >> >> >> > > >> I build efirt into the kernel and it's now doing a panic on >> >> > > >> boot. >> >> > > >> It >> >> > > >> appears to be due to this recent addition in dev/efidev/efirt.c >> >> > > >> by >> >> > r331241: >> >> > > >> >> >> > > >>> if (!efi_is_in_map(map, efihdr->memory_size / >> >> > efihdr->descriptor_size, >> >> > > >>> efihdr->descriptor_size, >> >> > > >>> (vm_offset_t)efi_runtime->rt_gettime)) >> >> > { >> >> > > >> >> >> > > >> The faulting address is for "efi_runtime->rt_gettime" (and is >> >> > > >> still >> >> > > >> a >> >> > > >> phys addr here). >> >> > > >> >> >> > > > >> >> > > > The following patch [1] (I can't guarantee how long he'll keep it >> >> > > > available there) should fix this class of problems. >> >> > > > >> >> > > > [1] https://people.freebsd.org/~andrew/0001-Enter-into-the- >> >> > EFI-environment-before-check-the-GetT.patch >> >> > > >> >> > > Now committed as of r331361. >> >> > > _______________________________________________ >> >> > > freebsd-current@freebsd.org mailing list >> >> > > https://lists.freebsd.org/mailman/listinfo/freebsd-current >> >> > > To unsubscribe, send any mail to "freebsd-current-unsubscribe@ >> >> > freebsd.org" >> >> > > >> >> > >> >> > >> >> > -- >> >> > Tomoaki AOKI <junchoon@dec.sakura.ne.jp> >> >> > _______________________________________________ >> >> > freebsd-current@freebsd.org mailing list >> >> > https://lists.freebsd.org/mailman/listinfo/freebsd-current >> >> > To unsubscribe, send any mail to >> >> > "freebsd-current-unsubscribe@freebsd.org" >> >> > >> >> _______________________________________________ >> >> freebsd-current@freebsd.org mailing list >> >> https://lists.freebsd.org/mailman/listinfo/freebsd-current >> >> To unsubscribe, send any mail to >> >> "freebsd-current-unsubscribe@freebsd.org" >> > >> > I've booted that image on my zbook 15. I show in the boot that I can >> > deliberately load efirt.ko ... and it doesn't help. I also show that I >> > can >> > "type blind" after the system boots ... so everything but the screen is >> > working. >> > >> > In case you can't quite make it out, I hit right cursor twice (move to >> > the >> > "live cd" choice) and hit enter. Then I type "root" enter and then >> > "reboot" >> > ... >> >> That is interesting indeed... that's perhaps the polar opposite of >> what was being experienced before- it looks like it's setting a higher >> resolution but the firmware isn't actually putting it into the higher >> resolution. The firmware then lies to us about what resolution it's >> actually in, so we tell the kernel the wrong thing. >> >> You should be able to confirm this, as I recall, by bumping into the >> loader prompt and inspecting the output of `gop get`. That should >> report a higher resolution than it's actually in. >> >> > https://youtu.be/tlmaVJ-3aq0 >> > >> > (The rock sound track is just free audio to mask a barking dog and a >> > radio >> > in the background.) >> > >> >> This was a most enjoyable soundtrack. > > Kyle Evans, Web/Database Developer College of Engineering, Kansas State University kevans91@ksu.edu (785) 532-4643 0012 Seaton Hall On Tue, Apr 3, 2018 at 3:05 PM, Zaphod Beeblebrox <zbeeble@gmail.com> wrote: > I did as you asked. You can see the result at: > https://owncloud.towernet.ca/s/6K3pGknCyGTi7du > > ... but the long and short of it is that even though the loader is printing > in the center of the screen (text mode?), it sees the 1920x1080 mode as the > "mode 0" ... I even tried "gop set 0" ... which it accepted, but then > continuing to boot produced what you see in the video. > > I think, since it's in the cetner of the screen, that we're looking at 80x25 > text... which is ... 800 x 600 -ish? The kernel definately wants graphics > again ... but ... I dunno ... is it getting it? Is the 80x25 text mode > emulated on a bitmapped screen? > > > On Mon, Apr 2, 2018 at 9:07 PM, Kyle Evans <kevans@freebsd.org> wrote: >> >> On Mon, Apr 2, 2018 at 5:51 PM, Zaphod Beeblebrox <zbeeble@gmail.com> >> wrote: >> > On Sun, Apr 1, 2018 at 7:41 PM, David NewHamlet <wheelcomplex@gmail.com> >> > wrote: >> >> >> >> Fixed in: FreeBSD-12.0-CURRENT-amd64-20180329-r331740-disc1.iso >> >> >> >> >> >> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=176694 >> >> >> >> On Mon, Apr 2, 2018 at 12:38 AM, Tomoaki AOKI >> >> <junchoon@dec.sakura.ne.jp> >> >> wrote: >> >> >> >> > Confirmed both loader (with boot1) part and efirt.ko part. >> >> > Working OK on my ThinkPad420 (with nvidia GPU) at r331864. >> >> > >> >> > No benefit (VGA resolution) but no new harm (no panic nor silent >> >> > reboot). >> >> > >> >> > *Maybe gracefully falling back to mode 0. >> >> > >> >> > As I'm on x11/nvidia-driver, completely no test is done with >> >> > drm-next. >> >> > >> >> > One more to mention. >> >> > I've cherry-picked r330868, r331241 and r331361 on stable/11 after >> >> > r331114, amd64 and works OK as on head. >> >> > Additional cherry-picking of r331365 is OK, too. >> >> > >> >> > Without r330868, my ThinkPad silently reboots within about 10-60 >> >> > minutes range, maybe by actual access to UEFI RS. >> >> > With r331241 without r331361 causes instant panic on loading >> >> > efirt.ko. >> >> > So all 3 (4) revs should be MFC'ed together. >> >> > >> >> > Sorry for delay. >> >> > >> >> > >> >> > On Thu, 22 Mar 2018 10:34:33 -0500 >> >> > Kyle Evans <kevans@freebsd.org> wrote: >> >> > >> >> > > On Thu, Mar 22, 2018 at 10:18 AM, Kyle Evans <kevans@freebsd.org> >> >> > > wrote: >> >> > > > On Thu, Mar 22, 2018 at 10:16 AM, Peter Lei <peter.lei@ieee.org> >> >> > wrote: >> >> > > >> On 3/22/18 8:56 AM, Tomoaki AOKI wrote: >> >> > > >>> Hi. >> >> > > >>> For problem 2, try reverting r331241 alone. >> >> > > >>> This worked for my ThinkPad T420. >> >> > > >> >> >> > > >> >> >> > > >> I also hit this after updating to latest and was about to post >> >> > > >> when >> >> > > >> I >> >> > > >> saw this thread - >> >> > > >> >> >> > > >> I build efirt into the kernel and it's now doing a panic on >> >> > > >> boot. >> >> > > >> It >> >> > > >> appears to be due to this recent addition in dev/efidev/efirt.c >> >> > > >> by >> >> > r331241: >> >> > > >> >> >> > > >>> if (!efi_is_in_map(map, efihdr->memory_size / >> >> > efihdr->descriptor_size, >> >> > > >>> efihdr->descriptor_size, >> >> > > >>> (vm_offset_t)efi_runtime->rt_gettime)) >> >> > { >> >> > > >> >> >> > > >> The faulting address is for "efi_runtime->rt_gettime" (and is >> >> > > >> still >> >> > > >> a >> >> > > >> phys addr here). >> >> > > >> >> >> > > > >> >> > > > The following patch [1] (I can't guarantee how long he'll keep it >> >> > > > available there) should fix this class of problems. >> >> > > > >> >> > > > [1] https://people.freebsd.org/~andrew/0001-Enter-into-the- >> >> > EFI-environment-before-check-the-GetT.patch >> >> > > >> >> > > Now committed as of r331361. >> >> > > _______________________________________________ >> >> > > freebsd-current@freebsd.org mailing list >> >> > > https://lists.freebsd.org/mailman/listinfo/freebsd-current >> >> > > To unsubscribe, send any mail to "freebsd-current-unsubscribe@ >> >> > freebsd.org" >> >> > > >> >> > >> >> > >> >> > -- >> >> > Tomoaki AOKI <junchoon@dec.sakura.ne.jp> >> >> > _______________________________________________ >> >> > freebsd-current@freebsd.org mailing list >> >> > https://lists.freebsd.org/mailman/listinfo/freebsd-current >> >> > To unsubscribe, send any mail to >> >> > "freebsd-current-unsubscribe@freebsd.org" >> >> > >> >> _______________________________________________ >> >> freebsd-current@freebsd.org mailing list >> >> https://lists.freebsd.org/mailman/listinfo/freebsd-current >> >> To unsubscribe, send any mail to >> >> "freebsd-current-unsubscribe@freebsd.org" >> > >> > I've booted that image on my zbook 15. I show in the boot that I can >> > deliberately load efirt.ko ... and it doesn't help. I also show that I >> > can >> > "type blind" after the system boots ... so everything but the screen is >> > working. >> > >> > In case you can't quite make it out, I hit right cursor twice (move to >> > the >> > "live cd" choice) and hit enter. Then I type "root" enter and then >> > "reboot" >> > ... >> >> That is interesting indeed... that's perhaps the polar opposite of >> what was being experienced before- it looks like it's setting a higher >> resolution but the firmware isn't actually putting it into the higher >> resolution. The firmware then lies to us about what resolution it's >> actually in, so we tell the kernel the wrong thing. >> >> You should be able to confirm this, as I recall, by bumping into the >> loader prompt and inspecting the output of `gop get`. That should >> report a higher resolution than it's actually in. >> >> > https://youtu.be/tlmaVJ-3aq0 >> > >> > (The rock sound track is just free audio to mask a barking dog and a >> > radio >> > in the background.) >> > >> >> This was a most enjoyable soundtrack. > >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CACNAnaHSpicqRzH=YAZWxVqSh6HnPSa=FBeTJxoZZOXRrS5NyQ>