Re: [Intel-gfx] [PATCH] drm/i915: Only allocate preempt context when required

2018-01-30 Thread Chris Wilson
Quoting Lis, Tomasz (2018-01-30 18:47:55) > > > On 2018-01-27 21:28, Chris Wilson wrote: > > If we remove some hardcoded assumptions about the preempt context having > > a fixed id, reserved from use by normal user contexts, we may only > > allocate the i915_gem_context when required. Then the su

Re: [Intel-gfx] [PATCH] drm/i915: Only allocate preempt context when required

2018-01-30 Thread Lis, Tomasz
On 2018-01-27 21:28, Chris Wilson wrote: If we remove some hardcoded assumptions about the preempt context having a fixed id, reserved from use by normal user contexts, we may only allocate the i915_gem_context when required. Then the subsequent decisions on using preemption reduce to having th

Re: [Intel-gfx] [PATCH] drm/i915: Only allocate preempt context when required

2018-01-30 Thread Chris Wilson
Quoting Michał Winiarski (2018-01-30 11:25:50) > On Tue, Jan 30, 2018 at 09:02:45AM +, Chris Wilson wrote: > > Quoting Daniele Ceraolo Spurio (2018-01-30 00:24:04) > > > > > > > > > > @@ -979,17 +979,19 @@ static int guc_clients_create(struct intel_guc > > > > *guc) > > > > } > > > >

Re: [Intel-gfx] [PATCH] drm/i915: Only allocate preempt context when required

2018-01-30 Thread Michał Winiarski
On Tue, Jan 30, 2018 at 09:02:45AM +, Chris Wilson wrote: > Quoting Daniele Ceraolo Spurio (2018-01-30 00:24:04) > > > > > > > @@ -979,17 +979,19 @@ static int guc_clients_create(struct intel_guc *guc) > > > } > > > guc->execbuf_client = client; > > > > > > - client = guc_c

Re: [Intel-gfx] [PATCH] drm/i915: Only allocate preempt context when required

2018-01-30 Thread Chris Wilson
Quoting Daniele Ceraolo Spurio (2018-01-30 00:24:04) > > > > @@ -979,17 +979,19 @@ static int guc_clients_create(struct intel_guc *guc) > > } > > guc->execbuf_client = client; > > > > - client = guc_client_alloc(dev_priv, > > - INTEL_INFO(dev_priv)

Re: [Intel-gfx] [PATCH] drm/i915: Only allocate preempt context when required

2018-01-29 Thread Daniele Ceraolo Spurio
@@ -979,17 +979,19 @@ static int guc_clients_create(struct intel_guc *guc) } guc->execbuf_client = client; - client = guc_client_alloc(dev_priv, - INTEL_INFO(dev_priv)->ring_mask, - GUC_CLIENT_PRIORITY_KMD_HIGH,

Re: [Intel-gfx] [PATCH] drm/i915: Only allocate preempt context when required

2018-01-29 Thread Daniele Ceraolo Spurio
On 27/01/18 12:28, Chris Wilson wrote: If we remove some hardcoded assumptions about the preempt context having a fixed id, reserved from use by normal user contexts, we may only allocate the i915_gem_context when required. Then the subsequent decisions on using preemption reduce to having the

[Intel-gfx] [PATCH] drm/i915: Only allocate preempt context when required

2018-01-27 Thread Chris Wilson
If we remove some hardcoded assumptions about the preempt context having a fixed id, reserved from use by normal user contexts, we may only allocate the i915_gem_context when required. Then the subsequent decisions on using preemption reduce to having the preempt context available. Signed-off-by: