Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30118 )
Change subject: arch/x86/boot: Call payload in protected mode ......................................................................
Patch Set 23:
(2 comments)
https://review.coreboot.org/c/coreboot/+/30118/23/src/arch/x86/boot.c File src/arch/x86/boot.c:
https://review.coreboot.org/c/coreboot/+/30118/23/src/arch/x86/boot.c@34 PS23, Line 34: NV_RAMSTAGE : protected_mode_call I think it is worth explaining why payloads are run in protected_mode here.
https://review.coreboot.org/c/coreboot/+/30118/23/src/arch/x86/c_start.S File src/arch/x86/c_start.S:
https://review.coreboot.org/c/coreboot/+/30118/23/src/arch/x86/c_start.S@257 PS23, Line 257: protected_mode_call Just wondering if it makes sense to put this in a separate file?