Attention is currently required from: Paul Menzel, Patrick Rudolph. Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56017 )
Change subject: [WIP]Documentation: Improve x86_64 ......................................................................
Patch Set 2:
(7 comments)
File Documentation/arch/x86/x86_64.md:
https://review.coreboot.org/c/coreboot/+/56017/comment/7baf9ce2_81801d0a PS1, Line 4: build
built
Done
https://review.coreboot.org/c/coreboot/+/56017/comment/df057fe3_103f5599 PS1, Line 5: UEFIs implementation
Is that the specification? If not: […]
Done
https://review.coreboot.org/c/coreboot/+/56017/comment/963ebfc1_d4eab8d0 PS1, Line 5: UEFIs
UEFI’s
Done
https://review.coreboot.org/c/coreboot/+/56017/comment/253ba1bd_aa9dd50a PS1, Line 5: only some
only runs(?) some
Done
https://review.coreboot.org/c/coreboot/+/56017/comment/4c7416c2_29ab5bd8 PS1, Line 9: following
below
Done
https://review.coreboot.org/c/coreboot/+/56017/comment/615f32d3_dd1c5239 PS1, Line 22: * The BIOS region is large enough to contain the page tables
“BIOS region” refers to Intel hardware, right?
I don't know the name on AMD hardware. Is there a way to describe it platform independent?
https://review.coreboot.org/c/coreboot/+/56017/comment/255682b1_596f7dc1 PS1, Line 48: contains
contain
Done