Re: [PATCH v2 1/2] perf: arm_spe: Correct setting the PERF_HES_STOPPED flag

From: Leo Yan

Date: Tue Jan 20 2026 - 13:15:50 EST


On Tue, Jan 20, 2026 at 04:42:39PM +0000, Will Deacon wrote:

[...]

> Ok, so looking at this and the next patch I wonder if we could simplify
> things a little by having arm_spe_perf_aux_output_begin() return an 'int'
> to indicate success/failure instead of touching 'hwc->state'.
>
> Then arm_spe_pmu_start() and the interrupt handler could call into
> arm_spe_pmu_stop() if they get an error code back. Would that work?

Yes, your suggested approach is cleaner. I will respin patches.

Thanks for the suggestions!

Leo