Re: [Intel-gfx] [PATCH 03/12] drm/i915: Remove engine->execlist_lock

2016-11-03 Thread Tvrtko Ursulin
On 03/11/2016 12:28, Chris Wilson wrote: On Thu, Nov 03, 2016 at 10:47:54AM +, Tvrtko Ursulin wrote: On 02/11/2016 17:50, Chris Wilson wrote: @@ -600,9 +598,8 @@ static void intel_lrc_irq_handler(unsigned long data) static void execlists_submit_request(struct drm_i915_gem_request *request

Re: [Intel-gfx] [PATCH 03/12] drm/i915: Remove engine->execlist_lock

2016-11-03 Thread Chris Wilson
On Thu, Nov 03, 2016 at 10:47:54AM +, Tvrtko Ursulin wrote: > > On 02/11/2016 17:50, Chris Wilson wrote: > >@@ -600,9 +598,8 @@ static void intel_lrc_irq_handler(unsigned long data) > > static void execlists_submit_request(struct drm_i915_gem_request *request) > > { > > struct intel_engine

Re: [Intel-gfx] [PATCH 03/12] drm/i915: Remove engine->execlist_lock

2016-11-03 Thread Tvrtko Ursulin
On 02/11/2016 17:50, Chris Wilson wrote: The execlist_lock is now completely subsumed by the engine->timeline->lock, and so we can remove the redundant layer of locking. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_debugfs.c | 4 ++-- drivers/gpu/drm/i915/i915_gem.c |

[Intel-gfx] [PATCH 03/12] drm/i915: Remove engine->execlist_lock

2016-11-02 Thread Chris Wilson
The execlist_lock is now completely subsumed by the engine->timeline->lock, and so we can remove the redundant layer of locking. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_debugfs.c | 4 ++-- drivers/gpu/drm/i915/i915_gem.c | 4 ++-- drivers/gpu/drm/i915/intel_engine_c