linux.git/arch/x86/kernel/microcode_intel.c, branch v3.10

x86/microcode_intel.h: Define functions and macros for early loading ucode

2013-01-31T21:18:50+00:00

Define some functions and macros that will be used in early loading ucode. Some
of them are moved from microcode_intel.c driver in order to be called in early
boot phase before module can be called.

Signed-off-by: Fenghua Yu 
Link: http://lkml.kernel.org/r/1356075872-3054-3-git-send-email-fenghua.yu@intel.com
Signed-off-by: H. Peter Anvin

x86, microcode: Add a refresh firmware flag to ->request_microcode_fw

2012-08-22T23:15:58+00:00

This is done in preparation for teaching the ucode driver to either load
a new ucode patches container from userspace or use an already cached
version. No functionality change in this patch.

Signed-off-by: Borislav Petkov 
Link: http://lkml.kernel.org/r/1344361461-10076-10-git-send-email-bp@amd64.org
Signed-off-by: H. Peter Anvin

x86/microcode: Ensure that module is only loaded on supported Intel CPUs

2012-05-07T12:37:14+00:00

Exit early when there's no support for a particular CPU family.

Signed-off-by: Srivatsa S. Bhat 
Cc: Greg Kroah-Hartman 
Cc: Andreas Herrmann 
Cc: Kay Sievers 
Cc: Dave Jones 
Cc: tigran@aivazian.fsnet.co.uk
Cc: Borislav Petkov 
Link: http://lkml.kernel.org/r/4F8BDB58.6070007@linux.vnet.ibm.com
Signed-off-by: Ingo Molnar

x86, intel: Output microcode revision in /proc/cpuinfo

2011-10-14T11:16:35+00:00

I got a request to make it easier to determine the microcode
update level on Intel CPUs. This patch adds a new "microcode"
field to /proc/cpuinfo.

The microcode level is also outputed on fatal machine checks
together with the other CPUID model information.

I removed the respective code from the microcode update driver,
it just reads the field from cpu_data. Also when the microcode
is updated it fills in the new values too.

I had to add a memory barrier to native_cpuid to prevent it
being optimized away when the result is not used.

This turns out to clean up further code which already got this
information manually. This is done in followon patches.

Signed-off-by: Andi Kleen 
Acked-by: H. Peter Anvin 
Link: http://lkml.kernel.org/r/1318466795-7393-1-git-send-email-andi@firstfloor.org
Signed-off-by: Ingo Molnar

x86/microcode: Fix double vfree() and remove redundant pointer checks before vfree()

2010-12-27T13:33:30+00:00

In arch/x86/kernel/microcode_intel.c::generic_load_microcode()
we have  this:

	while (leftover) {
		...
		if (get_ucode_data(mc, ucode_ptr, mc_size) ||
		    microcode_sanity_check(mc) < 0) {
			vfree(mc);
			break;
		}
		...
	}

	if (mc)
		vfree(mc);

This will cause a double free of 'mc'. This patch fixes that by
just  removing the vfree() call in the loop since 'mc' will be
freed nicely just  after we break out of the loop.

There's also a second change in the patch. I noticed a lot of
checks for  pointers being NULL before passing them to vfree().
That's completely  redundant since vfree() deals gracefully with
being passed a NULL pointer.  Removing the redundant checks
yields a nice size decrease for the object  file.

Size before the patch:
   text    data     bss     dec     hex filename
   4578     240    1032    5850    16da arch/x86/kernel/microcode_intel.o
Size after the patch:
   text    data     bss     dec     hex filename
   4489     240     984    5713    1651 arch/x86/kernel/microcode_intel.o

Signed-off-by: Jesper Juhl 
Acked-by: Tigran Aivazian 
Cc: Shaohua Li 
LKML-Reference: 
Signed-off-by: Ingo Molnar

Update broken web addresses in arch directory.

2010-10-18T09:03:21+00:00

The patch below updates broken web addresses in the arch directory.

Signed-off-by: Justin P. Mattock 
Signed-off-by: Maciej W. Rozycki 
Cc: Finn Thain 
Cc: Randy Dunlap 
Reviewed-by: Finn Thain 
Signed-off-by: Jiri Kosina

