[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200210120739.GE1907700@krava>
Date: Mon, 10 Feb 2020 13:07:39 +0100
From: Jiri Olsa <jolsa@...hat.com>
To: John Garry <john.garry@...wei.com>
Cc: peterz@...radead.org, mingo@...hat.com, acme@...nel.org,
mark.rutland@....com, alexander.shishkin@...ux.intel.com,
namhyung@...nel.org, will@...nel.org, ak@...ux.intel.com,
linuxarm@...wei.com, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, suzuki.poulose@....com,
james.clark@....com, zhangshaokun@...ilicon.com,
robin.murphy@....com
Subject: Re: [PATCH RFC 3/7] perf jevents: Add support for a system events PMU
On Fri, Jan 24, 2020 at 10:35:01PM +0800, John Garry wrote:
SNIP
> + return ret;
> +}
> +
> /*
> * If we fail to locate/process JSON and map files, create a NULL mapping
> * table. This would at least allow perf to build even if we can't find/use
> @@ -887,6 +917,7 @@ static int get_maxfds(void)
> */
> static FILE *eventsfp;
> static char *mapfile;
> +static char *mapfile_sys;
>
> static int is_leaf_dir(const char *fpath)
> {
> @@ -1024,6 +1055,11 @@ static int process_one_file(const char *fpath, const struct stat *sb,
> return 0;
> }
>
> + if (!strcmp(bname, "mapfile_sys.csv")) {
> + mapfile_sys = strdup(fpath);
we could release that in the cleanup code at the end of main
together with 'mapfile', which is also missing
jirka
Powered by blists - more mailing lists