diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2021-02-15 16:43:19 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2021-03-19 11:30:47 +0000 |
| commit | 50e849fc9b09773b2647bd074e9f08e1a5f028a3 (patch) | |
| tree | 87ddc19ae05f63994bcb07519b442868da8dc383 /src/include | |
| parent | 6f9b1a904923cab9905bd6521738e7b6617cf3c5 (diff) | |
| download | coreboot-50e849fc9b09773b2647bd074e9f08e1a5f028a3.tar.xz | |
cpu/x86/smm_stub.S: Drop smm_runtime from the stub
The parameters that the permanent handler requires are pushed directly
to the permanent handlers relocatable module params.
The paremeters that the relocation handler requires are not passed on
via arguments but are copied inside the ramstage. This is ok as the
relocation handler calls into ramstage.
Change-Id: Ice311d05e2eb0e95122312511d83683d7f0dee58
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50767
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'src/include')
| -rw-r--r-- | src/include/cpu/x86/smm.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/include/cpu/x86/smm.h b/src/include/cpu/x86/smm.h index 8b92e417e8..cb9aeac813 100644 --- a/src/include/cpu/x86/smm.h +++ b/src/include/cpu/x86/smm.h @@ -64,7 +64,6 @@ struct smm_runtime { struct smm_module_params { size_t cpu; - const struct smm_runtime *runtime; /* A canary value that has been placed at the end of the stack. * If (uintptr_t)canary != *canary then a stack overflow has occurred. */ @@ -88,7 +87,6 @@ struct smm_stub_params { u8 apic_id_to_cpu[CONFIG_MAX_CPUS]; /* STM's 32bit entry into SMI handler */ u32 start32_offset; - struct smm_runtime runtime; } __packed; /* smm_handler_t is called with arg of smm_module_params pointer. */ |