Re: [PATCH v6 4/5] sched: Rename arch_update_thermal_pressure into arch_update_hw_pressure

2024-04-30 Thread Vincent Guittot
H Konrad, On Tue, 30 Apr 2024 at 13:23, Konrad Dybcio wrote: > > On 26.03.2024 10:16 AM, Vincent Guittot wrote: > > Now that cpufreq provides a pressure value to the scheduler, rename > > arch_update_thermal_pressure into HW pressure to reflect that it returns > > a pre

[PATCH v6 5/5] sched/pelt: Remove shift of thermal clock

2024-03-26 Thread Vincent Guittot
other signals. Mark sysctl_sched_migration_cost boot parameter as deprecated Signed-off-by: Vincent Guittot Reviewed-by: Qais Yousef Reviewed-by: Lukasz Luba Tested-by: Lukasz Luba --- .../admin-guide/kernel-parameters.txt | 1 + kernel/sched/core.c| 2

[PATCH v6 4/5] sched: Rename arch_update_thermal_pressure into arch_update_hw_pressure

2024-03-26 Thread Vincent Guittot
example. Such high frequency signal needs filtering to be smoothed and provide an value that reflects the average available capacity into the scheduler time scale. Signed-off-by: Vincent Guittot Reviewed-by: Qais Yousef Reviewed-by: Lukasz Luba Tested-by: Lukasz Luba --- arch/arm/include/asm

[PATCH v6 3/5] thermal/cpufreq: Remove arch_update_thermal_pressure()

2024-03-26 Thread Vincent Guittot
which includes cpufreq cooling device. Remove the call to arch_update_thermal_pressure() in cpufreq cooling device as this is handled by cpufreq_get_pressure(). Signed-off-by: Vincent Guittot Reviewed-by: Lukasz Luba Acked-by: Viresh Kumar Tested-by: Lukasz Luba --- drivers/thermal/cpufreq_cool

[PATCH v6 2/5] sched: Take cpufreq feedback into account

2024-03-26 Thread Vincent Guittot
Aggregate the different pressures applied on the capacity of CPUs and create a new function that returns the actual capacity of the CPU: get_actual_cpu_capacity() Signed-off-by: Vincent Guittot Reviewed-by: Lukasz Luba Reviewed-by: Qais Yousef Tested-by: Lukasz Luba --- kernel/sched/fair.c

[PATCH v6 0/5] Rework system pressure interface to the scheduler

2024-03-26 Thread Vincent Guittot
ssure() Change since v1: - Use struct cpufreq_policy as parameter of cpufreq_update_pressure() - Fix typos and comments - Make sched_thermal_decay_shift boot param as deprecated Vincent Guittot (5): cpufreq: Add a cpufreq pressure feedback for the scheduler sched: Take cpufreq feedback into ac

[PATCH v6 1/5] cpufreq: Add a cpufreq pressure feedback for the scheduler

2024-03-26 Thread Vincent Guittot
Provide to the scheduler a feedback about the temporary max available capacity. Unlike arch_update_thermal_pressure, this doesn't need to be filtered as the pressure will happen for dozens ms or more. Signed-off-by: Vincent Guittot Acked-by: Rafael J. Wysocki Acked-by: Viresh Kumar Review

[PATCH v5 5/5] sched/pelt: Remove shift of thermal clock

2024-02-20 Thread Vincent Guittot
other signals. Mark sysctl_sched_migration_cost boot parameter as deprecated Signed-off-by: Vincent Guittot Reviewed-by: Qais Yousef --- .../admin-guide/kernel-parameters.txt | 1 + kernel/sched/core.c| 2 +- kernel/sched/fair.c| 10

[PATCH v5 4/5] sched: Rename arch_update_thermal_pressure into arch_update_hw_pressure

2024-02-20 Thread Vincent Guittot
example. Such high frequency signal needs filtering to be smoothed and provide an value that reflects the average available capacity into the scheduler time scale. Signed-off-by: Vincent Guittot Reviewed-by: Qais Yousef --- arch/arm/include/asm/topology.h | 6 ++--- arch/arm64

[PATCH v5 3/5] thermal/cpufreq: Remove arch_update_thermal_pressure()

2024-02-20 Thread Vincent Guittot
which includes cpufreq cooling device. Remove the call to arch_update_thermal_pressure() in cpufreq cooling device as this is handled by cpufreq_get_pressure(). Signed-off-by: Vincent Guittot Reviewed-by: Lukasz Luba Acked-by: Viresh Kumar --- drivers/thermal/cpufreq_cooling.c | 3 --- 1 file chang

[PATCH v5 2/5] sched: Take cpufreq feedback into account

2024-02-20 Thread Vincent Guittot
Aggregate the different pressures applied on the capacity of CPUs and create a new function that returns the actual capacity of the CPU: get_actual_cpu_capacity() Signed-off-by: Vincent Guittot Reviewed-by: Lukasz Luba Reviewed-by: Qais Yousef --- kernel/sched/fair.c | 45

[PATCH v5 1/5] cpufreq: Add a cpufreq pressure feedback for the scheduler

