Re: [PATCH 5/9] perf utils: add support for arch standard events

From: Jiri Olsa
Date: Thu Feb 08 2018 - 08:55:41 EST


On Wed, Feb 07, 2018 at 01:45:00AM +0800, John Garry wrote:

SNIP

>
> +static int is_json_file(const char *name)
> +{
> + const char *suffix;
> +
> + if (strlen(name) < 5)
> + return 0;
> +
> + suffix = name + strlen(name) - 5;
> +
> + if (strncmp(suffix, ".json", 5) == 0)
> + return 1;
> + return 0;
> +}
> +
> +static int preprocess_arch_std_files(const char *fpath, const struct stat *sb,
> + int typeflag, struct FTW *ftwbuf)
> +{
> + int level = ftwbuf->level;
> + int is_file = typeflag == FTW_F;
> +
> + if (level == 1 && is_file && is_json_file(fpath))
> + return json_events(fpath, save_arch_std_events, (void *)sb);

so any .json file will pass.. just wondering you'd want to put
some name restriction for recomended events file like this -recomended
suffix you used later.. but that can be added later in case we'll
need some other json files in here ;-)

jirka