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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
