Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752769Ab3GIQun (ORCPT ); Tue, 9 Jul 2013 12:50:43 -0400 Received: from mx1.redhat.com ([209.132.183.28]:48154 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752542Ab3GIQuX (ORCPT ); Tue, 9 Jul 2013 12:50:23 -0400 From: Jiri Olsa To: linux-kernel@vger.kernel.org Cc: Jiri Olsa , Arnaldo Carvalho de Melo , Peter Zijlstra , Ingo Molnar , Paul Mackerras , Corey Ashford , Frederic Weisbecker , Namhyung Kim , David Ahern , Thomas Renninger Subject: [RFC 0/4] perf tools: Remove event types data Date: Tue, 9 Jul 2013 18:48:55 +0200 Message-Id: <1373388539-9347-1-git-send-email-jolsa@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1942 Lines: 48 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 Cc: Arnaldo Carvalho de Melo Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Paul Mackerras Cc: Corey Ashford Cc: Frederic Weisbecker Cc: Namhyung Kim Cc: David Ahern Cc: Thomas Renninger --- 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/