>-----Original Message----- >From: Arjan van de Ven [mailto:[email protected]] >Sent: Friday, May 20, 2011 4:49 PM >To: Mathew, JohnX >Cc: [email protected] >Subject: Re: [PATCH] timer: ignore deferred timers > >On 5/20/2011 5:40 AM, John Mathew wrote: >> The current version of powertop does not check if a timer is >> deferred. So deferred timers will be displayed as a power consumer >> This patch utilizes the /proc/timer_stats to determine if a timer >> is deferred. If it is the timer is not considered in the powertop >> statistics. > >I actually looked at this approach before, and in non-antique kernels it >does not actually work unfortunately... >/proc/timer_stats did not list the same functions as we get for timer >handlers.
The patch is implemented such that if a handler is not found, it returns false. So it does not affect the functionality of powertop in case the timer_stats doesn't work correctly in a particular kernel, but improves the results in kernels that have timer_stats working correctly. --------------------------------------------------------------------- Intel Corporation SAS (French simplified joint stock company) Registered headquarters: "Les Montalets"- 2, rue de Paris, 92196 Meudon Cedex, France Registration Number: 302 456 199 R.C.S. NANTERRE Capital: 4,572,000 Euros This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies. _______________________________________________ Power mailing list [email protected] https://bughost.org/mailman/listinfo/power
