Re: [PATCH v3 1/2] drm/i915/gtt: Do not initialize drm_mm twice.

[Daniel Vetter <daniel@ffwll.ch>]

On Thu, Sep 17, 2015 at 11:59:19AM +0100, Chris Wilson wrote:
> On Wed, Sep 16, 2015 at 11:49:00AM +0200, Michał Winiarski wrote:
> 
> Rearrange the code such that i915_init_vm() is next to its callers
> inside i915_gem_gtt (and so we can make it static). After removing the
> dance around the files, it is clear that we are repeating some work
> inside the initializers (such as calling drm_mm_init() multiple times),
> so take advantage of the refactor to also remove some redundant code and
> clean up the interface.

Added to the commit message.

> > v2: Commit msg update,
> >     s/i915_init_vm/i915_address_space_init, move to i915_gem_gtt.c,
> >     init address_space during i915_gem_setup_global_gtt for ggtt.
> > v3: Do not init global_link - we are adding it to vm_list moments later,
> >     make i915_address_space_init static, use OOP style parameter order.
> > 
> > Cc: Chris Wilson <chris@chris-wilson.co.uk>
> > Cc: Michel Thierry <michel.thierry@intel.com>
> > Cc: Mika Kuoppala <mika.kuoppala@intel.com>
> > Signed-off-by: Michał Winiarski <michal.winiarski@intel.com>
> Reviewed-by Chris Wilson <chris@chris-wilson.co.uk>

Queued for -next, thanks for the patch.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx