[Intel-gfx] [PATCH v2 1/3] drm/i915: enumerate and init each supported region

Chris Wilson chris at chris-wilson.co.uk
Fri Oct 18 09:49:57 UTC 2019


Quoting Matthew Auld (2019-10-18 10:07:49)
> From: Abdiel Janulgue <abdiel.janulgue at linux.intel.com>
> 
> Nothing to enumerate yet...
> 
> Signed-off-by: Abdiel Janulgue <abdiel.janulgue at linux.intel.com>
> Signed-off-by: Matthew Auld <matthew.auld at intel.com>
> Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> ---
> @@ -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 <chris at chris-wilson.co.uk>
-Chris


More information about the Intel-gfx mailing list