Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752432AbbH1QSM (ORCPT ); Fri, 28 Aug 2015 12:18:12 -0400 Received: from mail.kernel.org ([198.145.29.136]:56149 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751505AbbH1QSK (ORCPT ); Fri, 28 Aug 2015 12:18:10 -0400 Date: Fri, 28 Aug 2015 13:17:48 -0300 From: Arnaldo Carvalho de Melo To: Masami Hiramatsu Cc: Peter Zijlstra , linux-kernel@vger.kernel.org, Adrian Hunter , Ingo Molnar , Paul Mackerras , Jiri Olsa , Namhyung Kim , Borislav Petkov , Hemant Kumar Subject: Re: [RFC PATCH perf/core v3 05/17] perf-buildid-cache: Use lsdir for looking up buildid caches Message-ID: <20150828161748.GD11407@kernel.org> References: <20150815114252.13642.62690.stgit@localhost.localdomain> <20150815114303.13642.62726.stgit@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150815114303.13642.62726.stgit@localhost.localdomain> X-Url: http://acmel.wordpress.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 Content-Length: 1903 Lines: 69 Em Sat, Aug 15, 2015 at 08:43:04PM +0900, Masami Hiramatsu escreveu: > 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. It is not applying to my perf/core branch, please check. This is after I applied that list_dir() helper. - Arnaldo > Signed-off-by: Masami Hiramatsu > --- > tools/perf/util/build-id.c | 30 +++++------------------------- > 1 file changed, 5 insertions(+), 25 deletions(-) > > diff --git a/tools/perf/util/build-id.c b/tools/perf/util/build-id.c > index b639cad..e9ef98e 100644 > --- a/tools/perf/util/build-id.c > +++ b/tools/perf/util/build-id.c > @@ -342,38 +342,18 @@ int build_id_cache__list_build_ids(const char *pathname, > { > 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, > NULL); > - if (!list || !dir_name) { > - ret = -ENOMEM; > - goto out; > - } > + if (!dir_name) > + return -ENOMEM; > > - /* List up all dirents */ > - dir = opendir(dir_name); > - if (!dir) { > + list = lsdir(dir_name, lsdir_no_dot_filter); > + if (!list) > 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: > + *result = list; > free(dir_name); > - if (ret) > - strlist__delete(list); > - else > - *result = list; > > return ret; > } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/