Re: [PATCH 00/43] Execlists v5

Jesse Barnes
2014-08-01T16:29:41Z

On Fri, 1 Aug 2014 17:09:50 +0100
Damien Lespiau <damien.lespiau@intel.com> wrote:

> On Thu, Jul 24, 2014 at 05:04:08PM +0100, Thomas Daniel wrote:
>
> All patches can have my r-b tag but patches 12, 34, 37, 39 which have
> minor comments (in terms of code changes) to address. I did look more at
> the low-level stuff (Vs the higher level abstractions).
>
> At this point, I believe the way forward is to merge that series to
> allow more people to beat on it. Step 1 is to make sure we don't regress
> the legacy ring buffers and now is a good time to land it as we start a
> new kernel cycle.
>
> Whether to enable it by default for BDW is an interesting question that
> may depend on the first round of QA.

Yeah I think we want to enable it on BDW too after getting some testing
and sanity checking. The legacy ring buffers aren't getting much
testing elsewhere and I'm afraid we'll run into issues that don't exist
with the execlist path if we stick with the legacy submission path (we
may have already hit one in fact).

--
Jesse Barnes, Intel Open Source Technology Center