Re: [PATCH] sched/pelt: Fix bracket typo

From: Randy Dunlap
Date: Wed Jun 08 2022 - 11:34:47 EST




On 6/8/22 08:00, Steven Lung wrote:
> The second bracket describing the range is inverted, this patch
> will fix it.
>
> Signed-off-by: Steven Lung <1030steven@xxxxxxxxx>
> ---
> kernel/sched/pelt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kernel/sched/pelt.c b/kernel/sched/pelt.c
> index 0f3107682..ed82cfba9 100644
> --- a/kernel/sched/pelt.c
> +++ b/kernel/sched/pelt.c
> @@ -233,7 +233,7 @@ ___update_load_sum(u64 now, struct sched_avg *sa,
> * When syncing *_avg with *_sum, we must take into account the current
> * position in the PELT segment otherwise the remaining part of the segment
> * will be considered as idle time whereas it's not yet elapsed and this will
> - * generate unwanted oscillation in the range [1002..1024[.

Is the above the same as range [1002..1024).
? I.e. 1002-1023 inclusive (or 1024 excluded)?

> + * generate unwanted oscillation in the range [1002..1024].
> *
> * The max value of *_sum varies with the position in the time segment and is
> * equals to :

--
~Randy