On 12 September 2017 at 14:37, Maarten Lankhorst <[email protected]> wrote: > When we want to make drm_atomic_commit interruptible, there are a lot of > places that call the lock function, which we don't have control over. > > Rather than trying to convert every single one, it's easier to toggle > interruptible waiting per acquire_ctx. If drm_modeset_acquire_init is > called with DRM_MODESET_ACQUIRE_INTERRUPTIBLE, then we will perform > interruptible waits in drm_modeset_lock and drm_modeset_backoff. > > Changes since v1: > - Fix locking example in drm_modeset_lock.c to be compatible > with interruptible waiting (xexaxo) and make it default. > Uninterruptible waiting shouldn't happen except in corner cases, > but the example will still apply if the flag is removed. > - Add drm_modeset_lock_single_interruptible() to documentation. > - Fix dead link to removed drm_modeset_lock_interruptible() in > drm_modeset_lock(). > > Signed-off-by: Maarten Lankhorst <[email protected]> > Reviewed-by: Daniel Vetter <[email protected]> #v1 > Cc: Emil Velikov <[email protected]> Thanks for the documentation updates/fixes Maarten. FWIW the series is Reviewed-by: Emil Velikov <[email protected]>
-Emil _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
