Re: [Intel-gfx] [PATCH 8/8] drm/i915: Gate engine stats collection with a static key

2017-09-26 Thread Tvrtko Ursulin
On 25/09/2017 18:56, Chris Wilson wrote: Quoting Tvrtko Ursulin (2017-09-25 16:15:43) From: Tvrtko Ursulin This reduces the cost of the software engine busyness tracking to a single no-op instruction when there are no listeners. v2: Rebase and some comments. v3: Rebase. v4: Checkpatch fixes.

Re: [Intel-gfx] [PATCH 8/8] drm/i915: Gate engine stats collection with a static key

2017-09-25 Thread Chris Wilson
Quoting Tvrtko Ursulin (2017-09-25 16:15:43) > From: Tvrtko Ursulin > > This reduces the cost of the software engine busyness tracking > to a single no-op instruction when there are no listeners. > > v2: Rebase and some comments. > v3: Rebase. > v4: Checkpatch fixes. > v5: Rebase. > v6: Use syst

[Intel-gfx] [PATCH 8/8] drm/i915: Gate engine stats collection with a static key

2017-09-25 Thread Tvrtko Ursulin
From: Tvrtko Ursulin This reduces the cost of the software engine busyness tracking to a single no-op instruction when there are no listeners. v2: Rebase and some comments. v3: Rebase. v4: Checkpatch fixes. v5: Rebase. v6: Use system_long_wq to avoid being blocked by struct_mutex users. v7:

Re: [Intel-gfx] [PATCH 8/8] drm/i915: Gate engine stats collection with a static key

2017-09-18 Thread Rogozhkin, Dmitry V
On Mon, 2017-09-18 at 12:38 +0100, Tvrtko Ursulin wrote: > static inline void intel_engine_context_out(struct intel_engine_cs > *engine) > { > unsigned long flags; > > - if (READ_ONCE(engine->stats.enabled) == 0) > - return; > - > - spin_lock_irqsave(&engine->s

[Intel-gfx] [PATCH 8/8] drm/i915: Gate engine stats collection with a static key

2017-09-18 Thread Tvrtko Ursulin
From: Tvrtko Ursulin This reduces the cost of the software engine busyness tracking to a single no-op instruction when there are no listeners. v2: Rebase and some comments. v3: Rebase. v4: Checkpatch fixes. v5: Rebase. v6: Use system_long_wq to avoid being blocked by struct_mutex users. Sig