[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110312044010.GB3634@zhy>
Date: Sat, 12 Mar 2011 12:40:10 +0800
From: Yong Zhang <yong.zhang0@...il.com>
To: Thomas Renninger <trenn@...e.de>
Cc: linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org,
linux-pm@...ts.linux-foundation.org, discuss@...swatts.org,
power@...host.org, cpufreq@...r.kernel.org,
linux@...inikbrodowski.net, Mattia Dongili <malattia@...ux.it>,
Len Brown <lenb@...nel.org>, Ingo Molnar <mingo@...e.hu>,
herrmann.der.user@...glemail.com, linux-omap@...r.kernel.org,
dri-devel@...ts.freedesktop.org
Subject: Re: [ANNOUNCE] cpupowerutils - cpufrequtils extended with quite
some features
On Sat, Mar 12, 2011 at 12:34:09PM +0800, Yong Zhang wrote:
> On Fri, Mar 11, 2011 at 12:46:59PM +0100, Thomas Renninger wrote:
> > Happy testing..., if you have a recent machine, you'll like it!
> >
> > After some testing phase it would be great to get this tool
> > merged into the kernel git repo under:
> > tools/power/cpupower
> > and replace the Intel HW only supporting tools in tools/power/x86.
>
> I got below output on my laptop:
>
> |Mperf || Idle_Stats
> CPU | C0 | Cx | Freq || POLL | C1 | C2 | C3
> 0| 36.00| 64.00| 820||1843411670376227.2|1843411670376692.2| 0.00| 0.00
> 1| 44.69| 55.31| 920|| 0.00| 0.00| 0.00| 0.00
>
> Seems below patch fix it:
forget the declaration of value :)
---
--- utils/helpers/sysfs-orig.c 2011-03-12 12:32:06.179827929 +0800
+++ utils/helpers/sysfs.c 2011-03-12 12:35:21.512796546 +0800
@@ -111,7 +111,7 @@
unsigned int idlestate,
enum idlestate_value which)
{
- unsigned long value;
+ unsigned long long value;
unsigned int len;
char linebuf[MAX_LINE_LEN];
char *endp;
@@ -126,7 +126,7 @@
return 0;
}
- value = strtoul(linebuf, &endp, 0);
+ value = strtoull(linebuf, &endp, 0);
if ( endp == linebuf || errno == ERANGE )
return 0;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists