Attention is currently required from: Benjamin Doron, Jérémy Compostella.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80336?usp=email )
Change subject: cpu/x86/smm: Set up page tables in safe SMRAM
......................................................................
Patch Set 3:
(1 comment)
Patchset:
PS3:
Okay, this approach is simpler than CB:79865.
That patch train has some more sophisticated code to generate page tables in both ramstage and smm. I like it better, as it seems better suited to use NX. Maybe this simple approach could be merged first and then revisited when the mmu runtime code is ready?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/80336?usp=email
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Icb3086abd577b9abb9966dd910a264a873ace4ed
Gerrit-Change-Number: 80336
Gerrit-PatchSet: 3
Gerrit-Owner: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Benjamin Doron
benjamin.doron00@gmail.com
Gerrit-Reviewer: Jérémy Compostella
jeremy.compostella@intel.com
Gerrit-Reviewer: Patrick Rudolph
patrick.rudolph@9elements.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Jérémy Compostella
jeremy.compostella@intel.com
Gerrit-Attention: Benjamin Doron
benjamin.doron00@gmail.com
Gerrit-Comment-Date: Tue, 06 Feb 2024 07:56:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Benjamin Doron
benjamin.doron00@gmail.com
Gerrit-MessageType: comment