Re: [PATCH 2/2] drm/i915: Add a module parameter to enable execlists

[Damien Lespiau <damien.lespiau@intel.com>]

On Mon, Apr 07, 2014 at 03:05:39PM +0100, Damien Lespiau wrote:
> Execlist are relatively new, and so it'd be wise to be able to merge
> that support disabled by default while still allowing a module parameter
> to enable that feature.
> 
> Even if we end up enabling execlists by default, it'll be handy to be
> able to switch back to ring submission to debug subtle problems that
> will inevitably arise.

After a quick chat with Daniel on IRC, we actually want that new code to
be enabled by default to get as much testing as possible, while
retaining the module parameter to quickly identify if a bug we're seeing
can be reproduced with the legacy ring submission or not.

> @@ -37,6 +37,7 @@ struct i915_params i915 __read_mostly = {
>  	.enable_fbc = -1,
>  	.enable_hangcheck = true,
>  	.enable_ppgtt = -1,
> +	.enable_execlists = 0,

So, this should be -1.

-- 
Damien