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@xxxxxxxxxxxxxxxxxx> > > Cc: Michel Thierry <michel.thierry@xxxxxxxxx> > > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxx> > > Signed-off-by: Michał Winiarski <michal.winiarski@xxxxxxxxx> > Reviewed-by Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
