Re: [Intel-gfx] [PATCH 4/5] drm/i915/perf: no head/tail ref in gen7_oa_read

2017-01-25 Thread Matthew Auld
On 24 January 2017 at 01:25, Robert Bragg 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

[Intel-gfx] [PATCH 4/5] drm/i915/perf: no head/tail ref in gen7_oa_read

2017-01-23 Thread Robert Bragg
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