summaryrefslogtreecommitdiff
path: root/src/cpu
diff options
context:
space:
mode:
Diffstat (limited to 'src/cpu')
-rw-r--r--src/cpu/x86/mp_init.c23
-rw-r--r--src/cpu/x86/smm/smm_module_loader.c89
2 files changed, 39 insertions, 73 deletions
diff --git a/src/cpu/x86/mp_init.c b/src/cpu/x86/mp_init.c
index fee5940b18..7d0a20fa70 100644
--- a/src/cpu/x86/mp_init.c
+++ b/src/cpu/x86/mp_init.c
@@ -760,10 +760,9 @@ static void adjust_smm_apic_id_map(struct smm_loader_params *smm_params)
}
static enum cb_err install_relocation_handler(int num_cpus, size_t real_save_state_size,
- size_t save_state_size, uintptr_t perm_smbase)
+ size_t save_state_size)
{
struct smm_loader_params smm_params = {
- .per_cpu_stack_size = CONFIG_SMM_STUB_STACK_SIZE,
.num_concurrent_stacks = num_cpus,
.real_cpu_save_state_size = real_save_state_size,
.per_cpu_save_state_size = save_state_size,
@@ -771,11 +770,7 @@ static enum cb_err install_relocation_handler(int num_cpus, size_t real_save_sta
.handler = smm_do_relocation,
};
- /* Allow callback to override parameters. */
- if (mp_state.ops.adjust_smm_params != NULL)
- mp_state.ops.adjust_smm_params(&smm_params, 0);
-
- if (smm_setup_relocation_handler(perm_smbase, &smm_params)) {
+ if (smm_setup_relocation_handler(&smm_params)) {
printk(BIOS_ERR, "%s: smm setup failed\n", __func__);
return CB_ERR;
}
@@ -798,17 +793,12 @@ static enum cb_err install_permanent_handler(int num_cpus, uintptr_t smbase,
* size and save state size for each CPU.
*/
struct smm_loader_params smm_params = {
- .per_cpu_stack_size = CONFIG_SMM_MODULE_STACK_SIZE,
.num_concurrent_stacks = num_cpus,
.real_cpu_save_state_size = real_save_state_size,
.per_cpu_save_state_size = save_state_size,
.num_concurrent_save_states = num_cpus,
};
- /* Allow callback to override parameters. */
- if (mp_state.ops.adjust_smm_params != NULL)
- mp_state.ops.adjust_smm_params(&smm_params, 1);
-
printk(BIOS_DEBUG, "Installing permanent SMM handler to 0x%08lx\n", smbase);
if (smm_load_module(smbase, smsize, &smm_params))
@@ -829,10 +819,15 @@ static void load_smm_handlers(void)
if (!is_smm_enabled())
return;
+ if (smm_setup_stack(mp_state.perm_smbase, mp_state.perm_smsize, mp_state.cpu_count,
+ CONFIG_SMM_MODULE_STACK_SIZE)) {
+ printk(BIOS_ERR, "Unable to install SMM relocation handler.\n");
+ smm_disable();
+ }
+
/* Install handlers. */
if (install_relocation_handler(mp_state.cpu_count, real_save_state_size,
- smm_save_state_size, mp_state.perm_smbase) !=
- CB_SUCCESS) {
+ smm_save_state_size) != CB_SUCCESS) {
printk(BIOS_ERR, "Unable to install SMM relocation handler.\n");
smm_disable();
}
diff --git a/src/cpu/x86/smm/smm_module_loader.c b/src/cpu/x86/smm/smm_module_loader.c
index ba179ca8b7..ce7cf20272 100644
--- a/src/cpu/x86/smm/smm_module_loader.c
+++ b/src/cpu/x86/smm/smm_module_loader.c
@@ -1,6 +1,7 @@
/* SPDX-License-Identifier: GPL-2.0-only */
#include <acpi/acpi_gnvs.h>
+#include <stddef.h>
#include <stdint.h>
#include <string.h>
#include <rmodule.h>
@@ -239,32 +240,26 @@ static int smm_place_entry_code(uintptr_t smbase, unsigned int num_cpus,
return 1;
}
-/*
- * Place stacks in base -> base + size region, but ensure the stacks don't
- * overlap the staggered entry points.
- */
-static uintptr_t smm_stub_place_stacks(const uintptr_t base, struct smm_loader_params *params)
-{
- size_t total_stack_size;
- uintptr_t stacks_top;
-
- /* If stack space is requested assume the space lives in the lower
- * half of SMRAM. */
- total_stack_size = params->per_cpu_stack_size *
- params->num_concurrent_stacks;
- printk(BIOS_DEBUG, "%s: cpus: %zx : stack space: needed -> %zx\n",
- __func__, params->num_concurrent_stacks,
- total_stack_size);
-
- /* There has to be at least one stack user. */
- if (params->num_concurrent_stacks < 1)
- return 0;
+static uintptr_t stack_top;
+static size_t g_stack_size;
- /* Stacks extend down to SMBASE */
- stacks_top = base + total_stack_size;
- printk(BIOS_DEBUG, "%s: exit, stack_top %lx\n", __func__, stacks_top);
+int smm_setup_stack(const uintptr_t perm_smbase, const size_t perm_smram_size,
+ const unsigned int total_cpus, const size_t stack_size)
+{
+ /* Need a minimum stack size and alignment. */
+ if (stack_size <= SMM_MINIMUM_STACK_SIZE || (stack_size & 3) != 0) {
+ printk(BIOS_ERR, "%s: need minimum stack size\n", __func__);
+ return -1;
+ }
- return stacks_top;
+ const size_t total_stack_size = total_cpus * stack_size;
+ if (total_stack_size >= perm_smram_size) {
+ printk(BIOS_ERR, "%s: Stack won't fit smram\n", __func__);
+ return -1;
+ }
+ stack_top = perm_smbase + total_stack_size;
+ g_stack_size = stack_size;
+ return 0;
}
/*
@@ -284,7 +279,7 @@ static int smm_stub_place_staggered_entry_points(char *base,
if (params->num_concurrent_save_states > 1 || stub_entry_offset != 0) {
rc = smm_place_entry_code((uintptr_t)base,
params->num_concurrent_save_states,
- (uintptr_t)params->stack_top, params);
+ stack_top, params);
}
return rc;
}
@@ -309,18 +304,15 @@ static int smm_stub_place_staggered_entry_points(char *base,
* permanent SMM handler.
* The CPU stack is decided at runtime in the stub and is treaded as a continuous
* region. As this might not fit the default SMRAM region, the same region used
- * by the permanent handler can be used during relocation. This is done via the
- * smram_start argument.
+ * by the permanent handler can be used during relocation.
*/
static int smm_module_setup_stub(const uintptr_t smbase, const size_t smm_size,
struct smm_loader_params *params,
- void *const fxsave_area,
- const uintptr_t smram_start)
+ void *const fxsave_area)
{
size_t total_save_state_size;
size_t smm_stub_size;
uintptr_t smm_stub_loc;
- uintptr_t stacks_top;
size_t size;
uintptr_t base;
size_t i;
@@ -361,13 +353,6 @@ static int smm_module_setup_stub(const uintptr_t smbase, const size_t smm_size,
return -1;
}
- /* Need a minimum stack size and alignment. */
- if (params->per_cpu_stack_size <= SMM_MINIMUM_STACK_SIZE ||
- (params->per_cpu_stack_size & 3) != 0) {
- printk(BIOS_ERR, "%s: need minimum stack size\n", __func__);
- return -1;
- }
-
smm_stub_size = rmodule_memory_size(&smm_stub);
/* Put the stub at the main entry point */
@@ -379,16 +364,10 @@ static int smm_module_setup_stub(const uintptr_t smbase, const size_t smm_size,
return -1;
}
- /* The stacks, if requested, live in the lower half of SMRAM space
- * for default handler, but for relocated handler it lives at the beginning
- * of SMRAM which is TSEG base
- */
- stacks_top = smm_stub_place_stacks(smram_start, params);
- if (stacks_top == 0) {
+ if (stack_top == 0) {
printk(BIOS_ERR, "%s: error assigning stacks\n", __func__);
return -1;
}
- params->stack_top = stacks_top;
/* Load the stub. */
if (rmodule_load((void *)smm_stub_loc, &smm_stub)) {
printk(BIOS_ERR, "%s: load module failed\n", __func__);
@@ -402,19 +381,15 @@ static int smm_module_setup_stub(const uintptr_t smbase, const size_t smm_size,
/* Setup the parameters for the stub code. */
stub_params = rmodule_parameters(&smm_stub);
- stub_params->stack_top = (uintptr_t)stacks_top;
- stub_params->stack_size = params->per_cpu_stack_size;
+ stub_params->stack_top = stack_top;
+ stub_params->stack_size = g_stack_size;
stub_params->c_handler = (uintptr_t)params->handler;
stub_params->fxsave_area = (uintptr_t)fxsave_area;
stub_params->fxsave_area_size = FXSAVE_SIZE;
- const size_t total_stack_size =
- params->num_concurrent_stacks * params->per_cpu_stack_size;
- printk(BIOS_DEBUG, "%s: stack_end = 0x%zx\n",
- __func__, stub_params->stack_top - total_stack_size);
printk(BIOS_DEBUG,
"%s: stack_top = 0x%x\n", __func__, stub_params->stack_top);
- printk(BIOS_DEBUG, "%s: stack_size = 0x%x\n",
+ printk(BIOS_DEBUG, "%s: per cpu stack_size = 0x%x\n",
__func__, stub_params->stack_size);
printk(BIOS_DEBUG, "%s: runtime.start32_offset = 0x%x\n", __func__,
stub_params->start32_offset);
@@ -439,7 +414,7 @@ static int smm_module_setup_stub(const uintptr_t smbase, const size_t smm_size,
* assumption is that the stub will be entered from the default SMRAM
* location: 0x30000 -> 0x40000.
*/
-int smm_setup_relocation_handler(const uintptr_t perm_smram, struct smm_loader_params *params)
+int smm_setup_relocation_handler(struct smm_loader_params *params)
{
uintptr_t smram = SMM_DEFAULT_BASE;
printk(BIOS_SPEW, "%s: enter\n", __func__);
@@ -459,7 +434,7 @@ int smm_setup_relocation_handler(const uintptr_t perm_smram, struct smm_loader_
printk(BIOS_SPEW, "%s: exit\n", __func__);
return smm_module_setup_stub(smram, SMM_DEFAULT_SIZE,
- params, fxsave_area_relocation, perm_smram);
+ params, fxsave_area_relocation);
}
/*
@@ -519,11 +494,9 @@ int smm_load_module(const uintptr_t smram_base, const size_t smram_size,
if (CONFIG(DEBUG_SMI))
memset((void *)smram_base, 0xcd, smram_size);
- total_stack_size = params->per_cpu_stack_size *
- params->num_concurrent_stacks;
+ total_stack_size = stack_top - smram_base;
total_size += total_stack_size;
/* Stacks are the base of SMRAM */
- params->stack_top = smram_base + total_stack_size;
/* MSEG starts at the top of SMRAM and works down */
if (CONFIG(STM)) {
@@ -580,8 +553,6 @@ int smm_load_module(const uintptr_t smram_base, const size_t smram_size,
printk(BIOS_DEBUG, "%s: smram_start: 0x%lx\n", __func__, smram_base);
printk(BIOS_DEBUG, "%s: smram_end: %lx\n", __func__, smram_base + smram_size);
- printk(BIOS_DEBUG, "%s: stack_top: %lx\n",
- __func__, params->stack_top);
printk(BIOS_DEBUG, "%s: handler start %p\n",
__func__, params->handler);
printk(BIOS_DEBUG, "%s: handler_size %zx\n",
@@ -619,5 +590,5 @@ int smm_load_module(const uintptr_t smram_base, const size_t smram_size,
cpus[i].ss_start + params->per_cpu_save_state_size;
}
- return smm_module_setup_stub(base, smram_size, params, fxsave_area, smram_base);
+ return smm_module_setup_stub(base, smram_size, params, fxsave_area);
}
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-23 22:03:16 +0000