[PATCH] drm/gem: Always initialize the gem object in object_init

2014-01-20 Thread David Herrmann
Hi On Mon, Jan 20, 2014 at 8:21 AM, Daniel Vetter wrote: > At least drm/i915 expects that the obj->dev pointer is set even in > failure paths. Specifically when the shmem initialization fails we > call i915_gem_object_free which needs to deref obj->base.dev to get at > the slab pointer in the

[PATCH] drm/gem: Always initialize the gem object in object_init

2014-01-20 Thread Chris Wilson
On Mon, Jan 20, 2014 at 08:21:54AM +0100, Daniel Vetter wrote: > At least drm/i915 expects that the obj->dev pointer is set even in > failure paths. Specifically when the shmem initialization fails we > call i915_gem_object_free which needs to deref obj->base.dev to get at > the slab pointer in

[PATCH] drm/gem: Always initialize the gem object in object_init

2014-01-20 Thread Daniel Vetter
At least drm/i915 expects that the obj->dev pointer is set even in failure paths. Specifically when the shmem initialization fails we call i915_gem_object_free which needs to deref obj->base.dev to get at the slab pointer in the device private structure. And the shmem allocation can easily fail