From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
While staring at some DRM_MODESET_LOCK_ALL_{BEGIN,END}() conversions
I decided I don't really like what those macros do.
The main problems that I see:
- the magic goto inside limits their usefulness to baically
a single statement, unless you're willing to look inside and
find out the name of the magic label
- can't help at all in the "we don't want to lock everything"
cases, which are quite numerous
- not at all obvious that there's a loop in there
So I figured I'd try to come up with something more universally useful.
Cc: Sean Paul <seanpaul@xxxxxxxxxxxx>
Cc: Daniel Vetter <daniel@xxxxxxxx>
Ville Syrjälä (4):
drm: Introduce drm_modeset_lock_ctx_retry()
drm: Introduce drm_modeset_lock_all_ctx_retry()
drm/i915: Extract intel_crtc_initial_commit()
drm/i915: Use drm_modeset_lock_ctx_retry() & co.
drivers/gpu/drm/drm_modeset_lock.c | 44 ++++
drivers/gpu/drm/i915/display/g4x_dp.c | 17 +-
drivers/gpu/drm/i915/display/intel_audio.c | 17 +-
drivers/gpu/drm/i915/display/intel_ddi.c | 16 +-
drivers/gpu/drm/i915/display/intel_display.c | 192 ++++++++----------
.../drm/i915/display/intel_display_debugfs.c | 45 ++--
drivers/gpu/drm/i915/display/intel_pipe_crc.c | 38 ++--
include/drm/drm_modeset_lock.h | 26 +++
8 files changed, 188 insertions(+), 207 deletions(-)
--
2.31.1
