From owner-freebsd-current@FreeBSD.ORG  Sun Sep 12 15:39:01 2010
Return-Path: <owner-freebsd-current@FreeBSD.ORG>
Delivered-To: freebsd-current@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 98BF6106566B;
	Sun, 12 Sep 2010 15:39:01 +0000 (UTC) (envelope-from avg@icyb.net.ua)
Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140])
	by mx1.freebsd.org (Postfix) with ESMTP id 477308FC13;
	Sun, 12 Sep 2010 15:39:00 +0000 (UTC)
Received: from porto.topspin.kiev.ua (porto-e.starpoint.kiev.ua
	[212.40.38.100])
	by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id SAA28410;
	Sun, 12 Sep 2010 18:38:58 +0300 (EEST)
	(envelope-from avg@icyb.net.ua)
Received: from localhost.topspin.kiev.ua ([127.0.0.1])
	by porto.topspin.kiev.ua with esmtp (Exim 4.34 (FreeBSD))
	id 1Ouodu-000Fyv-Lm; Sun, 12 Sep 2010 18:38:58 +0300
Message-ID: <4C8CF412.9080601@icyb.net.ua>
Date: Sun, 12 Sep 2010 18:38:58 +0300
From: Andriy Gapon <avg@icyb.net.ua>
User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-US;
	rv:1.9.2.8) Gecko/20100822 Lightning/1.0b2 Thunderbird/3.1.2
MIME-Version: 1.0
To: Alexander Motin <mav@FreeBSD.org>
References: <4C8BCAC5.5050008@root.org>	<mailpost.1284277196.1767764.83548.mailing.freebsd.current@FreeBSD.cs.nctu.edu.tw>	<4C8C8B64.8020907@FreeBSD.org>	<20100912182625.c49d3f1d.nork@FreeBSD.org>	<4C8C9F06.4090505@icyb.net.ua>	<20100912190537.621e357e.nork@FreeBSD.org>	<20100912190952.8c0d5726.nork@FreeBSD.org>
	<20100912192518.e791c191.nork@FreeBSD.org>
	<4C8CAC01.70004@icyb.net.ua> <4C8CAD7D.50602@FreeBSD.org>
	<4C8CF03F.1050902@icyb.net.ua>
In-Reply-To: <4C8CF03F.1050902@icyb.net.ua>
X-Enigmail-Version: 1.1.2
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Cc: freebsd-acpi@FreeBSD.org, FreeBSD-Current <freebsd-current@FreeBSD.org>,
	Norikatsu Shigemura <nork@FreeBSD.org>
Subject: Re: CPU C-state storange on Panasonic TOUGH BOOK CF-R9
X-BeenThere: freebsd-current@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Discussions about the use of FreeBSD-current
	<freebsd-current.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>, 
	<mailto:freebsd-current-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-current>
List-Post: <mailto:freebsd-current@freebsd.org>
List-Help: <mailto:freebsd-current-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>,
	<mailto:freebsd-current-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 12 Sep 2010 15:39:01 -0000

on 12/09/2010 18:22 Andriy Gapon said the following:
> Observations are correct, but incomplete; the conclusions are wrong.
> At the end of the boot there are message like this one:
> PROCESSOR-0722 [402244] cpu_cx_cst            : acpi_cpu0: Got C2 - 245 latency
> This is a result of re-evaluation of _CST because of a notification from ACPI.
> 

But still, as you suggest, a patch like the following should be tested and
committed:

--- a/sys/dev/acpica/acpi_cpu.c
+++ b/sys/dev/acpica/acpi_cpu.c
@@ -828,7 +828,8 @@ acpi_cpu_cx_list(struct acpi_cpu_softc *sc)
     sbuf_new(&sb, sc->cpu_cx_supported, sizeof(sc->cpu_cx_supported),
 	SBUF_FIXEDLEN);
     for (i = 0; i < sc->cpu_cx_count; i++) {
-	sbuf_printf(&sb, "C%d/%d ", i + 1, sc->cpu_cx_states[i].trans_lat);
+	sbuf_printf(&sb, "C%d/%d ", sc->cpu_cx_states[i].type,
+	    sc->cpu_cx_states[i].trans_lat);
 	if (sc->cpu_cx_states[i].type < ACPI_STATE_C3)
 	    sc->cpu_non_c3 = i;
     }

P.S. I restored acpi@ cc: which I think is quite relevant, but was somehow lost.
-- 
Andriy Gapon