Re: [PATCH] psi: Fix avg trigger being fired unexpectedly.

From: Yun Levi
Date: Mon Feb 26 2024 - 16:08:11 EST


Hi Suren.

> s/accumlated/accumulated
>
> > But, for PSI_AVGS, win->value should be initialized with group->total[PSI_AVGS].
> > Moreover, to get exact initial value for win->value, it should be set
> > under each trigger lock to synchronize each rtpoll/avgs_works.
>
> That last "to synchronize each rtpoll/avgs_works." part I would
> replace with "to avoid concurrent changes to group->total[]."
>
> Other than these nits LGTM. After fixing them feel free to add:
>
> Acked-by: Suren Baghdasaryan <surenb@xxxxxxxxxx>

Thanks. I'll fix my commit message ;)