From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Olsa Subject: Re: [PATCH 57/66] perf evsel: Introduce evsel_fprintf.h Date: Thu, 26 Sep 2019 13:15:18 +0200 Message-ID: <20190926111518.GC24257@krava> References: <20190926003244.13962-1-acme@kernel.org> <20190926003244.13962-58-acme@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20190926003244.13962-58-acme@kernel.org> Sender: linux-kernel-owner@vger.kernel.org To: Arnaldo Carvalho de Melo Cc: Ingo Molnar , Thomas Gleixner , Jiri Olsa , Namhyung Kim , Clark Williams , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Arnaldo Carvalho de Melo , Adrian Hunter List-Id: linux-perf-users.vger.kernel.org On Wed, Sep 25, 2019 at 09:32:35PM -0300, Arnaldo Carvalho de Melo wrote: > From: Arnaldo Carvalho de Melo > > We already had evsel_fprintf.c, add its counterpart, so that we can > reduce evsel.h a bit more. > > We needed a new perf_event_attr_fprintf.c file so as to have a separate > object to link with the python binding in tools/perf/util/python-ext-sources > and not drag symbol_conf, etc into the python binding. > > Cc: Adrian Hunter > Cc: Jiri Olsa > Cc: Namhyung Kim > Link: https://lkml.kernel.org/n/tip-06bdmt1062d9unzgqmxwlv88@git.kernel.org > Signed-off-by: Arnaldo Carvalho de Melo > --- > tools/perf/builtin-evlist.c | 1 + > tools/perf/builtin-sched.c | 1 + > tools/perf/builtin-script.c | 1 + > tools/perf/builtin-trace.c | 2 + > tools/perf/util/Build | 1 + > tools/perf/util/evsel.c | 153 +--------------------- > tools/perf/util/evsel.h | 51 +------- > tools/perf/util/evsel_fprintf.c | 1 + > tools/perf/util/evsel_fprintf.h | 50 +++++++ > tools/perf/util/header.c | 1 + > tools/perf/util/perf_event_attr_fprintf.c | 148 +++++++++++++++++++++ > tools/perf/util/python-ext-sources | 1 + > 12 files changed, 218 insertions(+), 193 deletions(-) > create mode 100644 tools/perf/util/evsel_fprintf.h > create mode 100644 tools/perf/util/perf_event_attr_fprintf.c hum, I see this file in this patch, but not when I checkout yours perf/core jirka