> 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.
I´ll add these two patches to the series and resubmit with default = -1
Thanks!
Oscar
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
