From owner-freebsd-acpi@FreeBSD.ORG Tue Jan 11 08:54:10 2011 Return-Path: Delivered-To: acpi@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2F0C71065674 for ; Tue, 11 Jan 2011 08:54:10 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from fallbackmx09.syd.optusnet.com.au (fallbackmx09.syd.optusnet.com.au [211.29.132.242]) by mx1.freebsd.org (Postfix) with ESMTP id 539828FC1B for ; Tue, 11 Jan 2011 08:54:08 +0000 (UTC) Received: from mail06.syd.optusnet.com.au (mail06.syd.optusnet.com.au [211.29.132.187]) by fallbackmx09.syd.optusnet.com.au (8.13.1/8.13.1) with ESMTP id p0B6R6WB010619 for ; Tue, 11 Jan 2011 17:27:06 +1100 Received: from c122-106-165-206.carlnfd1.nsw.optusnet.com.au (c122-106-165-206.carlnfd1.nsw.optusnet.com.au [122.106.165.206]) by mail06.syd.optusnet.com.au (8.13.1/8.13.1) with ESMTP id p0B6QxsZ031995 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Tue, 11 Jan 2011 17:27:01 +1100 Date: Tue, 11 Jan 2011 17:26:59 +1100 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: Alexander Motin In-Reply-To: <4D2B8944.6030200@FreeBSD.org> Message-ID: <20110111160606.H2006@besplex.bde.org> References: <201101101500.45783.jhb@freebsd.org> <4D2B67E4.70907@FreeBSD.org> <201101101515.47127.jhb@freebsd.org> <4D2B8944.6030200@FreeBSD.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: acpi@FreeBSD.org, "Arno J. Klaassen" Subject: Re: Tyan S3992-E: hpet no longer working X-BeenThere: freebsd-acpi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: ACPI and power management development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 11 Jan 2011 08:54:10 -0000 On Tue, 11 Jan 2011, Alexander Motin wrote: > Arno J. Klaassen wrote: >> Sure .. that said, the BIOS I use is the last official release for this >> board (Sept 2009) and not even a more recent beta-release is available. >> >> I would expect reporting a disabled device which cannot be enabled via >> de BIOS a bug deserving a newer release. >> >> Anyway, this bug isn't very harmful for me, but the non-hpet >> timecounters don't seem that fun either : >> >> # uptime >> 10:27PM up 2 days, 5:44 >> >> # sysctl kern.timecounter.hardware kern.timecounter.choice >> kern.timecounter.hardware: ACPI-safe >> kern.timecounter.choice: TSC(-100) i8254(0) ACPI-safe(850) dummy(-1000000) >> >> # vmstat -i | fgrep cpu: >> cpu0:timer 38599321 199 >> cpu6:timer 2151003 11 >> cpu1:timer 7121075 36 >> cpu3:timer 1808269 9 >> cpu5:timer 3832463 19 >> cpu2:timer 2399988 12 >> cpu7:timer 2013444 10 >> cpu4:timer 21630368 111 >> >> (default HZ ....) >> >> Maybe I should try downgrading the BIOS? > > So what here seems not funny to you? Lower timer interrupt rate is not a > bug but feature of 9-CURRENT. They (cpu*:timer) also aren't timecounters :-). Hmm, with hpet on FreeBSD cluster machines, there is now only hpet. How is statclock distributed with hpet? I never properly reviewed the latest "irqN"-printing changes in systat, and just noticed that they break printing of "irq" the usual case where the interrupt name starts with "irqN:" (then systat removes "irqN:" and never puts back "irq"). Not-so-quick fix: % Index: vmstat.c % =================================================================== % RCS file: /home/ncvs/src/usr.bin/systat/vmstat.c,v % retrieving revision 1.93 % diff -u -2 -r1.93 vmstat.c % --- vmstat.c 11 Dec 2010 08:32:16 -0000 1.93 % +++ vmstat.c 11 Jan 2011 06:20:01 -0000 % @@ -244,5 +244,10 @@ % *--cp1 = '\0'; % % - /* Convert "irqN: name" to "name irqN". */ % + /* % + * Convert "irqN: name" to "name irqN", "name N" or % + * "name". First reduce to "name"; then append % + * " irqN" if that fits, else " N" if that fits, % + * else drop all of "N". % + */ % if (strncmp(cp, "irq", 3) == 0) { % cp1 = cp + 3; % @@ -256,5 +261,8 @@ % cp2 = strdup(cp); % bcopy(cp1, cp, sz - (cp1 - cp) + 1); % - if (sz <= 10 + 4) { % + if (sz <= 10 + 1) { % + strcat(cp, " "); % + strcat(cp, cp2); % + } else if (sz <= 10 + 4) { % strcat(cp, " "); % strcat(cp, cp2 + 3); % @@ -266,5 +274,9 @@ % /* % * Convert "name irqN" to "name N" if the former is % - * longer than the field width. % + * longer than the field width. This handles some % + * cases where the original name did not start with % + * "irqN". We don't bother dropping partial "N"s in % + * this case. The "name" part may be too long in % + * either case; then we blindly truncate it. % */ % if ((cp1 = strstr(cp, "irq")) != NULL && This restores part of rev.1.90, updates the first comment to match the code changes in rev.1.90, and expands the last comment. Bruce