On 24 January 2017 at 01:25, Robert Bragg <robert@xxxxxxxxxxxxx> wrote: > This avoids redundantly passing an (inout) head and tail pointer to > gen7_append_oa_reports() from gen7_oa_read which doesn't need to > reference either itself. > > Moving the head/tail reads and writes into gen7_append_oa_reports should > have no functional effect except to avoid some redundant head pointer > writes in cases where nothing was copied to userspace. > > This is a stepping stone towards updating how the head and tail pointer > state is managed to improve the workaround for the OA unit's tail > pointer race. It reduces the number of places we need to read/write the > head and tail pointers. > > Signed-off-by: Robert Bragg <robert@xxxxxxxxxxxxx> Reviewed-by: Matthew Auld <matthew.auld@xxxxxxxxx> _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
