Quoting Matthew Auld (2019-10-18 10:07:49) > From: Abdiel Janulgue <[email protected]> > > Nothing to enumerate yet... > > Signed-off-by: Abdiel Janulgue <[email protected]> > Signed-off-by: Matthew Auld <[email protected]> > Cc: Joonas Lahtinen <[email protected]> > --- > @@ -2781,6 +2827,8 @@ void i915_ggtt_driver_release(struct drm_i915_private > *i915) > { > struct pagevec *pvec; > > + i915_gem_cleanup_memory_regions(i915); > + > fini_aliasing_ppgtt(&i915->ggtt); > > ggtt_cleanup_hw(&i915->ggtt); > @@ -2790,8 +2838,6 @@ void i915_ggtt_driver_release(struct drm_i915_private > *i915) > set_pages_array_wb(pvec->pages, pvec->nr); > __pagevec_release(pvec); > } > - > - i915_gem_cleanup_stolen(i915);
I had to do a quick double take, but it is just code moment and not premature stolen-as-region. Reviewed-by: Chris Wilson <[email protected]> -Chris _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
