From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [patch]: fixing of pmtimer overflow that make Cx states time incorrect Date: Mon, 9 Feb 2009 11:56:00 -0800 Message-ID: <20090209115600.4ecd65e5.akpm@linux-foundation.org> References: <1232342024.21839.6.camel@alexs-hp> <20090126225235.5e64d0ef.akpm@linux-foundation.org> <1233453863.3715.35.camel@localhost.localdomain> <1233539217.4069.24.camel@alexs-hp> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from smtp1.linux-foundation.org ([140.211.169.13]:41386 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754046AbZBIT67 (ORCPT ); Mon, 9 Feb 2009 14:58:59 -0500 In-Reply-To: <1233539217.4069.24.camel@alexs-hp> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: alex.shi@intel.com Cc: linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, Venkatesh Pallipadi , Len Brown On Mon, 02 Feb 2009 09:46:57 +0800 "alex.shi" wrote: > Yagui want to give a clear explanation to be used for commitment. So I resend > this again. > > On most boxes the ACPI PM timer is 24-bit counter that runs on 3.579545MHz > clock. In such case the max C-state sleep time should be less than 4687ms when > it is used to record C2/C3 duration time. > But on some boxes the max C-state sleep time is more than 4687ms. In such > case the overflow happens and the C-state duration time can't be counted > accurately. > > Use clocksource to get the C-state time instead of ACPI PM timer. and use > div64_u64 to convert US_TO_PM_TIME_TICKS in i386 mode. It seems that this bugfix was not applied, a cleanup patch was merged instead and this patch is now wrecked. I'll need to drop it, sorry.