Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-27 Thread Dave Gordon
On 26/04/16 15:00, Daniel Vetter wrote: On Mon, Apr 25, 2016 at 09:29:42AM +0100, Chris Wilson wrote: On Mon, Apr 25, 2016 at 08:31:07AM +0100, Dave Gordon wrote: On 22/04/16 19:51, Chris Wilson wrote: On Fri, Apr 22, 2016 at 07:45:15PM +0100, Chris Wilson wrote: On Fri, Apr 22, 2016 at

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-26 Thread Daniel Vetter
On Mon, Apr 25, 2016 at 09:29:42AM +0100, Chris Wilson wrote: > On Mon, Apr 25, 2016 at 08:31:07AM +0100, Dave Gordon wrote: > > On 22/04/16 19:51, Chris Wilson wrote: > > >On Fri, Apr 22, 2016 at 07:45:15PM +0100, Chris Wilson wrote: > > >>On Fri, Apr 22, 2016 at 07:22:55PM +0100, Dave Gordon

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-26 Thread Dave Gordon
On 26/04/16 11:35, Chris Wilson wrote: On Tue, Apr 26, 2016 at 10:52:41AM +0100, Dave Gordon wrote: On 26/04/16 09:49, Dave Gordon wrote: On 25/04/16 11:39, Chris Wilson wrote: On Mon, Apr 25, 2016 at 11:07:13AM +0100, Dave Gordon wrote: On 22/04/16 19:45, Chris Wilson wrote: [snip] And

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-26 Thread Chris Wilson
On Tue, Apr 26, 2016 at 10:52:41AM +0100, Dave Gordon wrote: > On 26/04/16 09:49, Dave Gordon wrote: > >On 25/04/16 11:39, Chris Wilson wrote: > >>On Mon, Apr 25, 2016 at 11:07:13AM +0100, Dave Gordon wrote: > >>>On 22/04/16 19:45, Chris Wilson wrote: > > [snip] > > And what exactly is that

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-26 Thread Dave Gordon
On 26/04/16 09:49, Dave Gordon wrote: On 25/04/16 11:39, Chris Wilson wrote: On Mon, Apr 25, 2016 at 11:07:13AM +0100, Dave Gordon wrote: On 22/04/16 19:45, Chris Wilson wrote: [snip] And what exactly is that atomic64_cmpxchg() serialising with? There are no other CPUs contending with the

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-26 Thread Dave Gordon
On 25/04/16 11:39, Chris Wilson wrote: On Mon, Apr 25, 2016 at 11:07:13AM +0100, Dave Gordon wrote: On 22/04/16 19:45, Chris Wilson wrote: On Fri, Apr 22, 2016 at 07:22:55PM +0100, Dave Gordon wrote: This patch simply changes the default value of "enable_guc_submission" >from 0 (never) to -1

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-25 Thread Chris Wilson
On Mon, Apr 25, 2016 at 11:07:13AM +0100, Dave Gordon wrote: > On 22/04/16 19:45, Chris Wilson wrote: > >On Fri, Apr 22, 2016 at 07:22:55PM +0100, Dave Gordon wrote: > >>This patch simply changes the default value of "enable_guc_submission" > >>from 0 (never) to -1 (auto). This means that GuC

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-25 Thread Dave Gordon
On 22/04/16 19:45, Chris Wilson wrote: On Fri, Apr 22, 2016 at 07:22:55PM +0100, Dave Gordon wrote: This patch simply changes the default value of "enable_guc_submission" from 0 (never) to -1 (auto). This means that GuC submission will be used if the platform has a GuC, the GuC supports the

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-25 Thread Chris Wilson
On Mon, Apr 25, 2016 at 08:31:07AM +0100, Dave Gordon wrote: > On 22/04/16 19:51, Chris Wilson wrote: > >On Fri, Apr 22, 2016 at 07:45:15PM +0100, Chris Wilson wrote: > >>On Fri, Apr 22, 2016 at 07:22:55PM +0100, Dave Gordon wrote: > >>>This patch simply changes the default value of

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-25 Thread Dave Gordon
On 22/04/16 19:51, Chris Wilson wrote: On Fri, Apr 22, 2016 at 07:45:15PM +0100, Chris Wilson wrote: On Fri, Apr 22, 2016 at 07:22:55PM +0100, Dave Gordon wrote: This patch simply changes the default value of "enable_guc_submission" from 0 (never) to -1 (auto). This means that GuC submission

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-22 Thread Chris Wilson
On Fri, Apr 22, 2016 at 07:45:15PM +0100, Chris Wilson wrote: > On Fri, Apr 22, 2016 at 07:22:55PM +0100, Dave Gordon wrote: > > This patch simply changes the default value of "enable_guc_submission" > > from 0 (never) to -1 (auto). This means that GuC submission will be > > used if the platform

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-22 Thread Chris Wilson
On Fri, Apr 22, 2016 at 07:22:55PM +0100, Dave Gordon wrote: > This patch simply changes the default value of "enable_guc_submission" > from 0 (never) to -1 (auto). This means that GuC submission will be > used if the platform has a GuC, the GuC supports the request submission > protocol, and any

[Intel-gfx] [PATCH 2/2] drm/i915/guc: default to using GuC submission where possible

2016-04-22 Thread Dave Gordon
This patch simply changes the default value of "enable_guc_submission" from 0 (never) to -1 (auto). This means that GuC submission will be used if the platform has a GuC, the GuC supports the request submission protocol, and any required GuC firmwware was successfully loaded. If any of these