diff options
author | Leah Rowe <leah@libreboot.org> | 2024-06-14 14:01:42 +0100 |
---|---|---|
committer | Leah Rowe <leah@libreboot.org> | 2024-06-14 14:51:27 +0100 |
commit | a4328cb11f927c3cceb01dcaf4d7fc7632076119 (patch) | |
tree | 0ccea46a0bb4f9d667cded1a2316c1e6d59de22a /config/coreboot/dell/patches/0002-fix-speedstep-on-x200-t400-Revert-cpu-intel-model_10.patch | |
parent | 167e1a0fbe6765aaa5f5d3dc682709aa3c31260b (diff) |
roms: remove dangerous runtime p/s/d options
p = payload
s = grub_scan_disk
d = displaymode
setting the payload is no longer safe, due to issue 216
and similar issues that might pop up in the future; it's
best left only to target.cfg, per board, so that we know
what config is safe/tested. don't let the user override it.
scandisk isn't safe to override because the given machine
may not have the type of device that the user specifies
displaymode is actually ok to set, because it simply whitelists
what configs pre-existing to actually use, but it's bloat
basically, the rule is this:
don't make it easy for the user to brick their hardware.
make it harder instead.
a user wily enough to go modifying their payload will probably
have read docs/maintain/ anyway and knows how to edit target.cfg
if they want another board configuration.
Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'config/coreboot/dell/patches/0002-fix-speedstep-on-x200-t400-Revert-cpu-intel-model_10.patch')
0 files changed, 0 insertions, 0 deletions