Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20200415210512.805-1-valentin.schneider@arm.com>
 <20200416105828.GN20730@hirez.programming.kicks-ass.net> <20200416110056.GT20760@hirez.programming.kicks-ass.net>
In-Reply-To: <20200416110056.GT20760@hirez.programming.kicks-ass.net>
From:   Vincent Guittot <vincent.guittot@linaro.org>
Date:   Thu, 16 Apr 2020 15:00:12 +0200
Message-ID: <CAKfTPtB=Jr2XXjdUokRneNWNCNt6k6yYvhTuWxgAF4qs_wS2sg@mail.gmail.com>
Subject: Re: [PATCH v3 0/9] sched: Streamline select_task_rq() & select_task_rq_fair()
To:     Peter Zijlstra <peterz@infradead.org>
Cc:     Valentin Schneider <valentin.schneider@arm.com>,
        linux-kernel <linux-kernel@vger.kernel.org>,
        Ingo Molnar <mingo@kernel.org>,
        Dietmar Eggemann <dietmar.eggemann@arm.com>,
        Juri Lelli <juri.lelli@redhat.com>,
        Steven Rostedt <rostedt@goodmis.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu, 16 Apr 2020 at 13:01, Peter Zijlstra <peterz@infradead.org> wrote:
>
> On Thu, Apr 16, 2020 at 12:58:28PM +0200, Peter Zijlstra wrote:
> > On Wed, Apr 15, 2020 at 10:05:03PM +0100, Valentin Schneider wrote:
> > > Valentin Schneider (9):
> > >   sched/fair: find_idlest_group(): Remove unused sd_flag parameter
> > >   sched/debug: Make sd->flags sysctl read-only
> > >   sched: Remove checks against SD_LOAD_BALANCE
> > >   sched/topology: Kill SD_LOAD_BALANCE
> > >   sched: Add WF_TTWU, WF_EXEC wakeup flags
> >
> > How about I queue two first 5, and you rework these last few?
>
> Argh, 4 ofcourse, that 5th patch doesn't make much sense if we have to
> rework those flags like I proposed.

Looks good to me too

>
> > >   sched: Kill select_task_rq()'s sd_flag parameter
> > >   sched/fair: Dissociate wakeup decisions from SD flag value
> > >   sched/fair: Split select_task_rq_fair want_affine logic
> > >   sched/topology: Define and use shortcut pointers for wakeup sd_flag scan
> >
> >