[Intel-gfx] [PATCH 08/17] drm/i915: Don't look at pg_dirty_rings for aliasing ppgtt
Chris Wilson
chris at chris-wilson.co.uk
Tue Apr 14 09:06:36 PDT 2015
On Tue, Apr 14, 2015 at 05:35:18PM +0200, Daniel Vetter wrote:
> We load the ppgtt ptes once per gpu reset/driver load/resume and
> that's all that's needed. Note that this only blows up when we're
> using the allocate_va_range funcs and not the special-purpose ones
> used. With this change we can get rid of that duplication.
Honestly, I would prefer the test to be rewritten so that the
information on which vm was being used was passed along and not that
magic sprinkled in the middle of nowhere. e.g. execbuffer knows exactly
what vm it bound the objects into, and yet towards the end we decide to
guess again. Also, I would rather the execbuffer test be moved to
i915_gem_context since it is scattering internal knowledge about.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list