[PATCH] drm/vmwgfx: Nuke preclose hook

2016-01-11 Thread Daniel Vetter
Again since the drm core takes care of event unlinking/disarming this is now just needless code. v2: I've completely missed eaction->fpriv_head and all the related code. We need to nuke that too to avoid accidentally deferencing the freed-up vmwgfx-private fpriv. v3: Also remove vmw_fpriv->fence_

[PATCH] drm/vmwgfx: Nuke preclose hook

2016-01-10 Thread Daniel Vetter
On Sun, Jan 10, 2016 at 11:17 PM, Thomas Hellstrom wrote: > In general, looks good. Two things though. > > 1) vc4_drv.h looks like it ended up in the wrong patch. Oops, will fix. > 2) Should be able to nuke also struct vmw_fpriv::fence_events and struct > vmw_event_fence_action::fpriv_head? Yes.

[PATCH] drm/vmwgfx: Nuke preclose hook

2016-01-10 Thread Thomas Hellstrom
In general, looks good. Two things though. 1) vc4_drv.h looks like it ended up in the wrong patch. 2) Should be able to nuke also struct vmw_fpriv::fence_events and struct vmw_event_fence_action::fpriv_head? /Thomas On 01/10/2016 11:02 PM, Daniel Vetter wrote: > Again since the drm core takes ca

[PATCH] drm/vmwgfx: Nuke preclose hook

2016-01-10 Thread Daniel Vetter
Again since the drm core takes care of event unlinking/disarming this is now just needless code. v2: I've completely missed eaction->fpriv_head and all the related code. We need to nuke that too to avoid accidentally deferencing the freed-up vmwgfx-private fpriv. Cc: Thomas Hellström Signed-off