diff options
author | Zhaolei <zhaolei@cn.fujitsu.com> | 2008-10-21 23:38:01 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-10-27 08:02:23 -0400 |
commit | 5d9881ea1440f046ee851bbaa2a2962543336a11 (patch) | |
tree | 28b3797d8bf356f288b4f9025925e617f599ac9c /kernel/marker.c | |
parent | 4de62748e69c31fc4fd5bc43b73e9cf60a17ec53 (diff) |
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.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/kernel/marker.c b/kernel/marker.c index 0f2a944329d3..2898b647d415 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, | |||
825 | if (!e->ptype) { | 825 | if (!e->ptype) { |
826 | if (num == 0 && e->single.func == probe) | 826 | if (num == 0 && e->single.func == probe) |
827 | return e->single.probe_private; | 827 | return e->single.probe_private; |
828 | else | ||
829 | break; | ||
830 | } else { | 828 | } else { |
831 | struct marker_probe_closure *closure; | 829 | struct marker_probe_closure *closure; |
832 | int match = 0; | 830 | int match = 0; |
@@ -838,6 +836,7 @@ void *marker_get_private_data(const char *name, marker_probe_func *probe, | |||
838 | return closure[i].probe_private; | 836 | return closure[i].probe_private; |
839 | } | 837 | } |
840 | } | 838 | } |
839 | break; | ||
841 | } | 840 | } |
842 | } | 841 | } |
843 | return ERR_PTR(-ENOENT); | 842 | return ERR_PTR(-ENOENT); |