diff options
| author | Zhaolei <zhaolei@cn.fujitsu.com> | 2008-10-22 11:38:01 +0800 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2008-10-27 13:02:23 +0100 |
| commit | 5d9881ea1440f046ee851bbaa2a2962543336a11 (patch) | |
| tree | 28b3797d8bf356f288b4f9025925e617f599ac9c /kernel | |
| parent | 4de62748e69c31fc4fd5bc43b73e9cf60a17ec53 (diff) | |
| download | op-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')
| -rw-r--r-- | kernel/marker.c | 3 |
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); |
