Re: [Intel-gfx] [PATCH 1/2] drm/i915: Add GuC support for engine busy stats

2017-12-22 Thread Tvrtko Ursulin
On 22/12/2017 10:50, Sagar Arun Kamble wrote: On 11/29/2017 6:03 PM, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Wire up the engine busy stats accounting to the GuC submission backend. Since there is not no Yeah. :)   context out interrupt we need to place the accounting callbacks per-re

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Add GuC support for engine busy stats

2017-12-22 Thread Sagar Arun Kamble
On 11/29/2017 6:03 PM, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Wire up the engine busy stats accounting to the GuC submission backend. Since there is not no context out interrupt we need to place the accounting callbacks per-request in order to correctly pair with user interrupts. v2

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Add GuC support for engine busy stats

2017-12-22 Thread Tvrtko Ursulin
Hi guys, Anyone has the bandwidth to review the below tiny patch, or suggest some else as potential reviewer? Truth be told I only tested it lightly, but it passed the perf_pmu unit tests. Goal is to bring engine busyness PMU reporting accuracy to the same level as execlists. Regards, Tv

[Intel-gfx] [PATCH 1/2] drm/i915: Add GuC support for engine busy stats

2017-11-29 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Wire up the engine busy stats accounting to the GuC submission backend. Since there is not context out interrupt we need to place the accounting callbacks per-request in order to correctly pair with user interrupts. v2: Rebase. v3: Commit update. Signed-off-by: Tvrtko Ursu