Commit f9150c8a authored by Hendrik Leppkes's avatar Hendrik Leppkes Committed by Janne Grunau

lagarith: frame multithreading

About 2x speedup going from 1 to 2 threads.
1.7s to 0.85s on foreman CIF.
Signed-off-by: 's avatarJanne Grunau <janne-libav@jannau.net>
parent 36a8c430
...@@ -30,6 +30,7 @@ ...@@ -30,6 +30,7 @@
#include "mathops.h" #include "mathops.h"
#include "dsputil.h" #include "dsputil.h"
#include "lagarithrac.h" #include "lagarithrac.h"
#include "thread.h"
enum LagarithFrameType { enum LagarithFrameType {
FRAME_RAW = 1, /**< uncompressed */ FRAME_RAW = 1, /**< uncompressed */
...@@ -506,7 +507,7 @@ static int lag_decode_frame(AVCodecContext *avctx, ...@@ -506,7 +507,7 @@ static int lag_decode_frame(AVCodecContext *avctx,
AVFrame *picture = data; AVFrame *picture = data;
if (p->data[0]) if (p->data[0])
avctx->release_buffer(avctx, p); ff_thread_release_buffer(avctx, p);
p->reference = 0; p->reference = 0;
p->key_frame = 1; p->key_frame = 1;
...@@ -520,7 +521,7 @@ static int lag_decode_frame(AVCodecContext *avctx, ...@@ -520,7 +521,7 @@ static int lag_decode_frame(AVCodecContext *avctx,
case FRAME_SOLID_RGBA: case FRAME_SOLID_RGBA:
avctx->pix_fmt = PIX_FMT_RGB32; avctx->pix_fmt = PIX_FMT_RGB32;
if (avctx->get_buffer(avctx, p) < 0) { if (ff_thread_get_buffer(avctx, p) < 0) {
av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n"); av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
return -1; return -1;
} }
...@@ -542,7 +543,7 @@ static int lag_decode_frame(AVCodecContext *avctx, ...@@ -542,7 +543,7 @@ static int lag_decode_frame(AVCodecContext *avctx,
if (frametype == FRAME_ARITH_RGB24 || frametype == FRAME_U_RGB24) if (frametype == FRAME_ARITH_RGB24 || frametype == FRAME_U_RGB24)
avctx->pix_fmt = PIX_FMT_RGB24; avctx->pix_fmt = PIX_FMT_RGB24;
if (avctx->get_buffer(avctx, p) < 0) { if (ff_thread_get_buffer(avctx, p) < 0) {
av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n"); av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
return -1; return -1;
} }
...@@ -602,7 +603,7 @@ static int lag_decode_frame(AVCodecContext *avctx, ...@@ -602,7 +603,7 @@ static int lag_decode_frame(AVCodecContext *avctx,
case FRAME_ARITH_YUY2: case FRAME_ARITH_YUY2:
avctx->pix_fmt = PIX_FMT_YUV422P; avctx->pix_fmt = PIX_FMT_YUV422P;
if (avctx->get_buffer(avctx, p) < 0) { if (ff_thread_get_buffer(avctx, p) < 0) {
av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n"); av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
return -1; return -1;
} }
...@@ -628,7 +629,7 @@ static int lag_decode_frame(AVCodecContext *avctx, ...@@ -628,7 +629,7 @@ static int lag_decode_frame(AVCodecContext *avctx,
case FRAME_ARITH_YV12: case FRAME_ARITH_YV12:
avctx->pix_fmt = PIX_FMT_YUV420P; avctx->pix_fmt = PIX_FMT_YUV420P;
if (avctx->get_buffer(avctx, p) < 0) { if (ff_thread_get_buffer(avctx, p) < 0) {
av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n"); av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
return -1; return -1;
} }
...@@ -678,7 +679,7 @@ static av_cold int lag_decode_end(AVCodecContext *avctx) ...@@ -678,7 +679,7 @@ static av_cold int lag_decode_end(AVCodecContext *avctx)
LagarithContext *l = avctx->priv_data; LagarithContext *l = avctx->priv_data;
if (l->picture.data[0]) if (l->picture.data[0])
avctx->release_buffer(avctx, &l->picture); ff_thread_release_buffer(avctx, &l->picture);
av_freep(&l->rgb_planes); av_freep(&l->rgb_planes);
return 0; return 0;
...@@ -692,6 +693,6 @@ AVCodec ff_lagarith_decoder = { ...@@ -692,6 +693,6 @@ AVCodec ff_lagarith_decoder = {
.init = lag_decode_init, .init = lag_decode_init,
.close = lag_decode_end, .close = lag_decode_end,
.decode = lag_decode_frame, .decode = lag_decode_frame,
.capabilities = CODEC_CAP_DR1, .capabilities = CODEC_CAP_DR1 | CODEC_CAP_FRAME_THREADS,
.long_name = NULL_IF_CONFIG_SMALL("Lagarith lossless"), .long_name = NULL_IF_CONFIG_SMALL("Lagarith lossless"),
}; };
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment