[FFmpeg-devel] [PATCH] avcodec/nvenc: add udu_sei option to import user data unregistered SEIs

[lance.lmwang at gmail.com]

On Fri, Dec 24, 2021 at 04:52:35PM +0100, Timo Rothenpieler wrote:
> On 24.12.2021 15:47, lance.lmwang at gmail.com wrote:
> > 
> > You can tested with libx264 output file, the size of unregistered SEIs(for encoding
> > setting) will > 503 always, if you don't make the size <= 503, you'll see below
> > error by trace_eaders filter.
> > Invalid SEI message: payload_size too large (xxx 504 bytes).
> > 
> > Note,  libavcodec/cbs_sei_syntax_template.c line 262, I print out "get_bits_left(rw) / 8".
> > 
> > I think the buffer size is OK, but nvenc sdk truncate the data after encoding.
> 
> What exactly are you referring to by nvenc sdk? Are you looking at some
> Nvidia sample that does this truncating?

I guess it's caused by nv sdk library failed to forward the SEI data output completely.
It's not related to the patch, so I tested with small size SEIs and veirfy the option
working as expected. 

> Cause if I try this with ffmpeg right now, encoding straight up fails if a
> too large SEI segment is attached, since nvenc runs out of space in the
> output buffer.

I haven created 5 frame by libx264 and test it, so maybe it's too short to
cause encoding failed. I have used bsf filter to add user unregistered before,
it haven't such limitation or issue. 



> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


-- 
Thanks,
Limin Wang