[libav-devel] [PATCH] qsv: adding Multi Frame Encode support

Li, Zhong zhong.li at intel.com
Wed Mar 28 11:05:11 CEST 2018


> > > a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index
> > > f6b1a0d67..a8b446c5b
> > > 100644
> > > --- a/libavcodec/qsvenc.c
> > > +++ b/libavcodec/qsvenc.c
> > > @@ -135,7 +135,7 @@ static void dump_video_param(AVCodecContext
> > > *avctx, QSVEncContext *q,  #if QSV_HAVE_CO2
> > >      mfxExtCodingOption2 *co2 =
> > > (mfxExtCodingOption2*)coding_opts[1];
> > >  #endif
> > > -#if QSV_HAVE_CO3
> > > +#if QSV_HAVE_CO3 && QSV_HAVE_QVBR
> > >      mfxExtCodingOption3 *co3 =
> > > (mfxExtCodingOption3*)coding_opts[2];
> > >  #endif
> > >
> >
> > Not sure why we need this change and how it impacts MFE.
> >
> see co3 used only when  QSV_HAVE_QVBR - so it is logical and fixes
> compilation issue.
> 
Yes it is, at least correct for current implantation. 
As I know co3 has other options than QVBR cases. 
But it is also ok to remove QSV_HAVE_QVBR when other co3 options are added.


More information about the libav-devel mailing list