On Fri, May 27, 2016 at 04:54:46PM +0100, Emil Velikov wrote: > On 27 May 2016 at 16:36, Chris Wilson <ch...@chris-wilson.co.uk> wrote: > > On Fri, May 27, 2016 at 05:29:43PM +0200, Nils Wallménius wrote: > >> Hi, Chris > >> > >> See comment below > >> > >> > err_setunique: > >> > if (drm_core_check_feature(dev, DRIVER_GEM)) > >> > @@ -653,8 +650,46 @@ err_minors: > >> > drm_fs_inode_free(dev->anon_inode); > >> > err_free: > >> > mutex_destroy(&dev->master_mutex); > >> > - kfree(dev); > >> > - return NULL; > >> > + return 0; > >> > >> Shouldn't this return != 0? > > > > You saw nothing, it always said return ret! > Chris can you please elaborate ? I believe Nils has a point here.
I mean, "thanks for bringing that to my attention, oops it was meant to say return ret ofc, fixed". -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
