[FFmpeg-devel] [PATCH v1 1/2] codec: vrawdepay: add decoder for RFC4175

[Damien Riegel]

On Fri, Feb 17, 2017 at 03:01:05PM -0500, Damien Riegel wrote:
> Hi,
> 
> On Thu, Feb 16, 2017 at 06:19:00PM +0000, Rostislav Pehlivanov wrote:
> > >
> > > No, do this in libavfilter and do not introduce another useless pseudo
> > > codec
> > >
> > 
> > *libavformat, sorry
> 
> The advantage of using a pseudo codec just to depack the stream is that
> the input and the codec are in separate threads in ffmpeg, so it can
> handle a heavier workload.

Please find attached a v2, with the implementation in libavformat. Note
that I don't want to send it as a patch of its own because the
performance issue is not addressed.

Basically, our test case is a raw input stream YUV 4:2:2 10 bits 1080p
at 60fps. With the pseudo-codec, we are able to transcode it to h264 and
dump it to a file. With unpacking done in the libavformat, the input
thread gets too busy and can't stand the load.

In the implementation you made [1] unpacking was done in libavcodec, so
why is it not an acceptable solution for mainline?

[1] https://github.com/funman/ffmpeg-sdi

Thanks,
-- 
Damien
-------------- next part --------------
A non-text attachment was scrubbed...
Name: v2-0001-rtp-rawvideo-add-handler-for-YCbCr-4-2-2.patch
Type: text/x-diff
Size: 11170 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170222/5f6038f9/attachment.patch>