From owner-freebsd-hackers@FreeBSD.ORG Tue Aug 16 16:04:03 2011 Return-Path: Delivered-To: hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 59844106566C; Tue, 16 Aug 2011 16:04:03 +0000 (UTC) (envelope-from joesuf4@gmail.com) Received: from mail-vx0-f182.google.com (mail-vx0-f182.google.com [209.85.220.182]) by mx1.freebsd.org (Postfix) with ESMTP id EC4728FC15; Tue, 16 Aug 2011 16:04:02 +0000 (UTC) Received: by vxh11 with SMTP id 11so60881vxh.13 for ; Tue, 16 Aug 2011 09:04:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; bh=OTFixKda/ANkZKbhy/r1i9eX9aWpomzXbUaPl7evfrY=; b=Ku+8gHPUCA3yxaZaFCb9HQyGd5uS0UUUgNQE0b2YjuOf8Xhz2/gXzmsZ8oGkDISf0m 6O1q1D8/cae2wIIxqdrS6YGdIgoKQOqrTE31mbh+cIVib/3HSqvQ9H0R/u9FaKTv/Hi+ jP0L35CEhmrzDA7c118lZ7CNCO4tXWiR/tptw= MIME-Version: 1.0 Received: by 10.220.107.12 with SMTP id z12mr575386vco.129.1313510642194; Tue, 16 Aug 2011 09:04:02 -0700 (PDT) Received: by 10.220.186.10 with HTTP; Tue, 16 Aug 2011 09:04:02 -0700 (PDT) In-Reply-To: <4E4A81DD.3030202@FreeBSD.org> References: <4E498326.2060308@FreeBSD.org> <4E4988F0.7060000@FreeBSD.org> <4E498E3D.7050100@FreeBSD.org> <4E4A81DD.3030202@FreeBSD.org> Date: Tue, 16 Aug 2011 12:04:02 -0400 Message-ID: From: Joe Schaefer To: Alexander Motin Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: hackers@freebsd.org Subject: Re: Clock stalls on Sabertooth 990FX X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 16 Aug 2011 16:04:03 -0000 On Tue, Aug 16, 2011 at 10:42 AM, Alexander Motin wrote: > Joe Schaefer wrote: >>>>>>> If changing timecounter won't help, try please this patch: >>>>>>> >>>>>>> --- acpi_hpet.c.prev =C2=A0 =C2=A02010-12-25 11:28:45.000000000 +02= 00 >>>>>>> +++ acpi_hpet.c 2011-05-11 14:30:59.000000000 +0300 >>>>>>> @@ -190,7 +190,7 @@ restart: >>>>>>> =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0bus_write_4(= sc->mem_res, HPET_TIMER_COMPARATOR(t->num), >>>>>>> =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0t->next); >>>>>>> =C2=A0 =C2=A0 =C2=A0 =C2=A0} >>>>>>> - =C2=A0 =C2=A0 =C2=A0 if (fdiv < 5000) { >>>>>>> + =C2=A0 =C2=A0 =C2=A0 if (1 || fdiv < 5000) { >>>>>>> =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0bus_read_4(s= c->mem_res, HPET_TIMER_COMPARATOR(t->num)); >>>>>>> =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0now =3D 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 =C2=A0compatitor regs will n= ot take >>> effect immediately, you need a read to reg to flush it; >> >> So far the patch performs flawlessly for me. =C2=A0I'm tempted to reenab= le 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@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 >