* [PATCH 2/2] perf core: remove short name compare in dsos__find()
2013-09-05 2:59 [PATCH 1/2] perf core: avoid traverse dsos list while find vdso Chenggang Qin
@ 2013-09-05 2:59 ` Chenggang Qin
2013-09-09 12:05 ` [PATCH 1/2] perf core: avoid traverse dsos list while find vdso Peter Zijlstra
1 sibling, 0 replies; 3+ messages in thread
From: Chenggang Qin @ 2013-09-05 2:59 UTC (permalink / raw)
To: linux-kernel
Cc: Chenggang Qin, David Ahern, Peter Zijlstra, Paul Mackerras,
Ingo Molnar, Arnaldo Carvalho de Melo, Arjan van de Ven,
Namhyung Kim, Yanmin Zhang, Wu Fengguang, Mike Galbraith,
Andrew Morton
From: Chenggang Qin <chenggang.qcg@taobao.com>
If the list traversal is avoided by the last patch, the short name compare in
dsos__find() is unnecessary. The purpose of short name compare is only to find
the dso of vdso. If the vdso can be found by a pointor, the short name compare
can be removed.
Thanks
Cc: David Ahern <dsahern@gmail.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Arnaldo Carvalho de Melo <acme@ghostprotocols.net>
Cc: Arjan van de Ven <arjan@linux.intel.com>
Cc: Namhyung Kim <namhyung@gmail.com>
Cc: Yanmin Zhang <yanmin.zhang@intel.com>
Cc: Wu Fengguang <fengguang.wu@intel.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Chenggang Qin <chenggang.qcg@taobao.com>
---
tools/perf/util/dso.c | 10 ++--------
tools/perf/util/dso.h | 3 +--
2 files changed, 3 insertions(+), 10 deletions(-)
diff --git a/tools/perf/util/dso.c b/tools/perf/util/dso.c
index c4374f0..6f7d5a9 100644
--- a/tools/perf/util/dso.c
+++ b/tools/perf/util/dso.c
@@ -513,16 +513,10 @@ void dsos__add(struct list_head *head, struct dso *dso)
list_add_tail(&dso->node, head);
}
-struct dso *dsos__find(struct list_head *head, const char *name, bool cmp_short)
+struct dso *dsos__find(struct list_head *head, const char *name)
{
struct dso *pos;
- if (cmp_short) {
- list_for_each_entry(pos, head, node)
- if (strcmp(pos->short_name, name) == 0)
- return pos;
- return NULL;
- }
list_for_each_entry(pos, head, node)
if (strcmp(pos->long_name, name) == 0)
return pos;
@@ -531,7 +525,7 @@ struct dso *dsos__find(struct list_head *head, const char *name, bool cmp_short)
struct dso *__dsos__findnew(struct list_head *head, const char *name)
{
- struct dso *dso = dsos__find(head, name, false);
+ struct dso *dso = dsos__find(head, name);
if (!dso) {
dso = dso__new(name);
diff --git a/tools/perf/util/dso.h b/tools/perf/util/dso.h
index d51aaf2..450199a 100644
--- a/tools/perf/util/dso.h
+++ b/tools/perf/util/dso.h
@@ -133,8 +133,7 @@ struct dso *dso__kernel_findnew(struct machine *machine, const char *name,
const char *short_name, int dso_type);
void dsos__add(struct list_head *head, struct dso *dso);
-struct dso *dsos__find(struct list_head *head, const char *name,
- bool cmp_short);
+struct dso *dsos__find(struct list_head *head, const char *name);
struct dso *__dsos__findnew(struct list_head *head, const char *name);
bool __dsos__read_build_ids(struct list_head *head, bool with_hits);
--
1.7.8.rc2.5.g815b
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH 1/2] perf core: avoid traverse dsos list while find vdso
2013-09-05 2:59 [PATCH 1/2] perf core: avoid traverse dsos list while find vdso Chenggang Qin
2013-09-05 2:59 ` [PATCH 2/2] perf core: remove short name compare in dsos__find() Chenggang Qin
@ 2013-09-09 12:05 ` Peter Zijlstra
1 sibling, 0 replies; 3+ messages in thread
From: Peter Zijlstra @ 2013-09-09 12:05 UTC (permalink / raw)
To: Chenggang Qin
Cc: linux-kernel, Chenggang Qin, David Ahern, Paul Mackerras,
Ingo Molnar, Arnaldo Carvalho de Melo, Arjan van de Ven,
Namhyung Kim, Yanmin Zhang, Wu Fengguang, Mike Galbraith,
Andrew Morton
> Subject: [PATCH 1/2] perf core: avoid traverse dsos list while find vdso
> tools/perf/util/vdso.c | 22 ++++++++--------------
That's not perf core that's perf tools.
^ permalink raw reply [flat|nested] 3+ messages in thread