Clock stalls on Sabertooth 990FX

[Joe Schaefer]

On Tue, Aug 16, 2011 at 10:42 AM, Alexander Motin <mav at freebsd.org> wrote:
> Joe Schaefer wrote:
>>>>>>> If changing timecounter won't help, try please this patch:
>>>>>>>
>>>>>>> --- acpi_hpet.c.prev    2010-12-25 11:28:45.000000000 +0200
>>>>>>> +++ acpi_hpet.c 2011-05-11 14:30:59.000000000 +0300
>>>>>>> @@ -190,7 +190,7 @@ restart:
>>>>>>>                bus_write_4(sc->mem_res, HPET_TIMER_COMPARATOR(t->num),
>>>>>>>                    t->next);
>>>>>>>        }
>>>>>>> -       if (fdiv < 5000) {
>>>>>>> +       if (1 || fdiv < 5000) {
>>>>>>>                bus_read_4(sc->mem_res, HPET_TIMER_COMPARATOR(t->num));
>>>>>>>                now = bus_read_4(sc->mem_res, HPET_MAIN_COUNTER);
>>>>>>>
>>>>>>> --
>>>>>>> Alexander Motin
>>>>>> Will do next.
>>>>>>
>>>>> Patch applied. Running with HPET eventtimer and no stalls during
>>>>> make buildworld -j12.
>>>>>
>>> it maybe help, I used to come across a bug on Linux with regard to HPET,
>>> some northbridge chipset (maybe amd's, where
>>> HPET sit on) has a problem, that writes to  compatitor regs will not take
>>> effect immediately, you need a read to reg to flush it;
>>
>> So far the patch performs flawlessly for me.  I'm tempted to reenable turbo
>> mode just for kicks (someday, not today- delighted with the uptime!)
>
> I am going to commit following patch:
> http://people.freebsd.org/~mav/hpet.paranoid.patch
> . It uses same assumptions as Linux. Try it please.

+1 to apply: (tested with buildworld -j18)

--------------------------------------------------------------
>>> World build completed on Tue Aug 16 12:02:20 EDT 2011
--------------------------------------------------------------
5255.533u 2443.815s 30:28.62 421.0%     6535+4429k 0+0io 223826pf+0w
sextant# uname -a
FreeBSD sextant.sunstarsys.com 9.0-BETA1 FreeBSD 9.0-BETA1 #1
r224899M: Tue Aug 16 10:55:12 EDT 2011
joe at sextant.sunstarsys.com:/usr/obj/usr/src/sys/DTRACE  amd64
sextant# uptime
12:02PM  up 32 mins, 3 users, load averages: 6.90, 12.58, 10.64


>
> --
> Alexander Motin
>