Commit 13c2235b authored by Edmundo Carmona Antoranz's avatar Edmundo Carmona Antoranz Committed by Peter Zijlstra

sched: Remove unnecessary variable from schedule_tail()

Since 565790d2 (sched: Fix balance_callback(), 2020-05-11), there
is no longer a need to reuse the result value of the call to finish_task_switch()
inside schedule_tail(), therefore the variable used to hold that value
(rq) is no longer needed.
Signed-off-by: default avatarEdmundo Carmona Antoranz <eantoranz@gmail.com>
Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20210306210739.1370486-1-eantoranz@gmail.com
parent 1e17fb8e
...@@ -4253,8 +4253,6 @@ static struct rq *finish_task_switch(struct task_struct *prev) ...@@ -4253,8 +4253,6 @@ static struct rq *finish_task_switch(struct task_struct *prev)
asmlinkage __visible void schedule_tail(struct task_struct *prev) asmlinkage __visible void schedule_tail(struct task_struct *prev)
__releases(rq->lock) __releases(rq->lock)
{ {
struct rq *rq;
/* /*
* New tasks start with FORK_PREEMPT_COUNT, see there and * New tasks start with FORK_PREEMPT_COUNT, see there and
* finish_task_switch() for details. * finish_task_switch() for details.
...@@ -4264,7 +4262,7 @@ asmlinkage __visible void schedule_tail(struct task_struct *prev) ...@@ -4264,7 +4262,7 @@ asmlinkage __visible void schedule_tail(struct task_struct *prev)
* PREEMPT_COUNT kernels). * PREEMPT_COUNT kernels).
*/ */
rq = finish_task_switch(prev); finish_task_switch(prev);
preempt_enable(); preempt_enable();
if (current->set_child_tid) if (current->set_child_tid)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment