RE: [RFC PATCH v2 14/23] sched/cache: Drop prefer_sibling restriction for llc_balance

[Jianyong Wu <wujianyong@hygon.cn>]

Hi Peter,

> -----Original Message-----
> From: Peter Zijlstra <peterz@infradead.org>
> Sent: Tuesday, September 1, 2026 6:29 PM
> To: Jianyong Wu <wujianyong@hygon.cn>
> Cc: Ingo Molnar <mingo@redhat.com>; Juri Lelli <juri.lelli@redhat.com>;
> Vincent Guittot <vincent.guittot@linaro.org>; Chen Yu
> <yu.c.chen@intel.com>; Tim Chen <tim.c.chen@linux.intel.com>; Dietmar
> Eggemann <dietmar.eggemann@arm.com>; Steven Rostedt
> <rostedt@goodmis.org>; Ben Segall <bsegall@google.com>; Mel Gorman
> <mgorman@suse.de>; Valentin Schneider <vschneid@redhat.com>; K
> Prateek Nayak <kprateek.nayak@amd.com>; Shrikanth Hegde
> <sshegde@linux.ibm.com>; Phil Auld <pauld@redhat.com>; Andrew
> Morton <akpm@linux-foundation.org>; David Hildenbrand
> <david@kernel.org>; linux-kernel@vger.kernel.org; linux-mm@kvack.org;
> jianyong.wu@outlook.com; Yuan Zhong <zhongyuan@hygon.cn>; Huangsj
> <huangsj@hygon.cn>; Fengyu Wang <wangfengyu@hygon.cn>; Zhiwei Ying
> <yingzhiwei@hygon.cn>; justin.he@arm.com
> Subject: Re: [RFC PATCH v2 14/23] sched/cache: Drop prefer_sibling
> restriction for llc_balance
> 
> On Thu, Aug 27, 2026 at 08:28:07PM +0800, Jianyong Wu wrote:
> > group_llc_balance performs balancing across LLC and NUMA domains.
> > The prefer_sibling constraint unnecessarily limits its scope, so remove
> > this requirement entirely from the branch condition.
> >
> > Signed-off-by: Jianyong Wu <wujianyong@hygon.cn>
> > ---
> >  kernel/sched/fair.c | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
> > index 584297528a9e..5c35a43e5e38 100644
> > --- a/kernel/sched/fair.c
> > +++ b/kernel/sched/fair.c
> > @@ -13274,9 +13274,9 @@ static struct sched_group
> *sched_balance_find_src_group(struct lb_env *env)
> >  	 * Try to move all excess tasks to a sibling domain of the busiest
> >  	 * group's child domain.
> >  	 */
> > -	if (sds.prefer_sibling && local->group_type == group_has_spare &&
> > -	    (busiest->group_type == group_llc_balance ||
> > -	    sibling_imbalance(env, &sds, busiest, local) > 1))
> > +	if (local->group_type == group_has_spare &&
> > +	    ((busiest->group_type == group_llc_balance) ||
> (sds.prefer_sibling &&
> > +	    sibling_imbalance(env, &sds, busiest, local) > 1)))
> >  		goto force_balance;
> 
> Not sure about blanked disable. Maybe only disable when cache aware
> scheduling is present and enabled?

The prefer_sibling bypass is already limited to cache aware scheduling.

group_llc_balance can only be selected when llc_balance() returns true,
and llc_balance() immediately return false when sched aware scheduling
is not active. When CONFIG_SCHED_CACHE is disabled, the corresponding
stub also cannot produce group_llc_balance.

Therefore, the changed branch cannot bypass prefer_sibling when cache
aware scheduling is absent or disabled.

Thanks
Jianyong