[Intel-gfx] [PATCH] drm/i915: Separate pinning of pages from i915_vma_insert()

Kumar Valsan, Prathap prathap.kumar.valsan at intel.com
Mon Aug 5 20:44:36 UTC 2019


On Fri, Aug 02, 2019 at 09:30:43PM +0100, Chris Wilson wrote:
> Quoting Prathap Kumar Valsan (2019-08-02 21:41:11)
> > Currently i915_vma_insert() is responsible for allocating drm mm node
> > and also allocating or gathering physical pages. Move the latter to a
> > separate function for better readability.
> 
> Close but if you look at the mutex patches, you'll see why it has to be
> before.
> -Chris

Looked at the Mutex patches. With async get_pages and async bind, pinning
and set pages are already separated out from i915_vma_insert(). So this
patch may not be needed.

Thanks,
Prathap


More information about the Intel-gfx mailing list