diff options
| author | Li Zefan <lizf@cn.fujitsu.com> | 2008-04-01 12:29:34 +0800 |
|---|---|---|
| committer | Tony Luck <tony.luck@intel.com> | 2008-04-09 10:17:03 -0700 |
| commit | 6794c7526651160a75e90322cb750dcceb310d34 (patch) | |
| tree | cd11dabc42e0682352f7c91679f646e1b75c6f6b | |
| parent | e91450161186a926d16d8fdc8669aa1998bce148 (diff) | |
[IA64] use goto to jump out do/while_each_thread
do_each_thread/while_each_thread is a double loop, so
should use 'goto' rather than 'break' to break out
the loop.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
| -rw-r--r-- | arch/ia64/kernel/perfmon.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/ia64/kernel/perfmon.c b/arch/ia64/kernel/perfmon.c index a2aabfdc80d..d1d24f4598d 100644 --- a/arch/ia64/kernel/perfmon.c +++ b/arch/ia64/kernel/perfmon.c @@ -4204,10 +4204,10 @@ pfm_check_task_exist(pfm_context_t *ctx) do_each_thread (g, t) { if (t->thread.pfm_context == ctx) { ret = 0; - break; + goto out; } } while_each_thread (g, t); - +out: read_unlock(&tasklist_lock); DPRINT(("pfm_check_task_exist: ret=%d ctx=%p\n", ret, ctx)); |