Re: [PATCH v2 01/15] sched: Add task enqueue/dequeue trace points

[bot+bpf-ci]

> sched: Add task enqueue/dequeue trace points

TL;DR: Commit message only: the changelog just restates the diff and doesn't
explain why the new sched_enqueue/sched_dequeue tracepoints are needed, a
question Peter Zijlstra raised on v2 that went unanswered.

>     sched: Add task enqueue/dequeue trace points
>
>     Add trace points into enqueue_task() and dequeue_task().
>
>     Signed-off-by: Nam Cao <namcao@xxxxxxxxxxxxx>
>     Suggested-by: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
>     Reviewed-by: K Prateek Nayak <kprateek.nayak@xxxxxxx>
>     Co-developed-by: Gabriele Monaco <gmonaco@xxxxxxxxxx>
>     Signed-off-by: Gabriele Monaco <gmonaco@xxxxxxxxxx>

Peter Zijlstra asked about the rationale for these tracepoints in the v2
thread:

  "As is I'm clueless as to why we want this, since the Changelog also
  offers none."

  https://lore.kernel.org/all/20261001154934.GT88198@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/

No reply was found in the archive, and the changelog above still consists of
the single sentence restating the diff.

Could the changelog explain why the sched_enqueue and sched_dequeue
tracepoints are needed?


---
AI reviewed your patch. Please fix the bug or email reply why it's not a bug.
See: https://github.com/kernel-patches/vmtest/blob/master/ci/claude/README.md

CI run summary: https://github.com/kernel-patches/bpf/actions/runs/36944413541