TrenchBoot / landing-zone

An open source implementation of an AMD-V Secure Loader.
GNU General Public License v2.0
23 stars 7 forks source link

lz_main: Don't opencode boot_param accesses #39

Closed andyhhp closed 4 years ago

andyhhp commented 4 years ago

Introduce struct boot_params and rewrite lz_main() with more normal looking C, dropping a load of local variables in the proces.

No functional change. Compiled code is identical (64, 64 LTO and 32), other than the 32bit LTO case which adds one redundant reload of a stack local variable for no obvious reason.

Signed-off-by: Andrew Cooper andrew.cooper3@citrix.com

Slightly RFC. Is "boot_params" the correct name here? zero_page appears to be a legacy name.