Re: [PATCH v5 1/4] sched/deadline: Make find_later_rq() choose a closer cpu in topology

From: Byungchul Park
Date: Wed Jul 12 2017 - 21:39:05 EST


On Wed, Jul 12, 2017 at 02:13:36PM +0100, Juri Lelli wrote:
> > @@ -1367,6 +1364,7 @@ static int find_later_rq(struct task_struct *task)
> > rcu_read_lock();
> > for_each_domain(cpu, sd) {
> > if (sd->flags & SD_WAKE_AFFINE) {

Hi,

> This is orthogonal to the proposed change, but I'm wondering if it make
> sense to do the following only for SD_WAKE_AFFINE domains. The

Actually I also wonder it..

> consideration applies to RT as well, actually. Also, find_later_rq gets
> called when trying to push tasks away as well and in that case checking
> for this flag seems inappropriate? Peter, Steve?
>
> Thanks,
>
> - Juri