Attention is currently required from: Angel Pons, Arthur Heymans, Lean Sheng Tan, Shuo Liu.
Patrick Rudolph has posted comments on this change by Shuo Liu. ( https://review.coreboot.org/c/coreboot/+/82705?usp=email )
Change subject: util/cbfstool: Disable paging for linux payload
......................................................................
Patch Set 2: Code-Review-2
(1 comment)
Patchset:
PS2:
Since coreboot doesn't set up paging, it must not disable paging. It cannot know if the current page executing code from and if (the stack) or heap is identity mapped. Blindly disabling paging is simply not possible.
@Intel After two years that you aware of the issue, please how about FIXING YOUR CODE instead?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/82705?usp=email
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I50a4ab2664910e5772b90b3dade8866f28361d87
Gerrit-Change-Number: 82705
Gerrit-PatchSet: 2
Gerrit-Owner: Shuo Liu
shuo.liu@intel.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Lean Sheng Tan
sheng.tan@9elements.com
Gerrit-Reviewer: Patrick Rudolph
patrick.rudolph@9elements.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Nico Huber
nico.h@gmx.de
Gerrit-Attention: Shuo Liu
shuo.liu@intel.com
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Attention: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Attention: Lean Sheng Tan
sheng.tan@9elements.com
Gerrit-Comment-Date: Thu, 30 May 2024 15:33:39 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes