git.samba.org
/
sfrench
/
cifs-2.6.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
8c7855d
)
perf: Simplify group_sched_in()
author
Peter Zijlstra
<peterz@infradead.org>
Thu, 29 Oct 2020 15:29:15 +0000
(16:29 +0100)
committer
Peter Zijlstra
<peterz@infradead.org>
Mon, 9 Nov 2020 17:12:35 +0000
(18:12 +0100)
Collate the error paths. Code duplication only leads to divergence and
extra bugs.
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link:
https://lkml.kernel.org/r/20201029162901.972161394@infradead.org
kernel/events/core.c
patch
|
blob
|
history
diff --git
a/kernel/events/core.c
b/kernel/events/core.c
index 9a5736617a8210ece067e0f6bfdb827b161fcb6b..f0e526866a1c1704d355ac6b41d5e97962a863a8 100644
(file)
--- a/
kernel/events/core.c
+++ b/
kernel/events/core.c
@@
-2580,11
+2580,8
@@
group_sched_in(struct perf_event *group_event,
pmu->start_txn(pmu, PERF_PMU_TXN_ADD);
- if (event_sched_in(group_event, cpuctx, ctx)) {
- pmu->cancel_txn(pmu);
- perf_mux_hrtimer_restart(cpuctx);
- return -EAGAIN;
- }
+ if (event_sched_in(group_event, cpuctx, ctx))
+ goto error;
/*
* Schedule in siblings as one group (if any):
@@
-2613,10
+2610,9
@@
group_error:
}
event_sched_out(group_event, cpuctx, ctx);
+error:
pmu->cancel_txn(pmu);
-
perf_mux_hrtimer_restart(cpuctx);
-
return -EAGAIN;
}