Re: [PATCH v1] sched/rt: Fix rt task's sched latency statistics error in sched_stat_wait trace_point

[Yafang Shao]

On Wed, Dec 20, 2023 at 10:33 PM Junwen Wu <wudaemon@xxxxxxx> wrote:
>
> >>I think we'd better use if (!on_rt_rq(rt_se))
> >>
> >> +               update_stats_wait_end_rt(rt_rq, rt_se);
> >>
> hi, Yafang.when execute update_stats_dequeue_rt, rt_se->on_rq is still 0, util dequeue_rt_stack,
> the method is not effect.

Ah, you are right.

> I this we can use if (p && p != rq->curr) /*Mark the end of the wait period if dequeueing task*/

Agreed.

> because schedstats is not supported for rt group,we only need to update rt_se that is realy task.
> --
> Best regards
>


-- 
Regards
Yafang