linux-kernel - Re: [PATCH 03/16] perf session: Add perf_session__deliver_synth

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]

Date:	Fri, 24 Oct 2014 13:41:53 +0300
From:	Adrian Hunter <adrian.hunter@...el.com>
To:	Namhyung Kim <namhyung@...nel.org>
CC:	Arnaldo Carvalho de Melo <acme@...nel.org>,
	Peter Zijlstra <peterz@...radead.org>,
	linux-kernel@...r.kernel.org, David Ahern <dsahern@...il.com>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Jiri Olsa <jolsa@...hat.com>,
	Paul Mackerras <paulus@...ba.org>,
	Stephane Eranian <eranian@...gle.com>
Subject: Re: [PATCH 03/16] perf session: Add perf_session__deliver_synth_event()

On 24/10/14 08:22, Namhyung Kim wrote:
> On Thu, 23 Oct 2014 13:45:11 +0300, Adrian Hunter wrote:
>> Add a function to deliver synthesized events from
>> within a session.
>>
>> Signed-off-by: Adrian Hunter <adrian.hunter@...el.com>
>> ---
>>  tools/perf/util/session.c | 14 ++++++++++++++
>>  tools/perf/util/session.h |  5 +++++
>>  2 files changed, 19 insertions(+)
>>
>> diff --git a/tools/perf/util/session.c b/tools/perf/util/session.c
>> index d70e37d..09635d1 100644
>> --- a/tools/perf/util/session.c
>> +++ b/tools/perf/util/session.c
>> @@ -907,6 +907,20 @@ static s64 perf_session__process_user_event(struct perf_session *session,
>>  	}
>>  }
>>  
>> +int perf_session__deliver_synth_event(struct perf_session *session,
>> +				      union perf_event *event,
>> +				      struct perf_sample *sample,
>> +				      struct perf_tool *tool)
>> +{
>> +	events_stats__inc(&session->stats, event->header.type);
>> +
>> +	if (event->header.type >= PERF_RECORD_USER_TYPE_START)
>> +		return perf_session__process_user_event(session, event, tool,
>> +							0);
>> +
>> +	return perf_session__deliver_event(session, event, sample, tool, 0);
>> +}
> 
> It seems that it's basically same as what perf_session__process_event()
> does.  So it might deserve a comment or better changelog why it's needed.

Intel PT decoding works by synthesizing events (primarily branch events)
that can then be consumed by existing tools.  This function will be used to
deliver those events.

> 
> Thanks,
> Namhyung
> 
> 
>> +
>>  static void event_swap(union perf_event *event, bool sample_id_all)
>>  {
>>  	perf_event__swap_op swap;
>> diff --git a/tools/perf/util/session.h b/tools/perf/util/session.h
>> index d8521ac..dc26ebf 100644
>> --- a/tools/perf/util/session.h
>> +++ b/tools/perf/util/session.h
>> @@ -127,6 +127,11 @@ extern volatile int session_done;
>>  
>>  #define session_done()	ACCESS_ONCE(session_done)
>>  
>> +int perf_session__deliver_synth_event(struct perf_session *session,
>> +				      union perf_event *event,
>> +				      struct perf_sample *sample,
>> +				      struct perf_tool *tool);
>> +
>>  int perf_event__process_id_index(struct perf_tool *tool,
>>  				 union perf_event *event,
>>  				 struct perf_session *session);
> 
> 

--
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/