From: "Xiang, Haihao" <haihao.xiang-at-intel.com@ffmpeg.org> To: "ffmpeg-devel@ffmpeg.org" <ffmpeg-devel@ffmpeg.org> Cc: "siriushu@hotmail.com" <siriushu@hotmail.com> Subject: Re: [FFmpeg-devel] [PATCH v1 1/1] lavc/qsvenc: disable SEI frame before each P-frame Date: Mon, 7 Aug 2023 02:06:41 +0000 Message-ID: <862eb1db0a1c73268e8bc7bf300cb27856ab907d.camel@intel.com> (raw) In-Reply-To: <OSZP286MB21429531501B073BDB525F00D50FA@OSZP286MB2142.JPNP286.PROD.OUTLOOK.COM> On So, 2023-08-06 at 19:54 +0800, Ting Hu wrote: > From: tinghu3 <siriushu@hotmail.com> > > In order to save bandwith for video conference application, > set pic_timing_sei of av_option to disable SEI frames. > It doesn't take effect. > > Signed-off-by: tinghu3 <siriushu@hotmail.com> > --- > libavcodec/qsvenc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c > index df63c182b0..7a5ea5391e 100644 > --- a/libavcodec/qsvenc.c > +++ b/libavcodec/qsvenc.c > @@ -937,7 +937,7 @@ static int init_video_param(AVCodecContext *avctx, > QSVEncContext *q) > q->extco.Header.BufferSz = sizeof(q->extco); > > q->extco.PicTimingSEI = q->pic_timing_sei ? > - MFX_CODINGOPTION_ON : > MFX_CODINGOPTION_UNKNOWN; > + MFX_CODINGOPTION_ON : > MFX_CODINGOPTION_OFF; PicTimingSEI and RefPicMarkRep are tristate flags. When a tristate flag is set to MFX_CODINGOPTION_UNKNOWN(0), the SDK encoder can choose a best configuration based on the current state and settings. We want to the SDK encoder set reasonable values for these flags Thanks Haihao > q->old_pic_timing_sei = q->pic_timing_sei; > > if (q->rdo >= 0) > @@ -968,6 +968,7 @@ static int init_video_param(AVCodecContext *avctx, > QSVEncContext *q) > q->extco.AUDelimiter = q->aud ? MFX_CODINGOPTION_ON : > MFX_CODINGOPTION_OFF; > } > > + q->extco.RefPicMarkRep = q->pic_timing_sei ? MFX_CODINGOPTION_ON : > MFX_CODINGOPTION_OFF; > q->extparam_internal[q->nb_extparam_internal++] = (mfxExtBuffer *)&q- > >extco; > > if (avctx->codec_id == AV_CODEC_ID_H264) { _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next prev parent reply other threads:[~2023-08-07 2:06 UTC|newest] Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-08-06 11:54 Ting Hu 2023-08-07 2:06 ` Xiang, Haihao [this message] 2023-08-08 12:39 ` sirius hu 2023-08-10 8:37 ` Xiang, Haihao
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=862eb1db0a1c73268e8bc7bf300cb27856ab907d.camel@intel.com \ --to=haihao.xiang-at-intel.com@ffmpeg.org \ --cc=ffmpeg-devel@ffmpeg.org \ --cc=siriushu@hotmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git