linux-kernel - [RFC 0/4] perf tools: Remove event types data

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-next>] [day] [month] [year] [list]

Date:	Tue,  9 Jul 2013 18:48:55 +0200
From:	Jiri Olsa <jolsa@...hat.com>
To:	linux-kernel@...r.kernel.org
Cc:	Jiri Olsa <jolsa@...hat.com>,
	Arnaldo Carvalho de Melo <acme@...stprotocols.net>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	Ingo Molnar <mingo@...e.hu>, Paul Mackerras <paulus@...ba.org>,
	Corey Ashford <cjashfor@...ux.vnet.ibm.com>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Namhyung Kim <namhyung@...nel.org>,
	David Ahern <dsahern@...il.com>,
	Thomas Renninger <trenn@...e.de>
Subject: [RFC 0/4] perf tools: Remove event types data

hi,
following up on the 'perf timechart' FIXME note and changing
its tracepoint match not to use event types data.

In my tests the old and new timechart outputs look the same,
so.. any other tester would be appreciated ;-)

Also as this was the only user of the event types data
the rest of the patchset is removing it out of the perf.

The event types data are referenced from the perf data
file header. The reference (offset & size) stays in the
header with 0s.

Also the event types data is already duplicated via the
event_desc FEATURE, so there's no information loss.

thanks for comments,
jirka


Signed-off-by: Jiri Olsa <jolsa@...hat.com>
Cc: Arnaldo Carvalho de Melo <acme@...stprotocols.net>
Cc: Peter Zijlstra <a.p.zijlstra@...llo.nl>
Cc: Ingo Molnar <mingo@...e.hu>
Cc: Paul Mackerras <paulus@...ba.org>
Cc: Corey Ashford <cjashfor@...ux.vnet.ibm.com>
Cc: Frederic Weisbecker <fweisbec@...il.com>
Cc: Namhyung Kim <namhyung@...nel.org>
Cc: David Ahern <dsahern@...il.com>
Cc: Thomas Renninger <trenn@...e.de>
---
 tools/perf/builtin-inject.c    |   7 ------
 tools/perf/builtin-record.c    |  13 -----------
 tools/perf/builtin-report.c    |   1 -
 tools/perf/builtin-script.c    |   1 -
 tools/perf/builtin-timechart.c |  94 ++++++++++++++++++++++++++++++++++++++++++-------------------------------------
 tools/perf/util/event.h        |   2 +-
 tools/perf/util/header.c       | 118 +---------------------------------------------------------------------------------------------------
 tools/perf/util/header.h       |  16 +-------------
 tools/perf/util/session.c      |  11 ----------
 tools/perf/util/tool.h         |   3 ---
 10 files changed, 54 insertions(+), 212 deletions(-)
--
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/