]> git.baikalelectronics.ru Git - kernel.git/commit
drm: trylock modest locking for fbdev panics
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Sun, 27 Jul 2014 17:09:33 +0000 (19:09 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 8 Aug 2014 15:47:01 +0000 (17:47 +0200)
commitccf6cd9dd8e591a146c68b159e3f8521cdb1ee7d
tree4bedf2841031c958813a71d8358ec1aa6178a76d
parentd54ede570c7fa90d2f60cba5f87381fa9e3b25c6
drm: trylock modest locking for fbdev panics

In the fbdev code we want to do trylocks only to avoid deadlocks and
other ugly issues. Thus far we've only grabbed the overall modeset
lock, but that already failed to exclude a pile of potential
concurrent operations. With proper atomic support this will be worse.

So add a trylock mode to the modeset locking code which attempts all
locks only with trylocks, if possible. We need to track this in the
locking functions themselves and can't restrict this to drivers since
driver-private w/w mutexes must be treated the same way.

There's still the issue that other driver private locks aren't handled
here at all, but well can't have everything. With this we will at
least not regress, even once atomic allows lots of concurrent kms
activity.

Aside: We should move the acquire context to stack-based allocation in
the callers to get rid of that awful WARN_ON(kmalloc_failed) control
flow which just blows up when memory is short. But that's material for
separate patches.

v2:
- Fix logic inversion fumble in the fb helper.
- Add proper kerneldoc.

Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/drm_fb_helper.c
drivers/gpu/drm/drm_modeset_lock.c
include/drm/drm_modeset_lock.h