Re: [Intel-gfx] [PATCH v5] drm/i915/execlists: Move WA_TAIL_DWORDS to callee

2016-02-25 Thread Dave Gordon
On 25/02/16 10:05, Chris Wilson wrote: On Wed, Feb 24, 2016 at 10:02:58AM +, Dave Gordon wrote: @@ -907,7 +942,8 @@ int intel_logical_ring_reserve_space(struct drm_i915_gem_request *request) * adding any commands to it then there might not actually be * sufficient room for

Re: [Intel-gfx] [PATCH v5] drm/i915/execlists: Move WA_TAIL_DWORDS to callee

2016-02-25 Thread Chris Wilson
On Wed, Feb 24, 2016 at 10:02:58AM +, Dave Gordon wrote: > @@ -907,7 +942,8 @@ int intel_logical_ring_reserve_space(struct > drm_i915_gem_request *request) >* adding any commands to it then there might not actually be >* sufficient room for the submission commands. >*/

[Intel-gfx] [PATCH v5] drm/i915/execlists: Move WA_TAIL_DWORDS to callee

2016-02-24 Thread Dave Gordon
From: Chris Wilson Currently emit-request starts writing to the ring and reserves space for a workaround to be emitted later whilst submitting the request. It is easier to read if the caller only allocates sufficient space for its access (then the reader can quickly verify that the ring begin all