2024-02-20 Thread Vincent Guittot
Provide to the scheduler a feedback about the temporary max available capacity. Unlike arch_update_thermal_pressure, this doesn't need to be filtered as the pressure will happen for dozens ms or more. Signed-off-by: Vincent Guittot Acked-by: Rafael J. Wysocki Acked-by: Viresh Kumar Review

[PATCH v5 0/5] Rework system pressure interface to the scheduler

2024-02-20 Thread Vincent Guittot
ments - Make sched_thermal_decay_shift boot param as deprecated Vincent Guittot (5): cpufreq: Add a cpufreq pressure feedback for the scheduler sched: Take cpufreq feedback into account thermal/cpufreq: Remove arch_update_thermal_pressure() sched: Rename arch_update_thermal_pressure

Re: [PATCH v4 2/5] sched: Take cpufreq feedback into account

2024-01-30 Thread Vincent Guittot
On Tue, 30 Jan 2024 at 01:50, Qais Yousef wrote: > > On 01/30/24 00:26, Qais Yousef wrote: > > On 01/09/24 17:46, Vincent Guittot wrote: > > > Aggregate the different pressures applied on the capacity of CPUs and > > > create a new function that returns

Re: [PATCH v3 0/5] Rework system pressure interface to the scheduler

2024-01-19 Thread Vincent Guittot
On Wed, 10 Jan 2024 at 19:10, Dietmar Eggemann wrote: > > On 09/01/2024 14:29, Vincent Guittot wrote: > > On Tue, 9 Jan 2024 at 12:34, Dietmar Eggemann > > wrote: > >> > >> On 08/01/2024 14:48, Vincent Guittot wrote: > >>> Following the consol

Re: [PATCH v3 2/5] sched: Take cpufreq feedback into account

2024-01-10 Thread Vincent Guittot
On Wed, 10 Jan 2024 at 14:51, Dietmar Eggemann wrote: > > On 09/01/2024 15:30, Vincent Guittot wrote: > > On Tue, 9 Jan 2024 at 12:22, Dietmar Eggemann > > wrote: > >> > >> On 08/01/2024 14:48, Vincent Guittot wrote: > >>> Aggregate the differe

Re: [PATCH v4 1/5] cpufreq: Add a cpufreq pressure feedback for the scheduler

2024-01-09 Thread Vincent Guittot
On Tue, 9 Jan 2024 at 17:49, Rafael J. Wysocki wrote: > > On Tue, Jan 9, 2024 at 5:47 PM Vincent Guittot > wrote: > > > > Provide to the scheduler a feedback about the temporary max available > > capacity. Unlike arch_update_thermal_pressure, this doesn't need to

[PATCH v4 5/5] sched/pelt: Remove shift of thermal clock

2024-01-09 Thread Vincent Guittot
other signals. Mark sysctl_sched_migration_cost boot parameter as deprecated Signed-off-by: Vincent Guittot --- .../admin-guide/kernel-parameters.txt | 1 + kernel/sched/core.c| 2 +- kernel/sched/fair.c| 10 ++ kernel/sched

[PATCH v4 4/5] sched: Rename arch_update_thermal_pressure into arch_update_hw_pressure

2024-01-09 Thread Vincent Guittot
example. Such high frequency signal needs filtering to be smoothed and provide a value that reflects the average available capacity into the scheduler time scale. Signed-off-by: Vincent Guittot --- arch/arm/include/asm/topology.h | 6 ++--- arch/arm64/include/asm/topology.h

[PATCH v4 3/5] thermal/cpufreq: Remove arch_update_thermal_pressure()

2024-01-09 Thread Vincent Guittot
which includes cpufreq cooling device. Remove the call to arch_update_thermal_pressure() in cpufreq cooling device as this is handled by cpufreq_get_pressure(). Signed-off-by: Vincent Guittot Reviewed-by: Lukasz Luba Acked-by: Viresh Kumar --- drivers/thermal/cpufreq_cooling.c | 3 --- 1 file chang

[PATCH v4 2/5] sched: Take cpufreq feedback into account

2024-01-09 Thread Vincent Guittot
Aggregate the different pressures applied on the capacity of CPUs and create a new function that returns the actual capacity of the CPU: get_actual_cpu_capacity() Signed-off-by: Vincent Guittot Reviewed-by: Lukasz Luba --- kernel/sched/fair.c | 45

[PATCH v4 1/5] cpufreq: Add a cpufreq pressure feedback for the scheduler

2024-01-09 Thread Vincent Guittot
Provide to the scheduler a feedback about the temporary max available capacity. Unlike arch_update_thermal_pressure, this doesn't need to be filtered as the pressure will happen for dozens ms or more. Signed-off-by: Vincent Guittot --- drivers/cpufreq/cpufreq.c

[PATCH v4 0/5] Rework system pressure interface to the scheduler

