[coreboot] [PATCH] disabling microcode update
Stefan Reinauer
stefan.reinauer at coreboot.org
Mon Feb 21 03:58:01 CET 2011
On 2/19/11 12:45 PM, xdrudis wrote:
> On Fri, Feb 18, 2011 at 10:19:31AM -0500, Ward Vandewege wrote:
>> > Hi Xavi,
>> >
>> > On Wed, Feb 16, 2011 at 02:45:02PM +0100, Xavi Drudis Ferran wrote:
>>> > > Should I send a patch making a Kconfig option to not upgrade microcode for fam10? Is there any interest in that ?
What's the particular rationale behind this? When disabling microcode
updates, why don't you also disable the other erratas?
> + Select this to apply (propietary?) patches to the cpu
I don't think the word proprietary (nor the question mark) applies very
well here.
> + microcode provided by AMD to correct issues in the CPU after
> + production, and distributed with coreboot (not necessarily
> + the latest microcode version produced by AMD, but only
> + applied if newer than the version in your CPU).
> +
> + Unselect to let FAM10 CPUs run with factory microcode. If
Here's some formatting problem. It's also unclear what you mean by
"factory microcode" imho.
> + you unselect this, no binary microcode patches will be
> + included in the image, so it will help you get an image
> + which you have the entire source code for and may simplify
> + license compliance (IANAL).
I don't see how this makes license compliance any easier. Also, please
refrain from using terms like IANAL. If we make claims they should be
either right to the best of our knowledge or we don't put them in the
source code, anyways.
BTW, some corporations had legal teams analyze the microcode license and
it was considered not problematic for inclusion in coreboot in the sense
of the GPL.
> --- src/cpu/amd/model_10xxx/update_microcode.c 2011-02-19 21:56:44.000000000 +0100
> +++ src/cpu/amd/model_10xxx/update_microcode.c 2011-02-19 22:09:17.000000000 +0100
> @@ -29,6 +29,7 @@
> #include<cpu/amd/microcode.h>
> #endif
>
> +#if CONFIG_UPDATE_CPU_MICROCODE
> static const u8 microcode_updates[] __attribute__ ((aligned(16))) = {
>
Please change the patch to not include update_microcode.c in the
Makefile for the case that microcode selection is disabled.
Stefan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20110220/19e1c9f9/attachment.html>
More information about the coreboot
mailing list