diff options
Diffstat (limited to 'video')
-rw-r--r-- | video/out/vo_libmpv.c | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/video/out/vo_libmpv.c b/video/out/vo_libmpv.c index 3f3127cb2a..b6c4d9fb06 100644 --- a/video/out/vo_libmpv.c +++ b/video/out/vo_libmpv.c @@ -45,6 +45,22 @@ * > mpv_render_context.update_lock * And: render thread > VO (wait for present) * VO > render thread (wait for present done, via timeout) + * + * Inherent locking bugs with advanced_control: + * + * In theory, it can deadlock on ctx->lock. Consider if the VO thread calls + * forget_frames(), which it does under ctx->lock (e.g. VOCTRL_RESET). + * If a frame was a DR image, dr_helper.c will call mp_dispatch_run(). + * This in turn will call the wakeup callback set with + * mpv_render_context_set_update_callback(). The API user will eventually + * call mpv_render_context_update(), which performs the dispatch queue work + * queued by dr_helper.c. + * And then the function tries to acquire ctx->lock. This can deadlock + * under specific circumstances. It will _not_ deadlock if the queued work + * made the caller release the lock. However, if the caller made queue + * some more work (like freeing a second frame), and will keep the lock + * until it gets a reply. Both threads will wait on each other, and no + * progress can be made. */ struct vo_priv { |