Re: [PATCH 1/2] x86, amd-ucode: Check UCODE_MAGIC before loading thecontainer file

From: Ingo Molnar
Date: Sun Nov 08 2009 - 09:56:07 EST



* Borislav Petkov <petkovbb@xxxxxxxxxxxxxx> wrote:

> Hi Ingo,
>
> On Sun, Nov 08, 2009 at 01:04:18PM +0100, Ingo Molnar wrote:
> > hm, there's no changelog. How was it found,
>
> code staring :)
>
> > what's the significance,
>
> required by design.
>
> > do we want it for .32 - i suspect we want - but we need a better
> > changelog for that.
>
> Yes. How's that?
>
> --
> >From f41bced57e05c8adc0d2c13864b6323c191d45d2 Mon Sep 17 00:00:00 2001
> From: Borislav Petkov <borislav.petkov@xxxxxxx>
> Date: Thu, 29 Oct 2009 14:00:54 +0100
> Subject: [PATCH] x86, amd-ucode: Check UCODE_MAGIC before loading the container file
>
> When loading a microcode binary, we should check the microcode magic
> value embedded in the image before continuing loading. The original
> check got lost somehow so re-add it to the firmware requesting path.
>
> Signed-off-by: Borislav Petkov <borislav.petkov@xxxxxxx>
> Signed-off-by: Andreas Herrmann <andreas.herrmann3@xxxxxxx>
> ---
> arch/x86/kernel/microcode_amd.c | 6 ++++++
> 1 files changed, 6 insertions(+), 0 deletions(-)

Looks good - thanks!

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/