Page MenuHomeFreeBSD

amd64: do not touch BIOS reset flag halfword, unless we boot through BIOS
ClosedPublic

Authored by kib on Jun 22 2021, 11:46 PM.
Tags
None
Referenced Files
F102815861: D30872.diff
Sun, Nov 17, 1:37 PM
Unknown Object (File)
Sat, Nov 2, 8:49 PM
Unknown Object (File)
Fri, Nov 1, 6:18 PM
Unknown Object (File)
Thu, Oct 24, 7:50 PM
Unknown Object (File)
Sep 23 2024, 6:36 PM
Unknown Object (File)
Sep 23 2024, 6:36 PM
Unknown Object (File)
Sep 23 2024, 6:36 PM
Unknown Object (File)
Sep 23 2024, 6:36 PM
Subscribers
None

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib requested review of this revision.Jun 22 2021, 11:46 PM
kib created this revision.
markj added inline comments.
sys/amd64/amd64/machdep.c
1183

I think it would be better to explicitly cast the address, i.e., (unsigned short *)0x472.

It seems like a strange place to do this operation. hammer_time() already checks for MODINFOMD_EFI_MAP to see if vty_set_preferred() should be called; we could do the same check earlier in order to set the BIOS flag, and perhaps save the result to avoid a second preload_search_info() call.

This revision is now accepted and ready to land.Jun 23 2021, 1:10 PM

Move the write to hammer_time()

This revision now requires review to proceed.Jun 23 2021, 8:01 PM
This revision is now accepted and ready to land.Jun 23 2021, 8:29 PM