drm: Hide hw.lock cleanup in filp->release better
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 14 Jun 2016 18:50:57 +0000 (20:50 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 16 Jun 2016 08:16:37 +0000 (10:16 +0200)
commit1a75a222f5ca1063249b5c92972d32dcc3c8966e
tree396a78ce075816b8cd4b6d0a86696016aed7eefb
parente1b1cf2f026b75d1b001816f708dc529b7384750
drm: Hide hw.lock cleanup in filp->release better

A few things:
- Rename the cleanup function from drm_master_release to
  drm_legacy_lock_release. It doesn't relase any master stuff, but
  just the legacy hw lock.
- Hide it in drm_lock.c, which allows us to make a few more functions
  static in there. To avoid forward decl we need to shuffle the code a
  bit though.
- Push the check for ->master into the function itself.
- Only call this for !DRIVER_MODESET.

End result: Another place that takes struct_mutex gone for good for
modern drivers.

v2: Remove leftover comment.

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1465930269-7883-3-git-send-email-daniel.vetter@ffwll.ch
drivers/gpu/drm/drm_fops.c
drivers/gpu/drm/drm_legacy.h
drivers/gpu/drm/drm_lock.c