Re: [PATCH] drm/i915: Disable full ppgtt by default

[Jesse Barnes <jbarnes@virtuousgeek.org>]

On Thu,  6 Mar 2014 12:14:21 +0100
Daniel Vetter <daniel.vetter@ffwll.ch> wrote:

> There are too many oustanding issues:
> 
> - Fence handling in the current code is broken. There's a patch series
>   from me, but it's blocked on and extended review (which includes
>   writing the testcases).
> 
> - IOMMU mapping handling is broken, we need to properly refcount it -
>   currently it gets destroyed when the first vma is unbound, so way
>   too early.
> 
> - There's a pending reset issue on snb. Since Mika's reset work and
>   full ppgtt have been pulled in in separate branches and ended up
>   intermittingly breaking each another it's unclear who's the exact
>   culprit here.
> 
> - We still have persistent evidince of crazy recursion bugs through
>   vma_unbind and ppgtt_relase, e.g.
> 
>   https://bugs.freedesktop.org/show_bug.cgi?id=73383
> 
>   This issue (and a few others meanwhile resolved) have blocked our
>   performance measuring/tuning group since 3 months.
> 
> - Secure batch dispatching is broken. This is blocking Brad Volkin's
>   command checker work since 3 months.

Do we have bugs and/or tasks filed for all these, or just the one?  We
need to make sure people are signed up to fix/review them, or it'll end
up staying disabled forever, and then we'll be stuck without a command
checker and some advanced features coming up...

-- 
Jesse Barnes, Intel Open Source Technology Center