From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932935AbbLCJMm (ORCPT ); Thu, 3 Dec 2015 04:12:42 -0500 Received: from mx1.redhat.com ([209.132.183.28]:52846 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932663AbbLCJGt (ORCPT ); Thu, 3 Dec 2015 04:06:49 -0500 From: Jiri Olsa To: Arnaldo Carvalho de Melo Cc: lkml , David Ahern , Ingo Molnar , Namhyung Kim , Peter Zijlstra , Adrian Hunter Subject: [PATCH 0/7] perf stat: Change event enable code Date: Thu, 3 Dec 2015 10:06:39 +0100 Message-Id: <1449133606-14429-1-git-send-email-jolsa@kernel.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org hi, while testing ftrace:function event I noticed we create stat counters as enabled (except for enable_on_exec couters). This way we count also filter setup and other config code which might be crucial for some events. Posponing the events enable once everything is ready. The last patch is RFC as I wasn't sure there's some hidden catch about perf_evlist__(enable|disable)_event functions I missed.. Adrian? thanks, jirka --- Jiri Olsa (7): perf tools: Use event maps directly in perf_evsel__enable perf tools: Introduce perf_evsel__disable function perf tools: Factor perf_evlist__(enable|disable) functions perf stat: Use perf_evlist__enable in handle_initial_delay perf stat: Create events as disabled perf stat: Move enable_on_exec setup under earlier code perf tools: Remove perf_evlist__(enable|disable)_event functions tools/perf/arch/x86/util/intel-bts.c | 4 ++-- tools/perf/arch/x86/util/intel-pt.c | 4 ++-- tools/perf/builtin-stat.c | 44 +++++++++++++++++++++++++++----------------- tools/perf/tests/keep-tracking.c | 2 +- tools/perf/tests/switch-tracking.c | 6 +++--- tools/perf/util/evlist.c | 74 ++++++++------------------------------------------------------------------ tools/perf/util/evlist.h | 4 ---- tools/perf/util/evsel.c | 15 ++++++++++++++- tools/perf/util/evsel.h | 3 ++- 9 files changed, 59 insertions(+), 97 deletions(-)