diff options
| author | Aaron Durbin <adurbin@chromium.org> | 2017-09-07 21:17:33 -0600 |
|---|---|---|
| committer | Aaron Durbin <adurbin@chromium.org> | 2017-09-11 01:17:45 +0000 |
| commit | 5a1f9a87cb2f6a3e0699f3d13db41c7da2963b88 (patch) | |
| tree | 516fa2ca773920f67411c1b0560c449f8a529fb7 /src/soc/intel/cannonlake | |
| parent | 0b80bd1cf4510b3a4f028b5bb5570a2806c22320 (diff) | |
| download | coreboot-5a1f9a87cb2f6a3e0699f3d13db41c7da2963b88.tar.xz | |
cpu/x86/mp_init: remove adjust_cpu_apic_entry()
The original purpose of adjust_cpu_apic_entry() was to set
up an APIC map. That map was effectively only used for mapping
*default* APIC id to CPU number in the SMM handler. The normal
AP startup path didn't need this mapping because it was whoever
won the race got the next cpu number. Instead of statically
calculating (and wrong) just initialize the default APIC id
map when the APs come online. Once the APs are online the SMM
handler is loaded and the mapping is utilized.
Change-Id: Idff3b8cfc17aef0729d3193b4499116a013b7930
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/21452
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Diffstat (limited to 'src/soc/intel/cannonlake')
| -rw-r--r-- | src/soc/intel/cannonlake/cpu.c | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/src/soc/intel/cannonlake/cpu.c b/src/soc/intel/cannonlake/cpu.c index 2faadfd492..4e065778fa 100644 --- a/src/soc/intel/cannonlake/cpu.c +++ b/src/soc/intel/cannonlake/cpu.c @@ -200,16 +200,6 @@ void soc_core_init(device_t cpu) } -static int adjust_apic_id(int index, int apic_id) -{ - unsigned int num_cores, num_threads; - - if (cpu_read_topology(&num_cores, &num_threads)) - return 2 * index; - else - return index; -} - static void post_mp_init(void) { /* Set Max Ratio */ @@ -225,7 +215,6 @@ static const struct mp_ops mp_ops = { .pre_mp_init = soc_fsp_load, .get_cpu_count = get_cpu_count, .get_microcode_info = get_microcode_info, - .adjust_cpu_apic_entry = adjust_apic_id, .post_mp_init = post_mp_init, }; |