Re: [PATCH v4 12/13] rv: Fix read_lock scope in per-task DA cleanup

From: Gabriele Monaco

Date: Thu Jul 02 2026 - 02:31:32 EST


On Wed, 2026-07-01 at 15:27 -0400, Steven Rostedt wrote:
> On Mon,  1 Jun 2026 17:38:39 +0200
> Gabriele Monaco <gmonaco@xxxxxxxxxx> wrote:
>
> > The da_monitor_reset_all() function for per-task monitors takes
> > tasklist_lock while iterating over tasks, then keeps it also while
> > iterating over idle tasks (one per CPU). The latter is not necessary
> > since the lock needs to guard only for_each_process_thread().
> >
> > Use a scoped_guard for more compact syntax and adjust the scope only
> > where the lock is necessary.
> >
> > Reviewed-by: Wen Yang <wen.yang@xxxxxxxxx>
> > Reviewed-by: Nam Cao <namcao@xxxxxxxxxxxxx>
> > Signed-off-by: Gabriele Monaco <gmonaco@xxxxxxxxxx>
>
> Hmm, this and patch 13 didn't get applied. Do you plan on sending them later?

Yes, I didn't add them to the pull request since they weren't quite fixes, then
ended up not sending a "normal" pull request for 7.2 just for these.
I'm going to put them together for the next round.

Thanks,
Gabriele