Re: [FFmpeg-devel] [PATCH 1/6] avcodec/pthread_frame: Don't prematurily presume frame-threading

2022-02-09 Thread Anton Khirnov
patchset looks good

-- 
Anton Khirnov
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".


[FFmpeg-devel] [PATCH 1/6] avcodec/pthread_frame: Don't prematurily presume frame-threading

2022-02-06 Thread Andreas Rheinhardt
Several of our decoders support both frame- as well as slice-threading;
in case of the latter avctx->internal->thread_ctx points to
a SliceThreadContext, not to a frame-thread PerThreadContext.
So only treat avctx->internal->thread_ctx as the latter after
having checked that frame-threading is active.

Signed-off-by: Andreas Rheinhardt 
---
 libavcodec/pthread_frame.c | 10 +++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c
index 85a6bc98c1..43e097ff06 100644
--- a/libavcodec/pthread_frame.c
+++ b/libavcodec/pthread_frame.c
@@ -949,7 +949,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
 
 static int thread_get_buffer_internal(AVCodecContext *avctx, ThreadFrame *f, 
int flags)
 {
-PerThreadContext *p = avctx->internal->thread_ctx;
+PerThreadContext *p;
 int err;
 
 f->owner[0] = f->owner[1] = avctx;
@@ -957,6 +957,7 @@ static int thread_get_buffer_internal(AVCodecContext 
*avctx, ThreadFrame *f, int
 if (!(avctx->active_thread_type & FF_THREAD_FRAME))
 return ff_get_buffer(avctx, f->f, flags);
 
+p = avctx->internal->thread_ctx;
 FF_DISABLE_DEPRECATION_WARNINGS
 if (atomic_load(>state) != STATE_SETTING_UP &&
 (avctx->codec->update_thread_context
@@ -1020,10 +1021,12 @@ FF_DISABLE_DEPRECATION_WARNINGS
 enum AVPixelFormat ff_thread_get_format(AVCodecContext *avctx, const enum 
AVPixelFormat *fmt)
 {
 enum AVPixelFormat res;
-PerThreadContext *p = avctx->internal->thread_ctx;
+PerThreadContext *p;
 if (!(avctx->active_thread_type & FF_THREAD_FRAME) || 
avctx->thread_safe_callbacks ||
 avctx->get_format == avcodec_default_get_format)
 return ff_get_format(avctx, fmt);
+
+p = avctx->internal->thread_ctx;
 if (atomic_load(>state) != STATE_SETTING_UP) {
 av_log(avctx, AV_LOG_ERROR, "get_format() cannot be called after 
ff_thread_finish_setup()\n");
 return -1;
@@ -1057,7 +1060,7 @@ void ff_thread_release_buffer(AVCodecContext *avctx, 
ThreadFrame *f)
 {
 #if FF_API_THREAD_SAFE_CALLBACKS
 FF_DISABLE_DEPRECATION_WARNINGS
-PerThreadContext *p = avctx->internal->thread_ctx;
+PerThreadContext *p;
 FrameThreadContext *fctx;
 AVFrame *dst;
 int ret = 0;
@@ -1084,6 +1087,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
 return;
 }
 
+p= avctx->internal->thread_ctx;
 fctx = p->parent;
 pthread_mutex_lock(>buffer_mutex);
 
-- 
2.32.0

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".