From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Renninger Subject: Re: [PATCH] tracing, perf: add more power related events Date: Wed, 22 Sep 2010 19:57:56 +0200 Message-ID: <201009221957.57845.trenn__38288.0881116309$1285178449$gmane$org@suse.de> References: <4C9A21AD.1000800@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4C9A21AD.1000800@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.linux-foundation.org Errors-To: linux-pm-bounces@lists.linux-foundation.org To: Arjan van de Ven , linux-pm@lists.linux-foundation.org Cc: Len Brown , linux-trace-users@vger.kernel.org, Peter Zijlstra , Jean Pihet , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Ingo Molnar , linux-omap@vger.kernel.org, arjan@infradead.org List-Id: linux-pm@vger.kernel.org On Wednesday 22 September 2010 17:33:01 Arjan van de Ven wrote: > On 9/22/2010 8:31 AM, Jean Pihet wrote: > > Hi, > > > > Here is a patch that redefines the power events API. The advantages > > are: easier maintainance of the kernel and the > > user space tools, a cleaner and more generic interface, more > > parameters for fine tracing and even documentation! > > > > Thomas, this patch has your patch above merged in ('power-trace: Use > > power_switch_state instead of power_start and power_end'). The revised > > ACPI patch is coming asap. > > > > The trace points for x86 and OMAP are also udated accordingly. > > > > The pytimechart tool needs an update for the new API. This can be done > > as soon as the kernel code gets merged in. > > unfortunately this code is changing a userspace ABI... we really > shouldn't do that if we can avoid it, > and here we can avoid it. Wow, this is your first post on this thread and you needed 1 min, 20 secs to reply and comment on work that probably took several weeks and which is quite an improvement. You did not even start to read the documentation/patch in this minute, did you? > applications ARE using this stuff! The current interfaces do not make sense at all. Things must be changed now before even more ARE using the broken stuff and things get more complicated. You should be happy that someone is helping/doing it. Thomas