All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/17] x86/microcode: Untangle loader from BLK_DEV_INITRD
@ 2016-02-03 11:33 Borislav Petkov
  2016-02-03 11:33 ` [PATCH 01/17] x86/microcode: Untangle " Borislav Petkov
                   ` (16 more replies)
  0 siblings, 17 replies; 37+ messages in thread
From: Borislav Petkov @ 2016-02-03 11:33 UTC (permalink / raw)
  To: X86 ML; +Cc: LKML

From: Borislav Petkov <bp@suse.de>

Hi,

here are a bunch of patches untangling the loader dependency from
BLK_DEV_INITRD and fixing the builtin microcode loading on Intel. So
we're basically supporting two early-loading methods, one is the builtin
microcode in the kernel (CONFIG_EXTRA_FIRMWARE) and the detached initrd.
Both are described in Documentation/x86/early-microcode.txt.

First two are CC:stable <#4.4> so that the fallout from the merge of the
early and late loader is addressed.

The rest is smallish fixes and more cleanup to the Intel loader,
hopefully making it more readable.

Thanks.

Borislav Petkov (16):
  x86/microcode: Untangle from BLK_DEV_INITRD
  x86/microcode/intel: Make early loader look for builtin microcode too
  x86/microcode: Remove redundant __setup() param parsing
  x86/microcode: Issue update message only once
  x86/microcode/AMD: Drop redundant printk prefix
  x86/microcode/intel: Rename local variables of type struct
    mc_saved_data
  x86/microcode/intel: Rename mc_saved_count to num_saved
  x86/microcode/intel: Rename mc_intel variable to mc
  x86/microcode/intel: Move the BUG_ON up and turn it into WARN_ON
  x86/microcode/intel: Cleanup apply_microcode_intel()
  x86/microcode/intel: Use *wrmsrl variants
  x86/microcode/intel: Rename mc_saved_in_initrd
  x86/microcode/intel: Remove unused arg of
    get_matching_model_microcode()
  x86/microcode/intel: Cleanup get_matching_model_microcode()
  x86/microcode/AMD: Issue microcode updated message later
  x86/microcode: Document builtin microcode loading method

Dan Carpenter (1):
  x86/microcode: Remove an unneeded NULL check

 Documentation/x86/early-microcode.txt |  25 +++
 arch/x86/Kconfig                      |  27 ++--
 arch/x86/include/asm/microcode.h      |  26 ++++
 arch/x86/kernel/cpu/microcode/amd.c   |  11 +-
 arch/x86/kernel/cpu/microcode/core.c  |  17 +-
 arch/x86/kernel/cpu/microcode/intel.c | 281 +++++++++++++++++-----------------
 6 files changed, 213 insertions(+), 174 deletions(-)

-- 
2.3.5

^ permalink raw reply	[flat|nested] 37+ messages in thread
* Re: [patch] x86, microcode: remove an unneeded NULL check
@ 2016-01-22 13:00 Borislav Petkov
  0 siblings, 0 replies; 37+ messages in thread
From: Borislav Petkov @ 2016-01-22 13:00 UTC (permalink / raw)
  To: kernel-janitors

On Mon, Jan 20, 2014 at 01:30:47PM +0300, Dan Carpenter wrote:
> "uci" is an element of the ucode_cpu_info[] array, it can't be NULL.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> 
> diff --git a/arch/x86/kernel/cpu/microcode/core.c b/arch/x86/kernel/cpu/microcode/core.c
> index 15c987698b0f..05db44e4198c 100644
> --- a/arch/x86/kernel/cpu/microcode/core.c
> +++ b/arch/x86/kernel/cpu/microcode/core.c
> @@ -382,7 +382,7 @@ static enum ucode_state microcode_init_cpu(int cpu, bool refresh_fw)
>  	enum ucode_state ustate;
>  	struct ucode_cpu_info *uci = ucode_cpu_info + cpu;
>  
> -	if (uci && uci->valid)
> +	if (uci->valid)
>  		return UCODE_OK;
>  
>  	if (collect_cpu_info(cpu))

Applied, after almost 2 years!

Thanks.

-- 
Regards/Gruss,
    Boris.

SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)
-- 

^ permalink raw reply	[flat|nested] 37+ messages in thread

end of thread, other threads:[~2016-02-09 12:26 UTC | newest]

Thread overview: 37+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-03 11:33 [PATCH 00/17] x86/microcode: Untangle loader from BLK_DEV_INITRD Borislav Petkov
2016-02-03 11:33 ` [PATCH 01/17] x86/microcode: Untangle " Borislav Petkov
2016-02-09 12:20   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 02/17] x86/microcode/intel: Make early loader look for builtin microcode too Borislav Petkov
2016-02-09 12:20   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 03/17] x86/microcode: Remove redundant __setup() param parsing Borislav Petkov
2016-02-09 12:21   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 04/17] x86/microcode: Remove an unneeded NULL check Borislav Petkov
2014-01-20 10:30   ` [patch] x86, microcode: remove " Dan Carpenter
2016-02-09 12:21     ` [tip:x86/microcode] x86/microcode: Remove " tip-bot for Dan Carpenter
2016-02-03 11:33 ` [PATCH 05/17] x86/microcode: Issue update message only once Borislav Petkov
2016-02-09 12:21   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 06/17] x86/microcode/AMD: Drop redundant printk prefix Borislav Petkov
2016-02-09 12:22   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 07/17] x86/microcode/intel: Rename local variables of type struct mc_saved_data Borislav Petkov
2016-02-09 12:22   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 08/17] x86/microcode/intel: Rename mc_saved_count to num_saved Borislav Petkov
2016-02-09 12:22   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 09/17] x86/microcode/intel: Rename mc_intel variable to mc Borislav Petkov
2016-02-09 12:23   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 10/17] x86/microcode/intel: Move the BUG_ON up and turn it into WARN_ON Borislav Petkov
2016-02-09 12:23   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 11/17] x86/microcode/intel: Cleanup apply_microcode_intel() Borislav Petkov
2016-02-09 12:24   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 12/17] x86/microcode/intel: Use *wrmsrl variants Borislav Petkov
2016-02-09 12:24   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 13/17] x86/microcode/intel: Rename mc_saved_in_initrd Borislav Petkov
2016-02-09 12:24   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 14/17] x86/microcode/intel: Remove unused arg of get_matching_model_microcode() Borislav Petkov
2016-02-09 12:25   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 15/17] x86/microcode/intel: Cleanup get_matching_model_microcode() Borislav Petkov
2016-02-09 12:25   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 16/17] x86/microcode/AMD: Issue microcode updated message later Borislav Petkov
2016-02-09 12:25   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
2016-02-03 11:33 ` [PATCH 17/17] x86/microcode: Document builtin microcode loading method Borislav Petkov
2016-02-09 12:26   ` [tip:x86/microcode] " tip-bot for Borislav Petkov
  -- strict thread matches above, loose matches on Subject: below --
2016-01-22 13:00 [patch] x86, microcode: remove an unneeded NULL check Borislav Petkov

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.