[coreboot] errata#89 patch for Family 0Fh Prozessors

[Josef Kellermann]

Am 03.02.2011 10:14, schrieb Alex G.:
> On 02/03/2011 10:17 AM, Josef Kellermann wrote:
>>> e are at it. I'm attaching some very old patch (r2978 ;)
>>> which contains some errata fixes too, if you have some spare time
>>> please try to check if it is correct.
>>>
>>> Signed-off-by: Rudolf Marek<r.marek at assembler.cz>
>>>
>>>
>>> Thanks,
>>> Rudolf
>>>
>> Hi,
>> are you sure ?
> You're right:
>
> AMD Publication #33610, page34
>
> "[Regarding erratum #131]
> Systems implementing the workaround for erratum #169 should not apply
> this workaround."
>
> So it seems AMD recommends either or, but not both.
>
> Page 59:
> "[Regarding erratum #169]
> The workaround for this erratum supersedes the workaround for erratum
> #131. When implementing
> this workaround, the workaround for erratum #131 should not be applied."
>
> So we better implement only 169, and explain in the comments why 131 is
> left out.
>
> Alex
>
>
Hi,
sorry for the misunderstanding.
Setting bit 32 in msr should be ->  'msr.hi |= (1<<  0)', no?

Josef