From mboxrd@z Thu Jan 1 00:00:00 1970 From: Corentin Chary Subject: Re: Possible bug in eeepc-laptop.c - EeePC 900 Date: Sat, 13 Mar 2010 10:16:19 +0100 Message-ID: <71cd59b01003130116h4265769v9e1dae95361629ce@mail.gmail.com> References: <71cd59b01003120049n1a9fbbf3qfd37f14f241a7f64@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from fg-out-1718.google.com ([72.14.220.155]:51788 "EHLO fg-out-1718.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933358Ab0CMJQV convert rfc822-to-8bit (ORCPT ); Sat, 13 Mar 2010 04:16:21 -0500 In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Fabio Comolli Cc: ACPI mailing list , Linux Kernel Mailing List , Len Brown On Fri, Mar 12, 2010 at 9:07 PM, Fabio Comolli wrote: > Hi > > On Fri, Mar 12, 2010 at 9:49 AM, Corentin Chary > wrote: >> On Thu, Mar 11, 2010 at 10:28 PM, Fabio Comolli wrote: >>> Hi. >>> >>> On Tue, Mar 9, 2010 at 11:58 PM, Fabio Comolli wrote: >>>> Hi. >>>> I have an EeePC 900 running 2.6.34-rc1. >>>> >>>> If I boot it on AC the cpu runs at full speed, 900MHz; if I boot i= t on >>>> battery it runs only at 630Mhz. Plugging / unplugging the AC does = not >>>> change the cpu frequency. Only a reboot can change the situation. >>>> >>>> I already tried to echo 0 or 1 to the >>>> /sys/devices/platform/eeepc/cpufv file; no effects, even if the fi= le >>>> changes its value. >>>> >>>> This is not a regression from 2.6.33: this behavior is also presen= t in >>>> that version. >>>> >>>> Does this ring any bells? This is really annoying, especially when >>>> trying to watch a movie on battery. Also 3D apps show a 30% >>>> performance drop, as expected. >>>> >>>> Regards, >>>> Fabio >>> >>> Well, it turns out that this is indeed a regression, but I don't kn= ow >>> from which kernel version. >>> I reverted (not cleanly) this patch: >>> >>> =A0 =A0 =A0 =A0http://patchwork.kernel.org/patch/28591 >>> >>> and now >>> >>> =A0 =A0 =A0 =A0echo 1 > /sys/devices/platform/eeepc/cpufv >>> >>> enables the powersave mode and >>> >>> =A0 =A0 =A0 =A0echo 0 > /sys/devices/platform/eeepc/cpufv >>> >>> enables the performance mode. >>> >>> Tested with the non-benchmark glxgears (275 frames/sec in powersave >>> mode and 405 in performance mode) and stellarium (14 frames and 20 >>> frames). >>> >>> Regards, >>> Fabio >>> >> >> Thanks for the report, >> Could you try to add a quick printk to show the value stored in >> set_acpi(CM_ASL_CPUFV, value); ? >> And also, could you send the result of acpidump ? >> Thanks, > > After the revert there is no set_acpi(CM_ASL_CPUFV, value) anymore. > Please find the acpidump in attachment. > > Regards, > Fabio Yep, I know, but un-revert the patch, and add a printk :p Could you also send the result of: /* clean boot */ cat cpufv echo 0 > cpufv cat cpufv echo 1 > cpufv cat cpufv Thanks, --=20 Corentin Chary http://xf.iksaif.net -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html