2024-01-09 Thread Vincent Guittot
- Fix uninitialized variables in cpufreq_update_pressure() Change since v2: - Rework cpufreq_update_pressure() Change since v1: - Use struct cpufreq_policy as parameter of cpufreq_update_pressure() - Fix typos and comments - Make sched_thermal_decay_shift boot param as deprecated Vincent Guittot (5): cp

Re: [PATCH v3 2/5] sched: Take cpufreq feedback into account

2024-01-09 Thread Vincent Guittot
On Tue, 9 Jan 2024 at 12:22, Dietmar Eggemann wrote: > > On 08/01/2024 14:48, Vincent Guittot wrote: > > Aggregate the different pressures applied on the capacity of CPUs and > > create a new function that returns the actual capacity of the CPU: > > get_actual_cpu_ca

Re: [PATCH v3 4/5] sched: Rename arch_update_thermal_pressure into arch_update_hw_pressure

2024-01-09 Thread Vincent Guittot
On Tue, 9 Jan 2024 at 12:56, Dietmar Eggemann wrote: > > On 08/01/2024 14:48, Vincent Guittot wrote: > > Now that cpufreq provides a pressure value to the scheduler, rename > > I.e. that thermal (e.g. IPA governor) switches from average > (rq->avg_(thermal/hw).load_

Re: [PATCH v3 0/5] Rework system pressure interface to the scheduler

2024-01-09 Thread Vincent Guittot
On Tue, 9 Jan 2024 at 12:34, Dietmar Eggemann wrote: > > On 08/01/2024 14:48, Vincent Guittot wrote: > > Following the consolidation and cleanup of CPU capacity in [1], this serie > > reworks how the scheduler gets the pressures on CPUs. We need to take into > > account

Re: [PATCH v3 1/5] cpufreq: Add a cpufreq pressure feedback for the scheduler

2024-01-09 Thread Vincent Guittot
On Mon, 8 Jan 2024 at 17:35, Dietmar Eggemann wrote: > > On 08/01/2024 14:48, Vincent Guittot wrote: > > Provide to the scheduler a feedback about the temporary max available > > capacity. Unlike arch_update_thermal_pressure, this doesn't need to be > > filtered as

Re: [PATCH v3 1/5] cpufreq: Add a cpufreq pressure feedback for the scheduler

2024-01-08 Thread Vincent Guittot
On Mon, 8 Jan 2024 at 17:35, Dietmar Eggemann wrote: > > On 08/01/2024 14:48, Vincent Guittot wrote: > > Provide to the scheduler a feedback about the temporary max available > > capacity. Unlike arch_update_thermal_pressure, this doesn't need to be > > filtered as

[PATCH v3 5/5] sched/pelt: Remove shift of thermal clock

2024-01-08 Thread Vincent Guittot
other signals. Mark deprecated sched_thermal_decay_shift boot parameter. Signed-off-by: Vincent Guittot --- .../admin-guide/kernel-parameters.txt | 1 + kernel/sched/core.c| 2 +- kernel/sched/fair.c| 10 ++ kernel/sched

[PATCH v3 4/5] sched: Rename arch_update_thermal_pressure into arch_update_hw_pressure

2024-01-08 Thread Vincent Guittot
example. Such high frequency signal needs filtering to be smoothed and provide an value that reflects the average available capacity into the scheduler time scale. Signed-off-by: Vincent Guittot --- arch/arm/include/asm/topology.h | 6 ++--- arch/arm64/include/asm/topology.h

[PATCH v3 3/5] thermal/cpufreq: Remove arch_update_thermal_pressure()

2024-01-08 Thread Vincent Guittot
which includes cpufreq cooling device. Remove the call to arch_update_thermal_pressure() in cpufreq cooling device as this is handled by cpufreq_get_pressure(). Signed-off-by: Vincent Guittot Reviewed-by: Lukasz Luba Acked-by: Viresh Kumar --- drivers/thermal/cpufreq_cooling.c | 3 --- 1 file chang

[PATCH v3 2/5] sched: Take cpufreq feedback into account

2024-01-08 Thread Vincent Guittot
Aggregate the different pressures applied on the capacity of CPUs and create a new function that returns the actual capacity of the CPU: get_actual_cpu_capacity() Signed-off-by: Vincent Guittot Reviewed-by: Lukasz Luba --- kernel/sched/fair.c | 45

[PATCH v3 1/5] cpufreq: Add a cpufreq pressure feedback for the scheduler

2024-01-08 Thread Vincent Guittot
Provide to the scheduler a feedback about the temporary max available capacity. Unlike arch_update_thermal_pressure, this doesn't need to be filtered as the pressure will happen for dozens ms or more. Signed-off-by: Vincent Guittot --- drivers/cpufreq/cpufreq.c

[PATCH v3 0/5] Rework system pressure interface to the scheduler

2024-01-08 Thread Vincent Guittot
ework cpufreq_update_pressure() Change since v1: - Use struct cpufreq_policy as parameter of cpufreq_update_pressure() - Fix typos and comments - Make sched_thermal_decay_shift boot param as deprecated Vincent Guittot (5): cpufreq: Add a cpufreq pressure feedback for the scheduler sched: Take cp