diff options
| author |   Peter Zijlstra <a.p.zijlstra@chello.nl> | 2009-03-13 12:21:29 +0100 |
|---|---|---|
| committer |   Ingo Molnar <mingo@elte.hu> | 2009-04-06 09:29:31 +0200 |
| commit | 755642322aa66fbc5421a35fd3e1733f73e20083 (patch) | |
| tree | cfcdd495d14de3b78cdf7f1d064e8534ea8593e1 /kernel | |
| parent | perf_counter: x86: fix 32-bit irq_period assumption (diff) | |
| download | linux-dev-755642322aa66fbc5421a35fd3e1733f73e20083.tar.xz linux-dev-755642322aa66fbc5421a35fd3e1733f73e20083.zip | |
perf_counter: use list_move_tail()
Instead of del/add use a move list-op.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/perf_counter.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/kernel/perf_counter.c b/kernel/perf_counter.c index b2e838959f3e..0fe22c916e29 100644 --- a/kernel/perf_counter.c +++ b/kernel/perf_counter.c @@ -89,8 +89,7 @@ list_del_counter(struct perf_counter *counter, struct perf_counter_context *ctx) list_for_each_entry_safe(sibling, tmp, &counter->sibling_list, list_entry) { - list_del_init(&sibling->list_entry); - list_add_tail(&sibling->list_entry, &ctx->counter_list); + list_move_tail(&sibling->list_entry, &ctx->counter_list); sibling->group_leader = sibling; } } @@ -959,8 +958,7 @@ static void rotate_ctx(struct perf_counter_context *ctx) */ perf_flags = hw_perf_save_disable(); list_for_each_entry(counter, &ctx->counter_list, list_entry) { - list_del(&counter->list_entry); - list_add_tail(&counter->list_entry, &ctx->counter_list); + list_move_tail(&counter->list_entry, &ctx->counter_list); break; } hw_perf_restore(perf_flags); |