On Tue, Aug 23, 2016 at 01:53:09PM +0300, Joonas Lahtinen wrote: > 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()? The entire function is devoid of local cleanup. Oh well. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx