Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one

2016-11-16 Thread Srivatsa, Anusha
sktop.org; Vivi, Rodrigo ><rodrigo.v...@intel.com> >Subject: Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel >parameter into one > >On Tue, Nov 15, 2016 at 10:06:47AM -0800, Srivatsa, Anusha wrote: >> >> >> >-Original Message- >

Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one

2016-11-16 Thread Tvrtko Ursulin
Mcgee, Jeff <jeff.mc...@intel.com> Cc: Ursulin, Tvrtko <tvrtko.ursu...@intel.com>; intel-gfx@lists.freedesktop.org; Vivi, Rodrigo <rodrigo.v...@intel.com> Subject: Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one On 14/11/2016 17:34, Srivatsa, Anusha w

Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one

2016-11-15 Thread Srivatsa, Anusha
sktop.org; Vivi, Rodrigo ><rodrigo.v...@intel.com> >Subject: Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel >parameter into one > >On Tue, Nov 15, 2016 at 10:06:47AM -0800, Srivatsa, Anusha wrote: >> >> >> >-Original Message- >

Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one

2016-11-15 Thread Jeff McGee
tel.com>; Mcgee, Jeff > ><jeff.mc...@intel.com> > >Cc: Ursulin, Tvrtko <tvrtko.ursu...@intel.com>; > >intel-gfx@lists.freedesktop.org; > >Vivi, Rodrigo <rodrigo.v...@intel.com> > >Subject: Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel > &

Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one

2016-11-15 Thread Srivatsa, Anusha
o.ursu...@intel.com>; >intel-gfx@lists.freedesktop.org; >Vivi, Rodrigo <rodrigo.v...@intel.com> >Subject: Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel >parameter into one > > >On 14/11/2016 17:34, Srivatsa, Anusha wrote: > >[snip] > >>> One ide

Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one

2016-11-15 Thread Tvrtko Ursulin
On 14/11/2016 17:34, Srivatsa, Anusha wrote: [snip] One idea could be to hide the guc loading form the user altogether and hence improve usability (decrease exposed complexity) by having only two parameters; i915.enable_guc_scheduling and i915.enable_huc. That's a good point. But with this

Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one

2016-11-14 Thread Srivatsa, Anusha
o.ursu...@intel.com>; >intel-gfx@lists.freedesktop.org; >Vivi, Rodrigo <rodrigo.v...@intel.com> >Subject: Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel >parameter into one > > >[corrected my email in cc] Thankyou! >On 12/11/2016 02:21, Jeff McGee wrote: >> On W

Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one

2016-11-14 Thread Tvrtko Ursulin
[corrected my email in cc] On 12/11/2016 02:21, Jeff McGee wrote: On Wed, Nov 09, 2016 at 11:11:07AM -0800, Anusha Srivatsa wrote: Replace i915.enable_guc_loading and i915.enable_guc_submission with a single parameter - i915.enable_guc. Where: -1 : Platform default (Only load GuC) 0 : Do not

Re: [Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one

2016-11-11 Thread Jeff McGee
On Wed, Nov 09, 2016 at 11:11:07AM -0800, Anusha Srivatsa wrote: > Replace i915.enable_guc_loading and i915.enable_guc_submission > with a single parameter - i915.enable_guc. Where: > > -1 : Platform default (Only load GuC) > 0 : Do not use GuC > 1 : Load GuC, do not use Command Submission

[Intel-gfx] [PATCH] drm/i915/GuC: Combine the two kernel parameter into one

2016-11-09 Thread Anusha Srivatsa
Replace i915.enable_guc_loading and i915.enable_guc_submission with a single parameter - i915.enable_guc. Where: -1 : Platform default (Only load GuC) 0 : Do not use GuC 1 : Load GuC, do not use Command Submission through GuC 2 : Load and use GuC for Command Submission Cc: Tvrtko Ursulin