2017-08-15 11:33 GMT+08:00 Steven Liu <lingjiujia...@gmail.com>:
> 2017-08-15 11:23 GMT+08:00 Jun Zhao <mypopy...@gmail.com>:
>>
>>
>> On 2017/8/15 10:41, Steven Liu wrote:
>>> 2017-08-15 8:48 GMT+08:00 Jun Zhao <mypopy...@gmail.com>:
>>>> ping?
>>> No Documentation for the coder options?
>>
>> Do you means man page or the other docs ?
> doc/encoders.texi
> This one.
>>
>>>>
>>>> On 2017/8/9 9:39, Jun Zhao wrote:
>>>>> V2: Follow libx264 "coder" option style, base on Hendrik Leppkes code
>>>>> review.
>>>>>
>>>> _______________________________________________
>>>> ffmpeg-devel mailing list
>>>> ffmpeg-devel@ffmpeg.org
>>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>> _______________________________________________
>>> ffmpeg-devel mailing list
>>> ffmpeg-devel@ffmpeg.org
>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>>
From 28e4dc3533be2b7c7493a08de7143c042c9923c1 Mon Sep 17 00:00:00 2001
From: Jun Zhao <jun.z...@intel.com>
Date: Tue, 8 Aug 2017 03:33:53 -0400
Subject: [PATCH V2] lavc/vaapi_encode_h264: add "coder" option support
Follow libx264 style to support coder option, default is
enabled cabac.
Signed-off-by: Yi A Wang <yi.a.w...@intel.com>
Signed-off-by: Jun Zhao <jun.z...@intel.com>
---
libavcodec/vaapi_encode_h264.c | 18 ++++++++++++++++--
1 file changed, 16 insertions(+), 2 deletions(-)
diff --git a/libavcodec/vaapi_encode_h264.c b/libavcodec/vaapi_encode_h264.c
index f9fcd805a4..4e1df182a0 100644
--- a/libavcodec/vaapi_encode_h264.c
+++ b/libavcodec/vaapi_encode_h264.c
@@ -168,6 +168,8 @@ typedef struct VAAPIEncodeH264Options {
int qp;
int quality;
int low_power;
+ // Entropy encoder type
+ int coder;
} VAAPIEncodeH264Options;
@@ -783,6 +785,8 @@ static int
vaapi_encode_h264_init_sequence_params(AVCodecContext *avctx)
VAEncPictureParameterBufferH264 *vpic = ctx->codec_picture_params;
VAAPIEncodeH264Context *priv = ctx->priv_data;
VAAPIEncodeH264MiscSequenceParams *mseq = &priv->misc_sequence_params;
+ VAAPIEncodeH264Options *opt =
+ (VAAPIEncodeH264Options*)ctx->codec_options_data;
int i;
{
@@ -927,8 +931,12 @@ static int
vaapi_encode_h264_init_sequence_params(AVCodecContext *avctx)
vpic->num_ref_idx_l0_active_minus1 = 0;
vpic->num_ref_idx_l1_active_minus1 = 0;
- vpic->pic_fields.bits.entropy_coding_mode_flag =
- ((avctx->profile & 0xff) != 66);
+ if (opt->coder) {
+ vpic->pic_fields.bits.entropy_coding_mode_flag =
+ ((avctx->profile & 0xff) != 66);
+ } else {
+ vpic->pic_fields.bits.entropy_coding_mode_flag = 0;
+ }
What about write like this?
vpic->pic_fields.bits.entropy_coding_mode_flag = opt->coder ?
(avctx->profile & 0xff) != 66 : 0;
vpic->pic_fields.bits.weighted_pred_flag = 0;
vpic->pic_fields.bits.weighted_bipred_idc = 0;
vpic->pic_fields.bits.transform_8x8_mode_flag =
@@ -1283,6 +1291,12 @@ static const AVOption vaapi_encode_h264_options[] = {
{ "low_power", "Use low-power encoding mode (experimental: only supported "
"on some platforms, does not support all features)",
OFFSET(low_power), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 1, FLAGS },
+ { "coder", "Entropy coder type",
+ OFFSET(coder), AV_OPT_TYPE_INT, { .i64 = 1 }, 0, 1, FLAGS, "coder" },
+ { "cavlc", NULL, 0, AV_OPT_TYPE_CONST, { .i64 = 0 }, INT_MIN,
INT_MAX, FLAGS, "coder" },
+ { "cabac", NULL, 0, AV_OPT_TYPE_CONST, { .i64 = 1 }, INT_MIN,
INT_MAX, FLAGS, "coder" },
+ { "vlc", NULL, 0, AV_OPT_TYPE_CONST, { .i64 = 0 }, INT_MIN,
INT_MAX, FLAGS, "coder" },
+ { "ac", NULL, 0, AV_OPT_TYPE_CONST, { .i64 = 1 }, INT_MIN,
INT_MAX, FLAGS, "coder" },
{ NULL },
};
--
2.11.0
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
