lkml.org 
[lkml]   [2013]   [Aug]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] linux-firmware/AMD: add readme files for amd-ucode
From
Date
I don't think this documentation is currently very useful.  See further
comments inline:

On Fri, 2013-07-19 at 17:43 -0500, Sherry Hurwitz wrote:
> File: README
> File: microcode_amd.bin.README
> File: microcode_amd_fam15h.bin.README
>
> Signed-off-by: Sherry Hurwitz <sherry.hurwitz@amd.com>
> ---
> amd-ucode/README | 46 +++++++++++++++++++++++++++++
> amd-ucode/microcode_amd.bin.README | 29 ++++++++++++++++++
> amd-ucode/microcode_amd_fam15h.bin.README | 13 ++++++++
> 3 files changed, 88 insertions(+)
> create mode 100644 amd-ucode/README
> create mode 100644 amd-ucode/microcode_amd.bin.README
> create mode 100644 amd-ucode/microcode_amd_fam15h.bin.README
>
> diff --git a/amd-ucode/README b/amd-ucode/README
> new file mode 100644
> index 0000000..a92831d
> --- /dev/null
> +++ b/amd-ucode/README
> @@ -0,0 +1,46 @@
> +This package provides latest microcode patches
> +for AMD processor families >= 0x10.
> +
> +See http://www.amd64.org/support/microcode.html
> +for details.

As Marc Dietrich pointed out, this URL is currently broken.

> +Microcode patches are included in container files:
> +- 'microcode_amd.bin' (for AMD CPU families 10h - 14h)
> +- 'microcode_amd_fam15h.bin' (for AMD CPU family 15h)
> +
> +Please read the file INSTALL for install instructions.
> +Please read the file LICENSE for licensing information.

These aren't the filenames used in linux-firmware.git.

> +The container files include following microcode patches:
> +
> + mc_patch_01000083_PUB-v4/mc_patch_01000083.asm
> + mc_patch_01000084_PUB-v4/mc_patch_01000084.asm
> + mc_patch_010000C7_PUB-v1/mc_patch_010000C7.asm
> + mc_patch_010000C8_PUB-v1/mc_patch_010000C8.asm
> + mc_patch_010000D9_PUB-v1/mc_patch_010000D9.asm
> + mc_patch_010000DA_PUB-v1/mc_patch_010000DA.asm
> + mc_patch_010000DB_PUB-v1/mc_patch_010000DB.asm
> + mc_patch_010000DC_PUB-v1/mc_patch_010000DC.asm
> + mc_patch_02000032_PUB-v3/mc_patch_02000032.asm
> + mc_patch_03000027_PUB-v1/mc_patch_03000027.asm
> + mc_patch_05000029_PUB-v1/mc_patch_05000029.asm
> + mc_patch_05000119_PUB-v1/mc_patch_05000119.asm
> + mc_patch_0600063D_PUB-v1/mc_patch_0600063D.asm
> + mc_patch_06000822_PUB-v1/mc_patch_06000822.asm
> + mc_patch_06001119_PUB-v2/mc_patch_06001119.asm

Are these filenames supposed to be meaningful? Are they referenced by
some other documentation?

> +***********************************************************************
> +Copyright 2008-2013 ADVANCED MICRO DEVICES, INC. All Rights Reserved.
> +
> +AMD is granting you permission to use this software and documentation
> +(if any) (collectively, the “Materials”) pursuant to the terms and
> +conditions of the Software License Agreement included with the
> +Materials. This header does NOT give you permission to use the
> +Materials or any rights under AMD’s intellectual property. Your use
> +of any portion of these Materials shall constitute your acceptance of
> +those terms and conditions. If you do not agree to the terms and
> +conditions of the Software License Agreement, you do not have
> +permission to use any portion of these Materials. If you do not have
> +a copy of the Software License Agreement, contact your AMD
> +representative for a copy.
> +

Isn't this redundant with LICENSE.amd-ucode?

> diff --git a/amd-ucode/microcode_amd.bin.README b/amd-ucode/microcode_amd.bin.README
> new file mode 100644
> index 0000000..617d951
> --- /dev/null
> +++ b/amd-ucode/microcode_amd.bin.README
> @@ -0,0 +1,29 @@
> +;******************************************************************************
> +; The associated microcode container file fixes the errata as documented in
> +; Revision Guide for AMD Family 10h Processors, order #41322,
> +; Revision Guide for AMD Family 11h Processors, order #41788,
> +; Revision Guide for AMD Family 12h Processors, order #44739,
> +; Revision Guide for AMD Family 14h Models 00h-0Fh Processors, order #47534,

Where can these be found? (I found *some* revision guides with a web
search, but nothing with errata numbers above 400.)

> +; for different revisions of AMD processors as follows:
> +;
> +; CPUIDFn[0000_0001]_EAX; ID; Errata fixed;
> +;
> +; 0x00100F22; 0x01000083; 244, 260, 280, 302, 308, 315, 342;
> +; 0x00100F23; 0x01000083; 244, 260, 280, 302, 308, 315, 342;
> +; 0x00100F2A; 0x01000084; 244, 260, 280, 302, 308, 315, 342;
> +; 0x00100F42; 0x010000DB; 342, 440, 573;
> +; 0x00100F43; 0x010000C8; 407, 440;
> +; 0x00100F52; 0x010000DB; 342, 440, 573;
> +; 0x00100F53; 0x010000C8; 407, 440;
> +; 0x00100F62; 0x010000C7; 407, 440;
> +; 0x00100F63; 0x010000C8; 407, 440;
> +; 0x00100F80; 0x010000DA; 419, 440, 573;
> +; 0x00100F81; 0x010000D9; #406, #407, #440, #573, #669;
> +; 0x00100F91; 0x010000D9; #406, #407, #440, #573, #669;
> +; 0x00100FA0; 0x010000DC; 438, 440, 573;
> +; 0x00200F31; 0x02000032; 311, 316;
> +; 0x00300F10; 0x03000027; #564, #573, #662, #686;
> +; 0x00500F10; 0x05000029; #461, #564, #594, #595, #784;
> +; 0x00500F20; 0x05000119; #461, #564, #594, #639, #662, #686, #784;
> +;******************************************************************************

Presumably when I see this in /proc/cpuinfo:

cpu family : 16
model : 6
model name : AMD Turion(tm) II Neo N54L Dual-Core Processor
stepping : 3
microcode : 0x10000c8

this means the relevant line is:

> +; 0x00100F43; 0x010000C8; 407, 440;
[...]

But can you make the documentation explicit about how to look this up
based on the information Linux provides?

Ben.

--
Ben Hutchings
The two most common things in the universe are hydrogen and stupidity.
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2013-08-08 23:21    [W:0.190 / U:0.160 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site