Patrick Rudolph uploaded patch set #2 to this change.

View Change

cpu/x86/mpinit/sipi_vector: Move code to C

Use a C helper function to do early x86 CPU init. This allows easier
code changes that are hopefully less error prone than assembly code.
This changes assumes that stack works even without microcode updates
installed on the specific secondary CPU core.

The code is compiled without SSE and without optimizations.

Use existing code to update microcode.

Tested on Supermicro X11SSH:
* The microcode is still updated on all core
* All cores call into MPinit code

Change-Id: Ib6c2ffa2ce8c42eee9f3fe8ec8cff933942207ca
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
---
M src/cpu/x86/Makefile.inc
M src/cpu/x86/mp_init.c
M src/cpu/x86/sipi_vector.S
A src/cpu/x86/sipi_vector_c_handler.c
M src/include/cpu/x86/mp.h
5 files changed, 150 insertions(+), 124 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/21/46821/2

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib6c2ffa2ce8c42eee9f3fe8ec8cff933942207ca
Gerrit-Change-Number: 46821
Gerrit-PatchSet: 2
Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Christian Walter <christian.walter@9elements.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: newpatchset