Re: [PATCH v2 06/22] sched/fair: Collect load-balancing stats for IPC classes

From: Ricardo Neri
Date: Tue Dec 13 2022 - 19:23:46 EST


On Thu, Dec 08, 2022 at 08:50:23AM +0000, Ionela Voinescu wrote:
> Hi,
>
> On Monday 28 Nov 2022 at 05:20:44 (-0800), Ricardo Neri wrote:
> [..]
> >
> > +struct sg_lb_ipcc_stats {
> > + int min_score; /* Min(score(rq->curr->ipcc)) */
> > + int min_ipcc; /* Min(rq->curr->ipcc) */
>
> Nit: this is not the minimum IPCC between the current tasks of all
> runqueues, but the IPCC specific to the task with the minimum score.

> Possibly there's not much to be done about the variable name, but the
> comment can be made more clear.

Very true. I will make the change.

Thanks and BR,
Ricardo