Received-SPF: pass (google.com: domain of linux-kernel+bounces-177388-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Sender: Ingo Molnar <mingo.kernel.org@gmail.com>
Date: Mon, 13 May 2024 11:09:55 +0200
From: Ingo Molnar <mingo@kernel.org>
To: Vincent Guittot <vincent.guittot@linaro.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	linux-kernel@vger.kernel.org, Peter Zijlstra <peterz@infradead.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Juri Lelli <juri.lelli@redhat.com>,
	Dietmar Eggemann <dietmar.eggemann@arm.com>,
	Steven Rostedt <rostedt@goodmis.org>,
	Ben Segall <bsegall@google.com>, Mel Gorman <mgorman@suse.de>,
	Daniel Bristot de Oliveira <bristot@redhat.com>,
	Valentin Schneider <vschneid@redhat.com>,
	Shrikanth Hegde <sshegde@linux.ibm.com>
Subject: Re: [GIT PULL] Scheduler changes for v6.10
Message-ID: <ZkHY42pybndB1K+J@gmail.com>
References: <ZkG0nxxBPB/03Q/l@gmail.com>
 <CAKfTPtB01UKskH_a1h1LtjoRq6O3NnqQuuh1L50O-yHRLQm-iw@mail.gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAKfTPtB01UKskH_a1h1LtjoRq6O3NnqQuuh1L50O-yHRLQm-iw@mail.gmail.com>


* Vincent Guittot <vincent.guittot@linaro.org> wrote:

> Hi Ingo,
> 
> On Mon, 13 May 2024 at 08:35, Ingo Molnar <mingo@kernel.org> wrote:
> >
> > Linus,
> >
> > Please pull the latest sched/core Git tree from:
> >
> >    git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git sched-core-2024-05-13
> >
> >    # HEAD: 97450eb909658573dcacc1063b06d3d08642c0c1 sched/pelt: Remove shift of thermal clock
> >
> > Scheduler changes for v6.10:
> >
> >  - Add cpufreq pressure feedback for the scheduler
> >
> >  - Rework misfit load-balancing wrt. affinity restrictions
> >
> >  - Clean up and simplify the code around ::overutilized and
> >    ::overload access.
> >
> >  - Simplify sched_balance_newidle()
> >
> >  - Bump SCHEDSTAT_VERSION to 16 due to a cleanup of CPU_MAX_IDLE_TYPES
> >    handling that changed the output.
> >
> >  - Rework & clean up <asm/vtime.h> interactions wrt. arch_vtime_task_switch()
> >
> >  - Reorganize, clean up and unify most of the higher level
> >    scheduler balancing function names around the sched_balance_*()
> >    prefix.
> >
> >  - Simplify the balancing flag code (sched_balance_running)
> >
> >  - Miscellaneous cleanups & fixes
> >
> >  Thanks,
> >
> >         Ingo
> >
> > ------------------>
> > Alexander Gordeev (5):
> >       sched/vtime: Remove confusing arch_vtime_task_switch() declaration
> >       sched/vtime: Get rid of generic vtime_task_switch() implementation
> >       s390/vtime: Remove unused __ARCH_HAS_VTIME_TASK_SWITCH leftover
> >       s390/irq,nmi: Include <asm/vtime.h> header directly
> >       sched/vtime: Do not include <asm/vtime.h> header
> >
> > Ingo Molnar (28):
> >       sched/balancing: Switch the 'DEFINE_SPINLOCK(balancing)' spinlock into an 'atomic_t sched_balance_running' flag
> >       sched/balancing: Change 'enum cpu_idle_type' to have more natural definitions
> >       sched/debug: Increase SCHEDSTAT_VERSION to 16
> >       sched/debug: Allow CONFIG_SCHEDSTATS even on !KERNEL_DEBUG kernels
> >       sched/balancing: Change comment formatting to not overlap Git conflict marker lines
> >       sched/balancing: Fix comments (trying to) refer to NOHZ_BALANCE_KICK
> >       sched/balancing: Update run_rebalance_domains() comments
> >       sched/balancing: Vertically align the comments of 'struct sg_lb_stats' and 'struct sd_lb_stats'
> >       sched/balancing: Update comments in 'struct sg_lb_stats' and 'struct sd_lb_stats'
> >       sched/balancing: Rename run_rebalance_domains() => sched_balance_softirq()
> >       sched/balancing: Rename scheduler_tick() => sched_tick()
> >       sched/balancing: Rename trigger_load_balance() => sched_balance_trigger()
> >       sched/balancing: Rename rebalance_domains() => sched_balance_domains()
> >       sched/balancing: Rename load_balance() => sched_balance_rq()
> >       sched/balancing: Rename find_busiest_queue() => sched_balance_find_src_rq()
> >       sched/balancing: Rename find_busiest_group() => sched_balance_find_src_group()
> >       sched/balancing: Rename update_blocked_averages() => sched_balance_update_blocked_averages()
> >       sched/balancing: Rename newidle_balance() => sched_balance_newidle()
> >       sched/balancing: Rename find_idlest_group_cpu() => sched_balance_find_dst_group_cpu()
> >       sched/balancing: Rename find_idlest_group() => sched_balance_find_dst_group()
> >       sched/balancing: Rename find_idlest_cpu() => sched_balance_find_dst_cpu()
> >       sched/balancing: Fix a couple of outdated function names in comments
> >       sched/fair: Fix typos in comments
> >       sched/fair: Rename root_domain::overload to ::overloaded
> >       sched/fair: Rename {set|get}_rd_overload() to {set|get}_rd_overloaded()
> >       sched/fair: Rename SG_OVERLOAD to SG_OVERLOADED
> >       sched/fair: Rename set_rd_overutilized_status() to set_rd_overutilized()
> >       sched/balancing: Simplify the sg_status bitmask and use separate ->overloaded and ->overutilized flags
> >
> > Qais Yousef (4):
> >       sched/topology: Export asym_cap_list
> >       sched/fair: Check if a task has a fitting CPU when updating misfit
> >       sched/topology: Remove root_domain::max_cpu_capacity
> >       sched/fair: Don't double balance_interval for migrate_misfit
> >
> > Shrikanth Hegde (7):
> >       sched/balancing: Remove reliance on 'enum cpu_idle_type' ordering when iterating [CPU_MAX_IDLE_TYPES] arrays in show_schedstat()
> >       sched/fair: Add EAS checks before updating root_domain::overutilized
> >       sched/fair: Introduce is_rd_overutilized() helper function to access root_domain::overutilized
> >       sched/fair: Simplify the continue_balancing logic in sched_balance_newidle()
> >       sched/fair: Combine EAS check with root_domain::overutilized access
> >       sched/fair: Check root_domain::overload value before update
> >       sched/fair: Use helper functions to access root_domain::overload
> >
> > Vincent Guittot (6):
> >       sched/fair: Fix update of rd->sg_overutilized
> >       cpufreq: Add a cpufreq pressure feedback for the scheduler
> >       sched/cpufreq: Take cpufreq feedback into account
> >       thermal/cpufreq: Remove arch_update_thermal_pressure()
> >       sched/cpufreq: Rename arch_update_thermal_pressure() => arch_update_hw_pressure()
> 
> There is a fix for the one above:
>  https://lore.kernel.org/lkml/20240425073709.379016-1-vincent.guittot@linaro.org/

So this is a cleanup in essence, so I delayed it for the next sched/urgent 
pile - will apply it once Linus pulls sched/core.

Thanks,

	Ingo