Re: [PATCH v2 09/14] perf cs-etm: Queue partial AUX records
From: James Clark
Date: Fri Oct 09 2026 - 05:29:33 EST
On 27/08/2026 18:09, Leo Yan wrote:
On Fri, Aug 21, 2026 at 10:49:07AM +0100, James Clark wrote:
In snapshot mode AUX records may straddle the boundary of multiple trace
buffers. Currently we only queue AUX records whose range is contained
wholly within a single buffer.
Fix it by queuing any partial AUX range that overlaps any buffer.
This issue has already existed before this series, so need a fix tag?
I saw it as more of an improvement than a fix. I was hesitant to put fix tags on everything because there can be so many different interactions between the changes.
Maybe we can leave the fix tags for the actual blockers and asserts etc?
Signed-off-by: James Clark <james.clark@xxxxxxxxxx>
Otherwise:
Reviewed-by: Leo Yan <leo.yan@xxxxxxx>