Arthur Heymans uploaded patch set #6 to this change.

View Change

[RFC]cpu/x86/smm: Add a unified way of handling save_states

This adds common code to handle SMM save states based on the SMM
revision. Runtime detection is done because on some systems different
SMM save states can occur depending on the installed CPU.

example:
SMM code needing access to eax from save state would do the following

struct smm_save_state_ops ops = get_save_state_ops();
uint64_t rax;
int node;

for (node = 0; node < CONFIG_MAX_CPUS; node++) {
if (ops.get_reg(node, RAX, &rax))
error...
rax &= MAX_UINT32;
do_stuff...
}

Change-Id: Ide7ccc44dbcc864e70463ef318dc1858b51183dc
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
---
M src/cpu/x86/smm/Makefile.inc
M src/cpu/x86/smm/smihandler.c
M src/cpu/x86/smm/smm_module_handler.c
A src/cpu/x86/smm/smm_save_state.c
M src/include/cpu/x86/smm.h
5 files changed, 309 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/61/36661/6

To view, visit change 36661. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ide7ccc44dbcc864e70463ef318dc1858b51183dc
Gerrit-Change-Number: 36661
Gerrit-PatchSet: 6
Gerrit-Owner: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Patrick Rudolph <siro@das-labor.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: newpatchset