From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965273AbbHKOjW (ORCPT ); Tue, 11 Aug 2015 10:39:22 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38846 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965009AbbHKOjV (ORCPT ); Tue, 11 Aug 2015 10:39:21 -0400 Date: Tue, 11 Aug 2015 16:39:17 +0200 From: Jiri Olsa To: Andi Kleen Cc: Andi Kleen , acme@kernel.org, jolsa@kernel.org, linux-kernel@vger.kernel.org, eranian@google.com, namhyung@kernel.org, peterz@infradead.org, mingo@kernel.org Subject: Re: [PATCH 1/9] perf, tools: Dont stop PMU parsing on alias parse error Message-ID: <20150811143917.GC10703@krava.brq.redhat.com> References: <1438995985-13631-1-git-send-email-andi@firstfloor.org> <1438995985-13631-2-git-send-email-andi@firstfloor.org> <20150811130736.GA1942@krava.brq.redhat.com> <20150811131457.GA4524@tassilo.jf.intel.com> <20150811132427.GA10703@krava.brq.redhat.com> <20150811134027.GE4524@tassilo.jf.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150811134027.GE4524@tassilo.jf.intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Aug 11, 2015 at 06:40:27AM -0700, Andi Kleen wrote: > On Tue, Aug 11, 2015 at 03:24:27PM +0200, Jiri Olsa wrote: > > On Tue, Aug 11, 2015 at 06:14:57AM -0700, Andi Kleen wrote: > > > > Which attribute parsing is failing for you? > > > > > > The new .agg-per-core attribute I added later in the series. > > > I think it will happen to any not-yet-known attribute. > > > > alias can contain only terms defined in formats directory, > > and the *.XXX attributes parsing does not return error code > > > > can't see the failure, please get some example > > Apply the kernel patch that adds several .agg-per-core attributes > Then try to use any cpu/.../ event > > % perf stat -e cpu/event=0x3c/ true > invalid or unsupported event: 'cpu/event=0x3c/' > > because the PMU parsing bailed out. ugh right, the new attribute wont be recognized.. how about recognizing attribute based on the '.' prefix being existing file rather than the suffix like in the attached patch jirka --- diff --git a/tools/perf/util/pmu.c b/tools/perf/util/pmu.c index d4b0e6454bc6..937ecc35a60e 100644 --- a/tools/perf/util/pmu.c +++ b/tools/perf/util/pmu.c @@ -258,21 +258,23 @@ static int perf_pmu__new_alias(struct list_head *list, char *dir, char *name, FI return __perf_pmu__new_alias(list, dir, name, NULL, buf); } -static inline bool pmu_alias_info_file(char *name) +static inline bool pmu_alias_attr_file(char *dir, char *name) { - size_t len; - - len = strlen(name); - if (len > 5 && !strcmp(name + len - 5, ".unit")) - return true; - if (len > 6 && !strcmp(name + len - 6, ".scale")) - return true; - if (len > 8 && !strcmp(name + len - 8, ".per-pkg")) - return true; - if (len > 9 && !strcmp(name + len - 9, ".snapshot")) - return true; + bool ret = false; + struct stat st; + char *path, *s; - return false; + if (asprintf(&path, "%s/%s", dir, name) == -1) + return false; + + s = strrchr(path, '.'); + if (s) { + *s = 0; + ret = !stat(path, &st); + } + + free(path); + return ret; } /* @@ -300,7 +302,7 @@ static int pmu_aliases_parse(char *dir, struct list_head *head) /* * skip info files parsed in perf_pmu__new_alias() */ - if (pmu_alias_info_file(name)) + if (pmu_alias_attr_file(dir, name)) continue; snprintf(path, PATH_MAX, "%s/%s", dir, name);