lkml.org 
[lkml]   [2016]   [May]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH perf/core v7 06/21] perf-buildid-cache: Use lsdir for looking up buildid caches
Em Wed, May 11, 2016 at 10:52:17PM +0900, Masami Hiramatsu escreveu:
> From: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
>
> Use new lsdir() for looking up buildid caches. This changes
> logic a bit to ignore all dot files, since the build-id
> cache must not start with dot.

Applied

> Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
> Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org>
> ---
> Changes in v7:
> - Remove unneeded local strlist.
> ---
> tools/perf/util/build-id.c | 30 ++++--------------------------
> 1 file changed, 4 insertions(+), 26 deletions(-)
>
> diff --git a/tools/perf/util/build-id.c b/tools/perf/util/build-id.c
> index 53df9aa..2cb6454 100644
> --- a/tools/perf/util/build-id.c
> +++ b/tools/perf/util/build-id.c
> @@ -369,39 +369,17 @@ static char *build_id_cache__dirname_from_path(const char *name,
> int build_id_cache__list_build_ids(const char *pathname,
> struct strlist **result)
> {
> - struct strlist *list;
> char *dir_name;
> - DIR *dir;
> - struct dirent *d;
> int ret = 0;
>
> - list = strlist__new(NULL, NULL);
> dir_name = build_id_cache__dirname_from_path(pathname, false, false);
> - if (!list || !dir_name) {
> - ret = -ENOMEM;
> - goto out;
> - }
> + if (!dir_name)
> + return -ENOMEM;
>
> - /* List up all dirents */
> - dir = opendir(dir_name);
> - if (!dir) {
> + *result = lsdir(dir_name, lsdir_no_dot_filter);
> + if (!*result)
> ret = -errno;
> - goto out;
> - }
> -
> - while ((d = readdir(dir)) != NULL) {
> - if (!strcmp(d->d_name, ".") || !strcmp(d->d_name, ".."))
> - continue;
> - strlist__add(list, d->d_name);
> - }
> - closedir(dir);
> -
> -out:
> free(dir_name);
> - if (ret)
> - strlist__delete(list);
> - else
> - *result = list;
>
> return ret;
> }

\
 
 \ /
  Last update: 2016-05-11 18:21    [W:1.055 / U:0.704 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site