[FFmpeg-devel] [PATCH 6/8] lavc/vaapi_hevc: add function to find exact va_profile for REXT

Fu, Linjie linjie.fu at intel.com
Sun Jan 12 10:16:57 EET 2020


> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
> Mark Thompson
> Sent: Thursday, January 9, 2020 06:44
> To: ffmpeg-devel at ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH 6/8] lavc/vaapi_hevc: add function to
> find exact va_profile for REXT
> 
> On 29/12/2019 16:30, Linjie Fu wrote:
> > Add vaapi_parse_rext_profile and use profile constraint flags to
> > determine the exact va_profile for HEVC_REXT.
> >
> > Add build object in Makefile for h265_profile_level dependency.
> >
> > Signed-off-by: Linjie Fu <linjie.fu at intel.com>
> > ---
> >  libavcodec/Makefile     |  2 +-
> >  libavcodec/vaapi_hevc.c | 69
> +++++++++++++++++++++++++++++++++++++++++++++++++
> >  libavcodec/vaapi_hevc.h | 24 +++++++++++++++++
> >  3 files changed, 94 insertions(+), 1 deletion(-)
> >  create mode 100644 libavcodec/vaapi_hevc.h
> >
> > diff --git a/libavcodec/Makefile b/libavcodec/Makefile
> > index c1f35b4..7cb914d 100644
> > --- a/libavcodec/Makefile
> > +++ b/libavcodec/Makefile
> > @@ -884,7 +884,7 @@ OBJS-$(CONFIG_HEVC_D3D11VA_HWACCEL)       +=
> dxva2_hevc.o
> >  OBJS-$(CONFIG_HEVC_DXVA2_HWACCEL)         += dxva2_hevc.o
> >  OBJS-$(CONFIG_HEVC_NVDEC_HWACCEL)         += nvdec_hevc.o
> >  OBJS-$(CONFIG_HEVC_QSV_HWACCEL)           += qsvdec_h2645.o
> > -OBJS-$(CONFIG_HEVC_VAAPI_HWACCEL)         += vaapi_hevc.o
> > +OBJS-$(CONFIG_HEVC_VAAPI_HWACCEL)         += vaapi_hevc.o
> h265_profile_level.o
> >  OBJS-$(CONFIG_HEVC_VDPAU_HWACCEL)         += vdpau_hevc.o
> >  OBJS-$(CONFIG_MJPEG_NVDEC_HWACCEL)        += nvdec_mjpeg.o
> >  OBJS-$(CONFIG_MJPEG_VAAPI_HWACCEL)        += vaapi_mjpeg.o
> > diff --git a/libavcodec/vaapi_hevc.c b/libavcodec/vaapi_hevc.c
> > index ab48b73..0604e23 100644
> > --- a/libavcodec/vaapi_hevc.c
> > +++ b/libavcodec/vaapi_hevc.c
> > @@ -27,6 +27,8 @@
> >  #include "hevcdec.h"
> >  #include "hwaccel.h"
> >  #include "vaapi_decode.h"
> > +#include "vaapi_hevc.h"
> > +#include "h265_profile_level.h"
> >
> >  typedef struct VAAPIDecodePictureHEVC {
> >      VAPictureParameterBufferHEVC pic_param;
> > @@ -494,6 +496,73 @@ static int
> vaapi_hevc_decode_slice(AVCodecContext *avctx,
> >      return 0;
> >  }
> >
> > +static int ptl_convert(const PTLCommon *general_ptl,
> H265RawProfileTierLevel *h265_raw_ptl)
> > +{
> > +    h265_raw_ptl->general_profile_space = general_ptl->profile_space;
> > +    h265_raw_ptl->general_tier_flag     = general_ptl->tier_flag;
> > +    h265_raw_ptl->general_profile_idc   = general_ptl->profile_idc;
> > +
> > +    memcpy(h265_raw_ptl->general_profile_compatibility_flag,
> > +                                  general_ptl->profile_compatibility_flag, 32 * sizeof(int));
> 
> Not int!

Fixed, thanks.

> 
> > +
> > +    h265_raw_ptl->general_progressive_source_flag          = general_ptl-
> >progressive_source_flag;
> > +    h265_raw_ptl->general_interlaced_source_flag           = general_ptl-
> >interlaced_source_flag;
> > +    h265_raw_ptl->general_non_packed_constraint_flag       = general_ptl-
> >non_packed_constraint_flag;
> > +    h265_raw_ptl->general_frame_only_constraint_flag       = general_ptl-
> >frame_only_constraint_flag;
> > +    h265_raw_ptl->general_max_12bit_constraint_flag        = general_ptl-
> >max_12bit_constraint_flag;
> > +    h265_raw_ptl->general_max_10bit_constraint_flag        = general_ptl-
> >max_10bit_constraint_flag;
> > +    h265_raw_ptl->general_max_8bit_constraint_flag         = general_ptl-
> >max_8bit_constraint_flag;
> > +    h265_raw_ptl->general_max_422chroma_constraint_flag    =
> general_ptl->max_422chroma_constraint_flag;
> > +    h265_raw_ptl->general_max_420chroma_constraint_flag    =
> general_ptl->max_420chroma_constraint_flag;
> > +    h265_raw_ptl->general_max_monochrome_constraint_flag   =
> general_ptl->max_monochrome_constraint_flag;
> > +    h265_raw_ptl->general_intra_constraint_flag            = general_ptl-
> >intra_constraint_flag;
> > +    h265_raw_ptl->general_one_picture_only_constraint_flag =
> general_ptl->one_picture_only_constraint_flag;
> > +    h265_raw_ptl->general_lower_bit_rate_constraint_flag   = general_ptl-
> >lower_bit_rate_constraint_flag;
> > +    h265_raw_ptl->general_max_14bit_constraint_flag        = general_ptl-
> >max_14bit_constraint_flag;
> > +    h265_raw_ptl->general_inbld_flag                       = general_ptl->inbld_flag;
> > +    h265_raw_ptl->general_level_idc                        = general_ptl->level_idc;
> 
> The names are all identical, so a little macroing could make this look a little
> less horrible.
> 
> #define copy_field(name) h265_raw_ptl->general_ ## name = general_ptl-
> >name
> 
> > +
> > +    return 0;
> > +}
> > +
> > +/*
> > + * Find exact va_profile for HEVC Range Extension
> > + */
> > +VAProfile ff_vaapi_parse_rext_profile(AVCodecContext *avctx)
> 
> Put hevc in the name as well.
> 
> > +{
> > +    const HEVCContext *h = avctx->priv_data;
> > +    const HEVCSPS *sps = h->ps.sps;
> > +    const PTL *ptl = &(sps->ptl);
> > +    const PTLCommon *general_ptl = &(ptl->general_ptl);
> > +    const H265ProfileDescriptor *profile = NULL;
> > +
> > +    H265RawProfileTierLevel *h265_raw_ptl =
> av_mallocz(sizeof(H265RawProfileTierLevel));
> 
> This structure isn't huge, it could be on the stack.
> 
> > +    /* convert PTLCommon to H265RawProfileTierLevel */
> > +    ptl_convert(general_ptl, h265_raw_ptl);
> > +
> > +    profile = ff_h265_get_profile(h265_raw_ptl);
> > +    av_freep(&h265_raw_ptl);
> > +
> > +    if (!profile)
> > +        return VAProfileNone;
> 
> I'd print out what the profile you find here is at VERBOSE, that would be very
> useful for working out why a particular stream isn't working.
> > +
> > +#if VA_CHECK_VERSION(1, 2, 0)
> > +    if (!strcmp(profile->name, "Main 4:2:2 10") ||
> > +        !strcmp(profile->name, "Main 4:2:2 10 Intra"))
> > +        return VAProfileHEVCMain422_10;
> > +    else if (!strcmp(profile->name, "Main 4:4:4") ||
> > +             !strcmp(profile->name, "Main 4:4:4 Intra"))
> > +        return VAProfileHEVCMain444;
> > +    else if (!strcmp(profile->name, "Main 4:4:4 10") ||
> > +             !strcmp(profile->name, "Main 4:4:4 10 Intra"))
> > +        return VAProfileHEVCMain444_10;
> > +#else
> > +    av_log(avctx, AV_LOG_WARNING, "HEVC profile %s is "
> > +                        "not supported with this VA version.\n", profile->name);
> 
> You get this message if you built with an old version but it couldn't have been
> supported anyway, which is a bit strange.  (Maybe it doesn't matter, though.)
> 
> > +#endif
> > +    return VAProfileNone;

Thanks for the review.

BTW, after some experiments on decoding with  -allow_profile_mismatch, I found that instead of
returning VAProfileNone, it'll be more compatible for user to get  a default VAProfile returned for
HEVC if AV_HWACCEL_FLAG_ALLOW_PROFILE_MISMATCH flag is set.
( Main profile for example, default selected in ff_h265_guess_level)  

Will updated a new version to fix  and cover the profile mismatch allowed situation.





More information about the ffmpeg-devel mailing list