linux-kernel - RE: [PATCH] thermal: trace: migrating thermal traces to use TRACE_DEFINE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <744357E9AAD1214791ACBA4B0B9092633A972721@SHSMSX101.ccr.corp.intel.com>
Date:	Tue, 8 Mar 2016 09:48:42 +0000
From:	"Zhang, Rui" <rui.zhang@...el.com>
To:	Michele DiGiorgio <michele.digiorgio@....com>,
	Steven Rostedt <rostedt@...dmis.org>
CC:	"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
	Ingo Molnar <mingo@...hat.com>,
	Javi Merino <javi.merino@....com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Punit Agrawal <punit.agrawal@....com>,
	Eduardo Valentin <edubezval@...il.com>
Subject: RE: [PATCH] thermal: trace: migrating thermal traces to use
 TRACE_DEFINE_ENUM() macros

Punit and Javi,

What do you think of this? I'd like to get your ACK before taking it.

Thanks,
rui

>  -----Original Message-----
> From: linux-pm-owner@...r.kernel.org [mailto:linux-pm-
> owner@...r.kernel.org] On Behalf Of Michele DiGiorgio
> Sent: Friday, March 04, 2016 1:11 AM
> To: Steven Rostedt <rostedt@...dmis.org>
> Cc: linux-pm@...r.kernel.org; Ingo Molnar <mingo@...hat.com>; Javi
> Merino <javi.merino@....com>; linux-kernel@...r.kernel.org; Punit
> Agrawal <punit.agrawal@....com>; Eduardo Valentin <edubezval@...il.com>
> Subject: Re: [PATCH] thermal: trace: migrating thermal traces to use
> TRACE_DEFINE_ENUM() macros
> 
> Hello Steven,
> 
> Thanks for acknowledging the patch.
> 
> Will you be merging it or should I refer to somebody else?
> 
> Kind regards,
> Michele
> 
> On 01/03/16 18:20, Steven Rostedt wrote:
> > On Tue,  1 Mar 2016 17:38:54 +0000
> > Michele Di Giorgio <michele.digiorgio@....com> wrote:
> >
> > > Userspace tools are not aware of how to convert the enums provided
> > > by the tracepoints to their corresponding strings.
> > >
> > > Adding TRACE_DEFINE_ENUM() macros allows to make the enums available
> > > to userspace to let the tools know what those enum values represent.
> > >
> > > In particular, for thermal zone trip types what we obtained before
> > > was something like:
> > >
> > > kworker/1:1-460   [001]   320.372732: thermal_zone_trip:
> thermal_zone=soc
> > >                 id=0 trip=1 trip_type=1
> > >
> > > Unfortunately, userspace tools do not know how to convert enum
> > > values to strings and as a consequence they can only forward the
> > > enum value to the output. By using TRACE_DEFINE_ENUM() macros for
> > > thermal traces we get the following trace line:
> > >
> > > kworker/1:1-460   [001]   320.372732: thermal_zone_trip:
> thermal_zone=soc
> > >                 id=0 trip=1 trip_type=PASSIVE
> > >
> > > Userspace tools are now able to better understand the meaning of the
> > > trip_type and provide the user with more readable information.
> > >
> >
> > Acked-by: Steven Rostedt <rostedt@...dmis.org>
> >
> > -- Steve
> >
> 
> IMPORTANT NOTICE: The contents of this email and any attachments are
> confidential and may also be privileged. If you are not the intended recipient,
> please notify the sender immediately and do not disclose the contents to any
> other person, use it for any purpose, or store or copy the information in any
> medium. Thank you.
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body
> of a message to majordomo@...r.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html