Re: [PATCH v2 15/20] rv: Convert the opid monitor to a hybrid automaton

From: Gabriele Monaco

Date: Mon Oct 13 2025 - 10:14:19 EST


On Fri, 2025-10-10 at 16:29 +0200, Nam Cao wrote:
> > + * If CONFIG_PREEMPTION is enabled, then the tracepoint
> > itself disables
> > + * preemption (adding one to the preempt_count). Since we
> > are
> > + * interested in the preempt_count at the time the
> > tracepoint was
> > + * hit, we consider 1 as still enabled.
> > + */
> > + if (IS_ENABLED(CONFIG_PREEMPTION))
> > + return (preempt_count() & PREEMPT_MASK) > 1;
>
> FYI, there is plan to keep preemption enabled during tracepoint
> handling. So keep that in mind when this monitor breaks.

Right, I've heard of it, thanks for the heads up. I'll adapt the monitors once
that happens.

Thanks,
Gabriele