This message generated a parse failure. Raw output follows here. Please use 'back' to navigate. From devnull@lkml.org Fri Apr 26 05:33:07 2024 Envelope-to: j@jasper.es Delivery-date: Fri, 18 Jan 2013 23:35:49 +0100 Received: from localhost ([127.0.0.1] helo=squeeze.vs19.net) by squeeze.vs19.net with esmtp (Exim 4.72) (envelope-from ) id 1TwKXH-0005JR-Nh for j@jasper.es; Fri, 18 Jan 2013 23:35:45 +0100 Received: from pop3.telfort.nl [213.205.33.252] by squeeze.vs19.net with POP3 (fetchmail-6.3.18) for (single-drop); Fri, 18 Jan 2013 23:35:43 +0100 (CET) Received: from cmgw-nl-1 (10.39.115.161) by mda-nl-3.mail.tiscali.sys (8.5.142) id 4FE1AB2F072A0466 for jasper@telfort.nl; Fri, 18 Jan 2013 23:24:49 +0100 Received: from cpsmtpb-ews08.kpnxchange.com ([213.75.39.13]) by cmgw-nl-1 with id paQo1k0040H1flR01aQo91; Fri, 18 Jan 2013 23:24:49 +0100 X-CNFS-Analysis: v=2.0 cv=Mc7uSuDf c=1 sm=2 a=UK1r566ZdBxH71SXbqIOeA==:17 a=3K5izOWuScsA:10 a=UDJLN6nE6_0A:10 a=WffSN0qW2HAA:10 a=FUFFwTGtsvEA:10 a=VwQbUJbxAAAA:8 a=D19gQVrFAAAA:8 a=5q2kDgg6UMm91oxvyqsA:9 a=CjuIK1q_8ugA:10 a=pfp7Q8OJeHhuxwiV:21 a=BiXE-IlQmC06tNLY:21 a=m8 Received: from cpsps-ews22.kpnxchange.com ([10.94.84.188]) by cpsmtpb-ews08.kpnxchange.com with Microsoft SMTPSVC(7.5.7601.17514); Fri, 18 Jan 2013 23:23:39 +0100 Received: from vger.kernel.org ([209.132.180.67]) by cpsps-ews22.kpnxchange.com with Microsoft SMTPSVC(7.5.7601.17514); Fri, 18 Jan 2013 23:23:39 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754549Ab3ARWYh (ORCPT ); Fri, 18 Jan 2013 17:24:37 -0500 Received: from mout.web.de ([212.227.17.12]:57559 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751092Ab3ARWYg (ORCPT ); Fri, 18 Jan 2013 17:24:36 -0500 Received: from netbook.localnet ([92.194.90.131]) by smtp.web.de (mrweb103) with ESMTPSA (Nemesis) id 0Lh6M7-1TBDhs44Fb-00oZZ0; Fri, 18 Jan 2013 23:24:28 +0100 From: Thomas Schlichter To: "Rafael J. Wysocki" Cc: Daniel Lezcano , Len Brown , Linus Torvalds , Peter De Schrijver , Andreas =?ISO-8859-1?Q?M=FCller?= , Julius Werner User-Agent: KMail/4.9.4 (Linux/3.8.0-rc4-00001-gfc300f4-dirty; KDE/4.9.4; i686; ; ) In-Reply-To: <1755809.pcSThauESZ@netbook> References: <2099093.cPXO6Nhn9v@netbook> <11530988.U26HMGnFLa@vostro.rjw.lan> <1755809.pcSThauESZ@netbook> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="nextPart2155860.WxWkS2UFC8" Content-Transfer-Encoding: 7Bit X-Provags-ID: V02:K0:/MiXOxdVzbuy1D+cG4JnMt2a821e5GxJ5wp48rDbe41 lo3o3K+or/gK4KY3o0NgUEtCijL6QCPF141JBVfEN1z6A74N+2 S92iUXOgf914sA0QuLjZGECEzoNHbSwh0rLC6e/7+zIgEKjr1O rIBuX3o77w67P6l8GMw6DN7FTopdhwVlYS/7wbi3R7Bcr6xmpS 65N1szI4N34UoEwQUOT1ZBksx77yb9T3Q51mkRCBeg= Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-Id: X-Mailing-List: linux-kernel@vger.kernel.org X-OriginalArrivalTime: 18 Jan 2013 22:23:39.0652 (UTC) FILETIME=[77406440:01CDF5CA] X-RcptDomain: telfort.nl This is a multi-part message in MIME format. --nextPart2155860.WxWkS2UFC8 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Dear Rafael, I just recognized that I did not really describe the problem that my patch fixes. Therefore I attached the output of following command after disconnecting the AC power without and with my patch: grep -r "" /sys/devices/system/cpu/cpu0/cpuidle/ One can clearly see that some information is missing and the C3 state is not used. Kind regards, Thomas Am Freitag, 18. Januar 2013, 21:34:29 schrieb Thomas Schlichter: > Dear Rafael, > > thank you for applying Daniel's patch. I tested 3.8-rc4 and found that one > patch ist still missing to fix the problem of not usable C-state after > disconnect. I had it attached with my last e-mail as patch 1. For your > conveniency, I have attached it here again. > > With that patch the problem is fixed for me. So please consider applying > this, too. > > Kind regards, > Thomas > > Am Sonntag, 13. Januar 2013, 21:04:45 schrieb Rafael J. Wysocki: > > On Sunday, January 13, 2013 03:41:34 PM Daniel Lezcano wrote: > > > On 01/13/2013 01:34 PM, Thomas Schlichter wrote: > > > > Hi, > > > > > > > > there is a long-standing regression about new C-states not working > > > > after > > > > disconnecting AC power from a laptop if the cpuidle driver "acpi-idle" > > > > is > > > > used. It was reported here: > > > > > > > > [1] https://bugzilla.kernel.org/show_bug.cgi?id=42870 (March 5th > > > > 2012) > > > > [2] https://bugzilla.kernel.org/show_bug.cgi?id=43349 (June 7th 2012) > > > > [3] https://lkml.org/lkml/2012/10/16/518 (October 19th 2012) > > > > > > > > In [1] Andreas proposed a patch that initialized the missing > > > > power_usage > > > > values from within acpi_idle in the same way as cpuidle does. > > > > In [2] I proposed a patch to use the power values provided by ACPI to > > > > initialize the power_usage variables. > > > > In [3] Julius proposed a patch to call the initialization function > > > > set_power_states() not only once, but always when the C-states > > > > change. > > > > > > > > Currently, Daniel Lezcano seems to be working on an intrusive change > > > > of > > > > not > > > > using the power_usage value at all for choosing a C-state: > > > > > > > > [4] https://lkml.org/lkml/2012/12/14/155 > > > > > > > > As I could not find any of these patches in any git trees to be merged > > > > for > > > > 3.8, I propose an other, least intrusive patch for the time being. It > > > > is > > > > attached an initializes _all_ power_usage values in the first place. > > > > > > > > As this is a real power consumption regression since 3.2, I really ask > > > > you to apply anything and push it to stable, too! > > > > > > Rafael, is possible to apply the patch [1/2] I previously sent ? > > > > > > https://patchwork.kernel.org/patch/1878691/ > > > > I need to talk about this with Len. That should happen tomorrow if > > everything goes well. > > > > > So we get this bug fixed. > > > > > > I will resend the patch [2/2] as soon as possible. > > > > OK > > > > Thanks, > > Rafael --nextPart2155860.WxWkS2UFC8 Content-Disposition: inline; filename="cpuidle_3.8-rc4.txt" Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="UTF-8"; name="cpuidle_3.8-rc4.txt" /sys/devices/system/cpu/cpu0/cpuidle/state0/desc:CPUIDLE CORE POLL IDLE /sys/devices/system/cpu/cpu0/cpuidle/state0/name:POLL /sys/devices/system/cpu/cpu0/cpuidle/state0/time:0 /sys/devices/system/cpu/cpu0/cpuidle/state0/disable:0 /sys/devices/system/cpu/cpu0/cpuidle/state0/power:4294967295 /sys/devices/system/cpu/cpu0/cpuidle/state0/usage:0 /sys/devices/system/cpu/cpu0/cpuidle/state0/latency:0 /sys/devices/system/cpu/cpu0/cpuidle/state1/desc:ACPI HLT /sys/devices/system/cpu/cpu0/cpuidle/state1/name:C1 /sys/devices/system/cpu/cpu0/cpuidle/state1/time:147579 /sys/devices/system/cpu/cpu0/cpuidle/state1/disable:0 /sys/devices/system/cpu/cpu0/cpuidle/state1/power:0 /sys/devices/system/cpu/cpu0/cpuidle/state1/usage:42 /sys/devices/system/cpu/cpu0/cpuidle/state1/latency:1 /sys/devices/system/cpu/cpu0/cpuidle/state2/desc:ACPI IOPORT 0x4014 /sys/devices/system/cpu/cpu0/cpuidle/state2/name:C2 /sys/devices/system/cpu/cpu0/cpuidle/state2/time:36250374 /sys/devices/system/cpu/cpu0/cpuidle/state2/disable:0 /sys/devices/system/cpu/cpu0/cpuidle/state2/power:0 /sys/devices/system/cpu/cpu0/cpuidle/state2/usage:2867 /sys/devices/system/cpu/cpu0/cpuidle/state2/latency:1 /sys/devices/system/cpu/cpu0/cpuidle/state3/desc: /sys/devices/system/cpu/cpu0/cpuidle/state3/name: /sys/devices/system/cpu/cpu0/cpuidle/state3/time:0 /sys/devices/system/cpu/cpu0/cpuidle/state3/disable:0 /sys/devices/system/cpu/cpu0/cpuidle/state3/power:0 /sys/devices/system/cpu/cpu0/cpuidle/state3/usage:0 /sys/devices/system/cpu/cpu0/cpuidle/state3/latency:0 --nextPart2155860.WxWkS2UFC8 Content-Disposition: inline; filename="cpuidle_3.8-rc4_patched.txt" Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="UTF-8"; name="cpuidle_3.8-rc4_patched.txt" /sys/devices/system/cpu/cpu0/cpuidle/state0/desc:CPUIDLE CORE POLL IDLE /sys/devices/system/cpu/cpu0/cpuidle/state0/name:POLL /sys/devices/system/cpu/cpu0/cpuidle/state0/time:0 /sys/devices/system/cpu/cpu0/cpuidle/state0/disable:0 /sys/devices/system/cpu/cpu0/cpuidle/state0/power:4294967295 /sys/devices/system/cpu/cpu0/cpuidle/state0/usage:0 /sys/devices/system/cpu/cpu0/cpuidle/state0/latency:0 /sys/devices/system/cpu/cpu0/cpuidle/state1/desc:ACPI HLT /sys/devices/system/cpu/cpu0/cpuidle/state1/name:C1 /sys/devices/system/cpu/cpu0/cpuidle/state1/time:4644 /sys/devices/system/cpu/cpu0/cpuidle/state1/disable:0 /sys/devices/system/cpu/cpu0/cpuidle/state1/power:0 /sys/devices/system/cpu/cpu0/cpuidle/state1/usage:35 /sys/devices/system/cpu/cpu0/cpuidle/state1/latency:1 /sys/devices/system/cpu/cpu0/cpuidle/state2/desc:ACPI IOPORT 0x4014 /sys/devices/system/cpu/cpu0/cpuidle/state2/name:C2 /sys/devices/system/cpu/cpu0/cpuidle/state2/time:44430 /sys/devices/system/cpu/cpu0/cpuidle/state2/disable:0 /sys/devices/system/cpu/cpu0/cpuidle/state2/power:0 /sys/devices/system/cpu/cpu0/cpuidle/state2/usage:52 /sys/devices/system/cpu/cpu0/cpuidle/state2/latency:1 /sys/devices/system/cpu/cpu0/cpuidle/state3/desc:ACPI IOPORT 0x4016 /sys/devices/system/cpu/cpu0/cpuidle/state3/name:C3 /sys/devices/system/cpu/cpu0/cpuidle/state3/time:25798363 /sys/devices/system/cpu/cpu0/cpuidle/state3/disable:0 /sys/devices/system/cpu/cpu0/cpuidle/state3/power:0 /sys/devices/system/cpu/cpu0/cpuidle/state3/usage:1826 /sys/devices/system/cpu/cpu0/cpuidle/state3/latency:146 --nextPart2155860.WxWkS2UFC8-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/