Re: [Intel-gfx] [PATCH 43/46] drm/i915: Allocate a status page for each timeline

2019-01-16 Thread Chris Wilson
Quoting John Harrison (2019-01-16 21:06:36) > On 1/15/2019 10:43, Chris Wilson wrote: > > Quoting John Harrison (2019-01-15 18:17:21) > >> On 1/15/2019 01:50, Chris Wilson wrote: > >>> Quoting John Harrison (2019-01-15 00:56:13) > On 1/7/2019 03:55, Chris Wilson wrote: > > +static int allo

Re: [Intel-gfx] [PATCH 43/46] drm/i915: Allocate a status page for each timeline

2019-01-16 Thread John Harrison
On 1/15/2019 10:43, Chris Wilson wrote: Quoting John Harrison (2019-01-15 18:17:21) On 1/15/2019 01:50, Chris Wilson wrote: Quoting John Harrison (2019-01-15 00:56:13) On 1/7/2019 03:55, Chris Wilson wrote: +static int alloc_hwsp(struct i915_timeline *timeline) +{ + struct drm_i915_privat

Re: [Intel-gfx] [PATCH 43/46] drm/i915: Allocate a status page for each timeline

2019-01-15 Thread Chris Wilson
Quoting John Harrison (2019-01-15 18:17:21) > On 1/15/2019 01:50, Chris Wilson wrote: > > Quoting John Harrison (2019-01-15 00:56:13) > >> On 1/7/2019 03:55, Chris Wilson wrote: > >>> +static int alloc_hwsp(struct i915_timeline *timeline) > >>> +{ > >>> + struct drm_i915_private *i915 = timelin

Re: [Intel-gfx] [PATCH 43/46] drm/i915: Allocate a status page for each timeline

2019-01-15 Thread John Harrison
On 1/15/2019 01:50, Chris Wilson wrote: Quoting John Harrison (2019-01-15 00:56:13) On 1/7/2019 03:55, Chris Wilson wrote: +static int alloc_hwsp(struct i915_timeline *timeline) +{ + struct drm_i915_private *i915 = timeline->i915; + struct i915_vma *vma; + int offset; + + mutex_

Re: [Intel-gfx] [PATCH 43/46] drm/i915: Allocate a status page for each timeline

2019-01-15 Thread Chris Wilson
Quoting John Harrison (2019-01-15 00:56:13) > On 1/7/2019 03:55, Chris Wilson wrote: > > +static int alloc_hwsp(struct i915_timeline *timeline) > > +{ > > + struct drm_i915_private *i915 = timeline->i915; > > + struct i915_vma *vma; > > + int offset; > > + > > + mutex_lock(&i915->gt

Re: [Intel-gfx] [PATCH 43/46] drm/i915: Allocate a status page for each timeline

2019-01-14 Thread John Harrison
On 1/7/2019 03:55, Chris Wilson wrote: Allocate a page for use as a status page by a group of timelines, as we only need a dword of storage for each (rounded up to the cacheline for safety) we can pack multiple timelines into the same page. Each timeline will then be able to track its own HW seqn

[Intel-gfx] [PATCH 43/46] drm/i915: Allocate a status page for each timeline

2019-01-07 Thread Chris Wilson
Allocate a page for use as a status page by a group of timelines, as we only need a dword of storage for each (rounded up to the cacheline for safety) we can pack multiple timelines into the same page. Each timeline will then be able to track its own HW seqno. v2: Reuse the common per-engine HWSP