Re: [Intel-gfx] [PATCH 11/18] drm/i915: Record space required for request emission

2016-09-19 Thread Joonas Lahtinen
On ma, 2016-09-19 at 12:32 +0100, Chris Wilson wrote: > On Mon, Sep 19, 2016 at 01:47:30PM +0300, Joonas Lahtinen wrote: > > On ke, 2016-09-14 at 07:52 +0100, Chris Wilson wrote: > > > @@ -2677,8 +2681,21 @@ static void intel_ring_default_vfuncs(struct > > > drm_i915_private *dev_priv, > > > e

Re: [Intel-gfx] [PATCH 11/18] drm/i915: Record space required for request emission

2016-09-19 Thread Chris Wilson
On Mon, Sep 19, 2016 at 01:47:30PM +0300, Joonas Lahtinen wrote: > On ke, 2016-09-14 at 07:52 +0100, Chris Wilson wrote: > > @@ -2677,8 +2681,21 @@ static void intel_ring_default_vfuncs(struct > > drm_i915_private *dev_priv, > >   engine->reset_hw = reset_ring_common; > >   > >   engine->emit_

Re: [Intel-gfx] [PATCH 11/18] drm/i915: Record space required for request emission

2016-09-19 Thread Joonas Lahtinen
On ke, 2016-09-14 at 07:52 +0100, Chris Wilson wrote: > @@ -1572,6 +1572,8 @@ static int gen8_emit_request(struct > drm_i915_gem_request *request) >   return intel_logical_ring_advance(request); >  } >   > +static const int gen8_emit_request_sz = 6 + WA_TAIL_DWORDS; Could argue these to be #d

Re: [Intel-gfx] [PATCH 11/18] drm/i915: Record space required for request emission

2016-09-15 Thread Tvrtko Ursulin
On 14/09/2016 18:33, Chris Wilson wrote: On Wed, Sep 14, 2016 at 02:30:20PM +0100, Tvrtko Ursulin wrote: On 14/09/2016 07:52, Chris Wilson wrote: In the next patch, we will use deferred request emission. That requires reserving sufficient space in the ringbuffer to emit the request, which firs

Re: [Intel-gfx] [PATCH 11/18] drm/i915: Record space required for request emission

2016-09-14 Thread Chris Wilson
On Wed, Sep 14, 2016 at 02:30:20PM +0100, Tvrtko Ursulin wrote: > > On 14/09/2016 07:52, Chris Wilson wrote: > >In the next patch, we will use deferred request emission. That requires > >reserving sufficient space in the ringbuffer to emit the request, which > >first requires us to know how large

Re: [Intel-gfx] [PATCH 11/18] drm/i915: Record space required for request emission

2016-09-14 Thread Tvrtko Ursulin
On 14/09/2016 07:52, Chris Wilson wrote: In the next patch, we will use deferred request emission. That requires reserving sufficient space in the ringbuffer to emit the request, which first requires us to know how large the request is. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i9

[Intel-gfx] [PATCH 11/18] drm/i915: Record space required for request emission

2016-09-13 Thread Chris Wilson
In the next patch, we will use deferred request emission. That requires reserving sufficient space in the ringbuffer to emit the request, which first requires us to know how large the request is. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_gem_request.c | 1 + drivers/gpu/drm/i915