Re: [Powertop] [PATCH v1] Allow frequency stats when cpuidle is not enabled

2012-12-12 Thread Chris Ferron
On 12/03/2012 10:07 PM, Rajagopal Venkat wrote: Powertop fails to display frequency stats when cpuidle subsystem is not enabled. Fix it. Signed-off-by: Rajagopal Venkat --- src/cpu/cpu.h | 7 ++- src/cpu/cpu_linux.cpp | 36 +++- 2 files changed,

Re: [Powertop] [PATCH v2 2/2] Add stubs to support Android platform

2012-09-28 Thread Chris Ferron
On 09/28/2012 10:10 AM, Sergey Senozhatsky wrote: On (09/28/12 09:54), Chris Ferron wrote: [..] Well to be fair, not that I agree any-more, but years ago it was common practice to disable exceptions (via the compiler) for C++ in very specialized "REAL" "Embedded Systems"

Re: [Powertop] [PATCH v2 2/2] Add stubs to support Android platform

2012-09-28 Thread Chris Ferron
On 09/26/2012 04:00 PM, Sergey Senozhatsky wrote: On (09/27/12 00:09), Magnus Fromreide wrote: That they fail to throw exceptions from new is no reason to disable set_new_handler, the newhandler is called by the runtime on out of memory and is intended to allow the user to try fixing the issue.

Re: [Powertop] [PATCH v2 1/2] Updates to support Android platform

2012-09-25 Thread Chris Ferron
On 09/24/2012 10:34 PM, Rajagopal Venkat wrote: On 24 September 2012 21:27, Chris Ferron wrote: On 09/24/2012 06:28 AM, Rajagopal Venkat wrote: This patch adds following minor changes to prepare powertop to support Android platform. - Add missing HAVE_CONFIG_H conditional check. - remove un

Re: [Powertop] [PATCH v2 1/2] Updates to support Android platform

2012-09-24 Thread Chris Ferron
On 09/24/2012 06:28 AM, Rajagopal Venkat wrote: This patch adds following minor changes to prepare powertop to support Android platform. - Add missing HAVE_CONFIG_H conditional check. - remove un-used ethtool_cmd_speed_set and ethtool_cmd_speed functions. - Minimize dependency on exception handl

Re: [Powertop] [PATCH] Fix timer and work perf events timestamp tracing

2012-09-05 Thread Chris Ferron
On 09/05/2012 05:44 AM, Sergey Senozhatsky wrote: Hi, On (09/05/12 15:52), Rajagopal Venkat wrote: Incorrect timer and work perf events timestamp tracing is one of the reason for reporting usage over 100%. This patch will resolve the issue by - rejecting the events for which entry timestamp is

Re: [Powertop] [PATCH] fix cpuidle state name parsing

2012-07-31 Thread Chris Ferron
> This patch fixes powertop to display cpuidle states on platforms > where cpuidle stateX directory name field does not contain > "CX" string. On some platforms, the name field contains meaningful > strings like WFI, Sleep, DeepSleep. > > Please review the patch. > Im not sure this is doing what yo

Re: [Powertop] [PATCH v3] conditionally disable pci if not supported

2012-07-19 Thread Chris Ferron
On 06/25/2012 10:14 AM, Rajagopal Venkat wrote: disable pci if not supported by host platform. When libpci is not found, define a macro which will be used for conditional compilation. Signed-off-by: Rajagopal Venkat --- configure.ac |5 +++-- src/lib.cpp | 20 ++-- sr