summaryrefslogtreecommitdiffstats
path: root/kernel/marker.c
diff options
context:
space:
mode:
authorZhaolei <zhaolei@cn.fujitsu.com>2008-10-22 11:38:01 +0800
committerIngo Molnar <mingo@elte.hu>2008-10-27 13:02:23 +0100
commit5d9881ea1440f046ee851bbaa2a2962543336a11 (patch)
tree28b3797d8bf356f288b4f9025925e617f599ac9c /kernel/marker.c
parent4de62748e69c31fc4fd5bc43b73e9cf60a17ec53 (diff)
downloadop-kernel-dev-5d9881ea1440f046ee851bbaa2a2962543336a11.zip
op-kernel-dev-5d9881ea1440f046ee851bbaa2a2962543336a11.tar.gz
markers: break the redundant loop in kernel/marker.c
Impact: cleanup, no functionality changed Because e->name is unique in list, we don't need to continue the iteration after matched. Signed-off-by: Zhao Lei <zhaolei@cn.fujitsu.com> Acked-by: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/marker.c')
-rw-r--r--kernel/marker.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/kernel/marker.c b/kernel/marker.c
index 0f2a944..2898b64 100644
--- a/kernel/marker.c
+++ b/kernel/marker.c
@@ -825,8 +825,6 @@ void *marker_get_private_data(const char *name, marker_probe_func *probe,
if (!e->ptype) {
if (num == 0 && e->single.func == probe)
return e->single.probe_private;
- else
- break;
} else {
struct marker_probe_closure *closure;
int match = 0;
@@ -838,6 +836,7 @@ void *marker_get_private_data(const char *name, marker_probe_func *probe,
return closure[i].probe_private;
}
}
+ break;
}
}
return ERR_PTR(-ENOENT);
OpenPOWER on IntegriCloud