x86/ucode: fix Intel case of resume handling on boot CPU
authorJan Beulich <jbeulich@suse.com>
Fri, 28 Sep 2012 07:28:11 +0000 (09:28 +0200)
committerJan Beulich <jbeulich@suse.com>
Fri, 28 Sep 2012 07:28:11 +0000 (09:28 +0200)
commit0e82871ca3222f91ffe8b96768ece1b89ce96d12
tree64929f5c6a36e50efc8e79ce5c14e0eff6273cd6
parent24c152c7fec0cf0d788500482cfed09c4a0c527d
x86/ucode: fix Intel case of resume handling on boot CPU

Checking the stored version doesn't tell us anything about the need to
apply the update (during resume, what is stored doesn't necessarily
match what is loaded).

Note that the check can be removed altogether because once switched to
use what was read from the CPU (uci->cpu_sig.rev, as used in the
subsequent pr_debug()), it would become redundant with the checks that
lead to microcode_update_match() returning the indication that an
update should be applied.

Note further that this was not an issue on APs since they start with
uci->mc.mc_intel being NULL.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Tested-by: Ben Guthro <ben@guthro.net>
Acked-by: Keir Fraser <keir@xen.org>
xen/arch/x86/microcode_intel.c