On Tue, Sep 15, 2015 at 08:01:14PM +0200, Michał Winiarski wrote: > +void i915_address_space_init(struct drm_device *dev, > + struct i915_address_space *vm) > +{ > + struct drm_i915_private *dev_priv = dev->dev_private; > + > + drm_mm_init(&vm->mm, vm->start, vm->total); > + vm->dev = dev; > + INIT_LIST_HEAD(&vm->active_list); > + INIT_LIST_HEAD(&vm->inactive_list); > + INIT_LIST_HEAD(&vm->global_link); > + list_add_tail(&vm->global_link, &dev_priv->vm_list); Oh, that INIT_LIST_HEAD(global_link) is redundant. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx