summaryrefslogtreecommitdiffstats
path: root/video
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2013-12-10 19:07:29 +0100
committerwm4 <wm4@nowhere>2013-12-10 20:07:39 +0100
commit9838bf55657c3944150afec44668c3c3b559c1ff (patch)
tree6c4b37bddb6a7594a7abe361f2c5d2bad1280039 /video
parent218b9d3737134a9f619c49992a5fb34ad9e3733b (diff)
downloadmpv-9838bf55657c3944150afec44668c3c3b559c1ff.tar.bz2
mpv-9838bf55657c3944150afec44668c3c3b559c1ff.tar.xz
video: move video filter chain initialization from decoder to player
This should help fixing some issues (like not draining video frames correctly on reinit), as well as decoupling the decoder, filter chain, and VO code. I also wanted to make the hardware video decoding fallback work properly if software-only video filters are inserted. This currently has the issue that the fallback is too violent, and throws away a bunch of demuxer packets needed to restart software decoding properly. But keeping "backup" packets turned out as too hacky, so I'm not doing this, at least not yet.
Diffstat (limited to 'video')
-rw-r--r--video/decode/dec_video.c11
-rw-r--r--video/decode/dec_video.h9
-rw-r--r--video/decode/lavc.h2
-rw-r--r--video/decode/vd.h7
-rw-r--r--video/decode/vd_lavc.c70
5 files changed, 37 insertions, 62 deletions
diff --git a/video/decode/dec_video.c b/video/decode/dec_video.c
index 9a43ada4c4..4e97ef33ef 100644
--- a/video/decode/dec_video.c
+++ b/video/decode/dec_video.c
@@ -114,11 +114,6 @@ int video_get_colors(struct dec_video *d_video, const char *item, int *value)
return 0;
}
-void video_reinit_vo(struct dec_video *d_video)
-{
- video_vd_control(d_video, VDCTRL_REINIT_VO, NULL);
-}
-
void video_uninit(struct dec_video *d_video)
{
if (d_video->vd_driver) {
@@ -377,15 +372,13 @@ struct mp_image *video_decode(struct dec_video *d_video,
return mpi;
}
-int mpcodecs_reconfig_vo(struct dec_video *d_video,
- const struct mp_image_params *params)
+int video_reconfig_filters(struct dec_video *d_video,
+ const struct mp_image_params *params)
{
struct MPOpts *opts = d_video->opts;
struct mp_image_params p = *params;
struct sh_video *sh = d_video->header->video;
- d_video->vf_reconfig_count++;
-
mp_msg(MSGT_DECVIDEO, MSGL_V,
"VIDEO: %dx%d %5.3f fps %5.1f kbps (%4.1f kB/s)\n",
p.w, p.h, sh->fps, sh->i_bps * 0.008,
diff --git a/video/decode/dec_video.h b/video/decode/dec_video.h
index 29e23ff322..7d30e6fd91 100644
--- a/video/decode/dec_video.h
+++ b/video/decode/dec_video.h
@@ -32,13 +32,14 @@ struct dec_video {
struct MPOpts *opts;
struct vf_chain *vfilter; // video filter chain
const struct vd_functions *vd_driver;
- long vf_reconfig_count; // incremented each mpcodecs_reconfig_vo() call
- struct mp_image_params vf_input; // video filter input params
struct mp_hwdec_info hwdec_info; // video output hwdec handles
struct sh_stream *header;
char *decoder_desc;
+ struct mp_image_params decoder_output; // last output of the decoder
+ struct mp_image_params vf_input; // video filter input params
+
void *priv; // for free use by vd_driver
// Last PTS from decoder (set with each vd_driver->decode() call)
@@ -89,7 +90,9 @@ struct mp_image *video_decode(struct dec_video *d_video,
int video_get_colors(struct dec_video *d_video, const char *item, int *value);
int video_set_colors(struct dec_video *d_video, const char *item, int value);
void video_reset_decoding(struct dec_video *d_video);
-void video_reinit_vo(struct dec_video *d_video);
int video_vd_control(struct dec_video *d_video, int cmd, void *arg);
+int video_reconfig_filters(struct dec_video *d_video,
+ const struct mp_image_params *params);
+
#endif /* MPLAYER_DEC_VIDEO_H */
diff --git a/video/decode/lavc.h b/video/decode/lavc.h
index 5007b80a0f..c623700089 100644
--- a/video/decode/lavc.h
+++ b/video/decode/lavc.h
@@ -30,8 +30,6 @@ typedef struct lavc_ctx {
enum AVPixelFormat pix_fmt;
int do_hw_dr1;
int best_csp;
- struct mp_image_params image_params;
- struct mp_image_params vo_image_params;
enum AVDiscard skip_frame;
const char *software_fallback_decoder;
diff --git a/video/decode/vd.h b/video/decode/vd.h
index bf3a8e4374..35c2739aa8 100644
--- a/video/decode/vd.h
+++ b/video/decode/vd.h
@@ -42,12 +42,9 @@ typedef struct vd_functions
extern const vd_functions_t *const mpcodecs_vd_drivers[];
enum vd_ctrl {
- VDCTRL_GET_PARAMS = 1, // retrieve struct mp_image_params
- VDCTRL_RESET, // reset decode state after seeking
+ VDCTRL_RESET = 1, // reset decode state after seeking
VDCTRL_QUERY_UNSEEN_FRAMES, // current decoder lag
- VDCTRL_REINIT_VO, // reinit filter/VO chain
+ VDCTRL_FORCE_HWDEC_FALLBACK, // force software decoding fallback
};
-int mpcodecs_reconfig_vo(struct dec_video *vd, const struct mp_image_params *params);
-
#endif /* MPLAYER_VD_H */
diff --git a/video/decode/vd_lavc.c b/video/decode/vd_lavc.c
index 7683f6c6de..ed87a16b50 100644
--- a/video/decode/vd_lavc.c
+++ b/video/decode/vd_lavc.c
@@ -383,8 +383,6 @@ static void init_avctx(struct dec_video *vd, const char *decoder,
ctx->do_dr1 = ctx->do_hw_dr1 = 0;
ctx->pix_fmt = AV_PIX_FMT_NONE;
- ctx->image_params = (struct mp_image_params){0};
- ctx->vo_image_params = (struct mp_image_params){0};
ctx->hwdec = hwdec;
ctx->avctx = avcodec_alloc_context3(lavc_codec);
AVCodecContext *avctx = ctx->avctx;
@@ -494,7 +492,8 @@ static void uninit(struct dec_video *vd)
uninit_avctx(vd);
}
-static void update_image_params(struct dec_video *vd, AVFrame *frame)
+static void update_image_params(struct dec_video *vd, AVFrame *frame,
+ struct mp_image_params *out_params)
{
vd_ffmpeg_ctx *ctx = vd->priv;
int width = frame->width;
@@ -514,7 +513,7 @@ static void update_image_params(struct dec_video *vd, AVFrame *frame)
int d_w, d_h;
vf_set_dar(&d_w, &d_h, width, height, aspect);
- ctx->image_params = (struct mp_image_params) {
+ *out_params = (struct mp_image_params) {
.imgfmt = ctx->best_csp,
.w = width,
.h = height,
@@ -741,47 +740,26 @@ static int decode(struct dec_video *vd, struct demux_packet *packet,
if (!got_picture)
return 0;
- update_image_params(vd, ctx->pic);
+ struct mp_image_params params;
+ update_image_params(vd, ctx->pic, &params);
vd->codec_pts = mp_pts_from_av(ctx->pic->pkt_pts, NULL);
vd->codec_dts = mp_pts_from_av(ctx->pic->pkt_dts, NULL);
// Note: potentially resets ctx->pic as it is transferred to mpi
struct mp_image *mpi = image_from_decoder(vd);
assert(mpi->planes[0]);
- mp_image_set_params(mpi, &ctx->image_params);
+ mp_image_set_params(mpi, &params);
if (ctx->hwdec && ctx->hwdec->process_image)
mpi = ctx->hwdec->process_image(ctx, mpi);
- struct mp_image_params vo_params;
- mp_image_params_from_image(&vo_params, mpi);
-
- if (!mp_image_params_equals(&vo_params, &ctx->vo_image_params)) {
- mp_image_pool_clear(ctx->non_dr1_pool);
- if (mpcodecs_reconfig_vo(vd, &vo_params) < 0) {
- talloc_free(mpi);
- return -1;
- }
- ctx->vo_image_params = vo_params;
- }
-
*out_image = mpi;
return 1;
}
-static struct mp_image *decode_with_fallback(struct dec_video *vd,
- struct demux_packet *packet, int flags)
+static int force_fallback(struct dec_video *vd)
{
vd_ffmpeg_ctx *ctx = vd->priv;
- if (!ctx->avctx)
- return NULL;
-
- struct mp_image *mpi = NULL;
- int res = decode(vd, packet, flags, &mpi);
- if (res >= 0)
- return mpi;
-
- // Failed hardware decoding? Try again in software.
if (ctx->software_fallback_decoder) {
uninit_avctx(vd);
mp_tmsg(MSGT_DECVIDEO, MSGL_ERR, "Error using hardware "
@@ -789,16 +767,27 @@ static struct mp_image *decode_with_fallback(struct dec_video *vd,
const char *decoder = ctx->software_fallback_decoder;
ctx->software_fallback_decoder = NULL;
init_avctx(vd, decoder, NULL);
- if (ctx->avctx) {
- mpi = NULL;
- if (vd->vfilter && vd->vfilter->initialized < 0)
- vd->vfilter->initialized = 0;
+ return ctx->avctx ? CONTROL_OK : CONTROL_ERROR;
+ }
+ return CONTROL_FALSE;
+}
+
+static struct mp_image *decode_with_fallback(struct dec_video *vd,
+ struct demux_packet *packet, int flags)
+{
+ vd_ffmpeg_ctx *ctx = vd->priv;
+ if (!ctx->avctx)
+ return NULL;
+
+ struct mp_image *mpi = NULL;
+ int res = decode(vd, packet, flags, &mpi);
+ if (res < 0) {
+ // Failed hardware decoding? Try again in software.
+ if (force_fallback(vd) == CONTROL_OK)
decode(vd, packet, flags, &mpi);
- return mpi;
- }
}
- return NULL;
+ return mpi;
}
static int control(struct dec_video *vd, int cmd, void *arg)
@@ -816,13 +805,8 @@ static int control(struct dec_video *vd, int cmd, void *arg)
delay += avctx->thread_count - 1;
*(int *)arg = delay;
return CONTROL_TRUE;
- case VDCTRL_REINIT_VO:
- if (ctx->vo_image_params.imgfmt)
- mpcodecs_reconfig_vo(vd, &ctx->vo_image_params);
- return true;
- case VDCTRL_GET_PARAMS:
- *(struct mp_image_params *)arg = ctx->vo_image_params;
- return ctx->vo_image_params.imgfmt ? true : CONTROL_NA;
+ case VDCTRL_FORCE_HWDEC_FALLBACK:
+ return force_fallback(vd);
}
return CONTROL_UNKNOWN;
}