On ma, 2016-08-22 at 09:03 +0100, Chris Wilson wrote: > @@ -4471,7 +4471,9 @@ int i915_gem_init(struct drm_device *dev) > */ > intel_uncore_forcewake_get(dev_priv, FORCEWAKE_ALL); > > - i915_gem_init_userptr(dev_priv); > + ret = i915_gem_init_userptr(dev_priv); > + if (ret) > + goto out_unlock; Don't we need a new teardown label after successful i915_gem_init_userptr()? With that added, Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