x86: Improve Intel microcode loader performance

2010-03-11T12:49:06+00:00

We've noticed that on large SGI UV system configurations,
running microcode.ctl can take very long periods of time.  This
is due to the large number of vmalloc/vfree calls made by the
Intel generic_load_microcode() logic.

By reusing allocated space, the following patch reduces the time
to run microcode.ctl on a 1024 cpu system from approximately 80
seconds down to 1 or 2 seconds.

Signed-off-by: Dimitri Sivanich 
Acked-by: Dmitry Adamushko 
Cc: Avi Kivity 
Cc: Bill Davidsen 
LKML-Reference: <20100305174203.GA19638@sgi.com>
Signed-off-by: Ingo Molnar

x86: Remove trailing spaces in messages

2010-02-07T16:47:51+00:00

Signed-off-by: Frans Pop 
Cc: Avi Kivity 
Cc: x86@kernel.org
LKML-Reference: <1265478443-31072-10-git-send-email-elendil@planet.nl>
[ Left out the KVM bits. ]
Signed-off-by: Ingo Molnar

arch/x86/kernel/microcode*: Use pr_fmt() and remove duplicated KERN_ERR prefix

2009-12-09T07:25:57+00:00

- Use #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
- Remove "microcode: " prefix from each pr_
- Fix duplicated KERN_ERR prefix
- Coalesce pr_ format strings
- Add a space after an exclamation point

No other change in output.

Signed-off-by: Joe Perches 
Cc: Andy Whitcroft 
Cc: Andreas Herrmann 
LKML-Reference: <1260340250.27677.191.camel@Joe-Laptop.home>
Signed-off-by: Ingo Molnar

x86: microcode: use smp_call_function_single instead of set_cpus_allowed, cleanup of synchronization logic

2009-05-12T08:36:44+00:00

* Solve issues described in 6f66cbc63081fd70e3191b4dbb796746780e5ae1
  in a way that doesn't resort to set_cpus_allowed();

* in fact, only collect_cpu_info and apply_microcode callbacks
  must run on a target cpu, others will do just fine on any other.
  smp_call_function_single() (as suggested by Ingo) is used to run
  these callbacks on a target cpu.

* cleanup of synchronization logic of the 'microcode_core' part

  The generic 'microcode_core' part guarantees that only a single cpu
  (be it a full-fledged cpu, one of the cores or HT)
  is being updated at any particular moment of time.

  In general, there is no need for any additional sync. mechanism in
  arch-specific parts (the patch removes existing spinlocks).

  See also the "Synchronization" section in microcode_core.c.

* return -EINVAL instead of -1 (which is translated into -EPERM) in
  microcode_write(), reload_cpu() and mc_sysdev_add(). Other suggestions
  for an error code?

* use 'enum ucode_state' as return value of request_microcode_{fw, user}
  to gain more flexibility by distinguishing between real error cases
  and situations when an appropriate ucode was not found (which is not an
  error per-se).

* some minor cleanups

Thanks a lot to Hugh Dickins for review/suggestions/testing!

   Reference: http://marc.info/?l=linux-kernel&m=124025889012541&w=2

[ Impact: refactor and clean up microcode driver locking code ]

Signed-off-by: Dmitry Adamushko 
Acked-by: Hugh Dickins 
Cc: Andrew Morton 
Cc: Rusty Russell 
Cc: Andreas Herrmann 
Cc: Peter Oruba 
Cc: Arjan van de Ven 
LKML-Reference: <1242078507.5560.9.camel@earth>
[ did some more cleanups ]
Signed-off-by: Ingo Molnar 
 arch/x86/include/asm/microcode.h  |   25 ++
 arch/x86/kernel/microcode_amd.c   |   58 ++----
 arch/x86/kernel/microcode_core.c  |  326 +++++++++++++++++++++-----------------
 arch/x86/kernel/microcode_intel.c |   92 +++-------
 4 files changed, 261 insertions(+), 240 deletions(-)

(~20 new comment lines)