Skip to content

Commit

Permalink
x86/microcode/intel: Fish out the stashed microcode for the BSP
Browse files Browse the repository at this point in the history
I'm such a moron! The simple solution of saving the BSP patch
for use on resume was too simple (and wrong!), hint:
sizeof(struct microcode_intel).

What needs to be done instead is to fish out the microcode patch
we have stashed previously and apply that on the BSP in case the
late loader hasn't been utilized.

So do that instead.

Signed-off-by: Borislav Petkov <bp@suse.de>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Link: http://lkml.kernel.org/r/20141208110820.GB20057@pd.tnic
Signed-off-by: Ingo Molnar <mingo@kernel.org>
  • Loading branch information
Borislav Petkov authored and Ingo Molnar committed Dec 10, 2014
1 parent e913b8c commit 25cdb9c
Showing 1 changed file with 12 additions and 14 deletions.
26 changes: 12 additions & 14 deletions arch/x86/kernel/cpu/microcode/intel_early.c
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,6 @@ static struct mc_saved_data {
struct microcode_intel **mc_saved;
} mc_saved_data;

static struct microcode_intel bsp_patch;

static enum ucode_state
generic_load_microcode_early(struct microcode_intel **mc_saved_p,
unsigned int mc_saved_count,
Expand Down Expand Up @@ -717,8 +715,7 @@ _load_ucode_intel_bsp(struct mc_saved_data *mc_saved_data,
unsigned long *mc_saved_in_initrd,
unsigned long initrd_start_early,
unsigned long initrd_end_early,
struct ucode_cpu_info *uci,
struct microcode_intel *bsp)
struct ucode_cpu_info *uci)
{
enum ucode_state ret;

Expand All @@ -729,10 +726,8 @@ _load_ucode_intel_bsp(struct mc_saved_data *mc_saved_data,
ret = load_microcode(mc_saved_data, mc_saved_in_initrd,
initrd_start_early, uci);

if (ret == UCODE_OK) {
if (ret == UCODE_OK)
apply_microcode_early(uci, true);
memcpy(bsp, uci->mc, sizeof(*bsp));
}
}

void __init
Expand All @@ -741,12 +736,10 @@ load_ucode_intel_bsp(void)
u64 ramdisk_image, ramdisk_size;
unsigned long initrd_start_early, initrd_end_early;
struct ucode_cpu_info uci;
struct microcode_intel *bsp_p;
#ifdef CONFIG_X86_32
struct boot_params *boot_params_p;

boot_params_p = (struct boot_params *)__pa_nodebug(&boot_params);
bsp_p = (struct microcode_intel *)__pa_nodebug(&bsp_patch);
ramdisk_image = boot_params_p->hdr.ramdisk_image;
ramdisk_size = boot_params_p->hdr.ramdisk_size;
initrd_start_early = ramdisk_image;
Expand All @@ -755,17 +748,16 @@ load_ucode_intel_bsp(void)
_load_ucode_intel_bsp(
(struct mc_saved_data *)__pa_nodebug(&mc_saved_data),
(unsigned long *)__pa_nodebug(&mc_saved_in_initrd),
initrd_start_early, initrd_end_early, &uci, bsp_p);
initrd_start_early, initrd_end_early, &uci);
#else
bsp_p = &bsp_patch;
ramdisk_image = boot_params.hdr.ramdisk_image;
ramdisk_size = boot_params.hdr.ramdisk_size;
initrd_start_early = ramdisk_image + PAGE_OFFSET;
initrd_end_early = initrd_start_early + ramdisk_size;

_load_ucode_intel_bsp(&mc_saved_data, mc_saved_in_initrd,
initrd_start_early, initrd_end_early,
&uci, bsp_p);
&uci);
#endif
}

Expand Down Expand Up @@ -805,11 +797,17 @@ void load_ucode_intel_ap(void)
void reload_ucode_intel(void)
{
struct ucode_cpu_info uci;
enum ucode_state ret;

if (!bsp_patch.hdr.rev)
if (!mc_saved_data.mc_saved_count)
return;

uci.mc = &bsp_patch;
collect_cpu_info_early(&uci);

ret = generic_load_microcode_early(mc_saved_data.mc_saved,
mc_saved_data.mc_saved_count, &uci);
if (ret != UCODE_OK)
return;

apply_microcode_early(&uci, false);
}

0 comments on commit 25cdb9c

Please sign in to comment.