On 24 January 2017 at 01:25, Robert Bragg <[email protected]> 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 <[email protected]>
Reviewed-by: Matthew Auld <[email protected]>
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
