[coreboot] [PATCH] v3: Work around broken MTRR setup in VIA CAR
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Sun Nov 2 15:49:42 CET 2008
On 02.11.2008 04:18, Corey Osgood wrote:
> On Sat, Nov 1, 2008 at 6:32 PM, Carl-Daniel Hailfinger <
> c-d.hailfinger.devel.2006 at gmx.net> wrote:
>
>
>> On 01.11.2008 23:21, Carl-Daniel Hailfinger wrote:
>>
>>> Once we touch the MTRRs in VIA disable_car(), the CPU resets. Since
>>> workarounds are better than instant reboots, mangle the code so that it
>>> only switches stacks and flushes the cache.
>>>
>>> There are two genuine fixes in there as well: Switch %esp before CAR is
>>> disabled. That way, debugging becomes easier and the stack is always
>>>
>> valid.
>>
>> And one of the nastier bugs easily happening in C: We had a pointer to a
>> const struct, but we wanted a const pointer to a struct. This kills the
>> (correct) warning about that code.
>>
>>
>>> Many thanks to Corey for testing countless iterations of that code.
>>>
>>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net
>>>
>>> Attached for poor gmail users.
>>>
>>>
>> New version attached. It also fixes a missing semicolon that somehow
>> snuck into the earlier patch.
>>
>> [...]
>>
>
> I don't think we need that anymore ;)
Thanks for the hint about the testing leftovers.
> Tested and
> Acked-by: Corey Osgood <corey.osgood at gmail.com>
>
Thanks, committed in r978.
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
More information about the coreboot
mailing list