[FFmpeg-devel] [PATCH] vaapi_h265: general_level_idc should times 3.

[Mark Thompson]

On 28/11/17 07:46, Ruiling Song wrote:
> Signed-off-by: Ruiling Song <ruiling.song at intel.com>
> ---
>  libavcodec/vaapi_encode_h265.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/vaapi_encode_h265.c b/libavcodec/vaapi_encode_h265.c
> index 3ae92a7..32b8bc6 100644
> --- a/libavcodec/vaapi_encode_h265.c
> +++ b/libavcodec/vaapi_encode_h265.c
> @@ -219,7 +219,7 @@ static int vaapi_encode_h265_init_sequence_params(AVCodecContext *avctx)
>          .general_non_packed_constraint_flag = 1,
>          .general_frame_only_constraint_flag = 1,
>  
> -        .general_level_idc     = avctx->level,
> +        .general_level_idc     = avctx->level * 3,
>      };
>      vps->profile_tier_level.general_profile_compatibility_flag[avctx->profile & 31] = 1;
>  
> 
The documentation has always said "profile and level set the values of general_profile_idc and general_level_idc respectively" (<http://ffmpeg.org/ffmpeg-codecs.html#VAAPI-encoders>) - the code disagreed for a while, but that was made consistent in 00179664bccd1dd6fa0d1c40db453528757bf6f7.

Why do you want to change it to be general_level_idc / 3 instead?

Thanks,

- Mark