diff options
author | Jan Beulich <jbeulich@novell.com> | 2011-10-24 18:00:20 +0100 |
---|---|---|
committer | Jan Beulich <jbeulich@novell.com> | 2011-10-24 18:00:20 +0100 |
commit | a16d59ea4b1b1463ac110e986a3c4e7c983a2544 (patch) | |
tree | c4bfb052004f28c89e2dbd10a18335905c78dc10 /xen/arch/x86/microcode_amd.c | |
parent | d740d811925385c09553cbe6dee8e77c1d43b198 (diff) | |
download | xen-a16d59ea4b1b1463ac110e986a3c4e7c983a2544.tar.gz xen-a16d59ea4b1b1463ac110e986a3c4e7c983a2544.tar.bz2 xen-a16d59ea4b1b1463ac110e986a3c4e7c983a2544.zip |
x86/ucode-amd: fix regression from c/s 23871:503ee256fecf
microcode_fits() must return distinct values for the success and
no-fit-but-no-error cases, so the caller can react accordingly. Make
it return 1 in the success case, and adjust its single caller.
Also remove an impossible code path - install_equiv_cpu_table(), which
gets called prior to microcode_fits(), never leaves equiv_cpu_table
being NULL without also returning an error.
Note that this is still awaiting testing on a system where the
regression was actually observed (which also requires a new enough
microcode_ctl package). Note also that this will need to be
backported to 4.0 and 4.1 (or the broken c/s that got backported
there reverted).
Signed-off-by: Jan Beulich <jbeulich@novell.com>
Committed-by: Keir Fraser <keir@xen.org>
Diffstat (limited to 'xen/arch/x86/microcode_amd.c')
-rw-r--r-- | xen/arch/x86/microcode_amd.c | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/xen/arch/x86/microcode_amd.c b/xen/arch/x86/microcode_amd.c index 0aa90f88da..92dc124c7e 100644 --- a/xen/arch/x86/microcode_amd.c +++ b/xen/arch/x86/microcode_amd.c @@ -76,14 +76,6 @@ static int microcode_fits(void *mc, int cpu) /* We should bind the task to the CPU */ BUG_ON(cpu != raw_smp_processor_id()); - if ( equiv_cpu_table == NULL ) - { - printk(KERN_INFO "microcode: CPU%d microcode update with " - "version 0x%x (current=0x%x)\n", - cpu, mc_header->patch_id, uci->cpu_sig.rev); - goto out; - } - current_cpu_id = cpuid_eax(0x00000001); for ( i = 0; equiv_cpu_table[i].installed_cpu != 0; i++ ) @@ -96,7 +88,7 @@ static int microcode_fits(void *mc, int cpu) } if ( !equiv_cpu_id ) - return 0; + return 0; if ( (mc_header->processor_rev_id) != equiv_cpu_id ) { @@ -113,8 +105,7 @@ static int microcode_fits(void *mc, int cpu) "update with version 0x%x (current=0x%x)\n", cpu, mc_header->patch_id, uci->cpu_sig.rev); -out: - return 0; + return 1; } static int apply_microcode(int cpu) @@ -285,7 +276,7 @@ static int cpu_request_microcode(int cpu, const void *buf, size_t size) while ( (ret = get_next_ucode_from_buffer_amd(mc, buf, size, &offset)) == 0) { error = microcode_fits(mc, cpu); - if (error != 0) + if (error <= 0) continue; error = apply_microcode(cpu); |