Re: [PATCH v2 2/2] sched/rt: Remove unnecessary condition in push_rt_task()

2017-02-15 Thread Steven Rostedt
On Thu, 16 Feb 2017 11:05:54 +0900 Byungchul Park wrote: > Once pick_next_pushable_task(rq) returns a task, it guarantees that > the task's cpu is rq->cpu, so task_cpu(next_task) is always rq->cpu if > task == next_task. Remove the redundant condition and make code

Re: [PATCH v2 2/2] sched/rt: Remove unnecessary condition in push_rt_task()

2017-02-15 Thread Steven Rostedt
On Thu, 16 Feb 2017 11:05:54 +0900 Byungchul Park wrote: > Once pick_next_pushable_task(rq) returns a task, it guarantees that > the task's cpu is rq->cpu, so task_cpu(next_task) is always rq->cpu if > task == next_task. Remove the redundant condition and make code simpler. > >

[PATCH v2 2/2] sched/rt: Remove unnecessary condition in push_rt_task()

2017-02-15 Thread Byungchul Park
Once pick_next_pushable_task(rq) returns a task, it guarantees that the task's cpu is rq->cpu, so task_cpu(next_task) is always rq->cpu if task == next_task. Remove the redundant condition and make code simpler. pick_next_pushable_task(rq) has BUG_ON(rq_cpu != task_cpu(task)) when it returns a

[PATCH v2 2/2] sched/rt: Remove unnecessary condition in push_rt_task()

2017-02-15 Thread Byungchul Park
Once pick_next_pushable_task(rq) returns a task, it guarantees that the task's cpu is rq->cpu, so task_cpu(next_task) is always rq->cpu if task == next_task. Remove the redundant condition and make code simpler. pick_next_pushable_task(rq) has BUG_ON(rq_cpu != task_cpu(task)) when it returns a