[FFmpeg-devel] [PATCH] avcodec/dxva2_av1: fix superres_denom parameter

Xiang, Haihao haihao.xiang at intel.com
Wed Sep 8 08:46:40 EEST 2021


On Wed, 2021-09-08 at 13:19 +0800, Tong Wu wrote:
> Defined in spec 5.9.8. When superres is enabled, SuperresDenom equals
> "coded_denom + SUPERRES_DENOM_MIN" instead of coded_denom.
> 
> Signed-off-by: Tong Wu <tong1.wu at intel.com>
> ---
>  libavcodec/dxva2_av1.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/dxva2_av1.c b/libavcodec/dxva2_av1.c
> index aa14e473df..c30b57799c 100644
> --- a/libavcodec/dxva2_av1.c
> +++ b/libavcodec/dxva2_av1.c
> @@ -73,7 +73,7 @@ static int fill_picture_parameters(const AVCodecContext
> *avctx, AVDXVAContext *c
>      pp->max_height = seq->max_frame_height_minus_1 + 1;
>  
>      pp->CurrPicTextureIndex = ff_dxva2_get_surface_index(avctx, ctx, h-
> >cur_frame.tf.f);
> -    pp->superres_denom      = frame_header->use_superres ? frame_header-
> >coded_denom : AV1_SUPERRES_NUM;
> +    pp->superres_denom      = frame_header->use_superres ? frame_header-
> >coded_denom + AV1_SUPERRES_DENOM_MIN : AV1_SUPERRES_NUM;
>      pp->bitdepth            = get_bit_depth_from_seq(seq);
>      pp->seq_profile         = seq->seq_profile;
>  

LGTM

-Haihao


More information about the ffmpeg-devel mailing list