drm/i915: Add request to execbuf params and add explicit cleanup
authorJohn Harrison <John.C.Harrison@Intel.com>
Fri, 29 May 2015 16:43:30 +0000 (17:43 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 23 Jun 2015 12:02:01 +0000 (14:02 +0200)
commit6a6ae79a761ddc95b67254e256f82f6d7c9c44d3
tree28aa6f2081767af8a74b812ecfbb019d22b14a2b
parent217e46b576ef0d5eed10ddfeb2b29bd3de289e95
drm/i915: Add request to execbuf params and add explicit cleanup

Rather than just having a local request variable in the execbuff code, the
request pointer is now stored in the execbuff params structure. Also added
explicit cleanup of the request (plus wiping the OLR to match) in the error
case. This means that the execbuff code is no longer dependent upon the OLR
keeping track of the request so as to not leak it when things do go wrong. Note
that in the success case, the i915_add_request() at the end of the submission
function will tidy up the request and clear the OLR.

For: VIZ-5115
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
Reviewed-by: Tomas Elf <tomas.elf@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_drv.h
drivers/gpu/drm/i915/i915_gem_execbuffer.c