[FFmpeg-devel] [PATCH 0/9] Merge lazy filter initialization in ffmpeg CLI

Michael Niedermayer michael at niedermayer.cc
Tue Feb 14 19:11:22 EET 2017


On Mon, Feb 13, 2017 at 12:57:30PM +0100, Michael Niedermayer wrote:
> On Mon, Feb 13, 2017 at 10:31:19AM +0100, wm4 wrote:
> > On Fri, 10 Feb 2017 15:25:13 +0100
> > Michael Niedermayer <michael at niedermayer.cc> wrote:
> > 
> > > On Fri, Feb 10, 2017 at 03:22:28PM +0100, Michael Niedermayer wrote:
> > > > On Fri, Feb 10, 2017 at 03:15:29PM +0100, Michael Niedermayer wrote:  
> > > > > On Fri, Feb 10, 2017 at 01:35:32PM +0100, wm4 wrote:  
> > > > > > These patches merge the previously skipped Libav commits, which made
> > > > > > avconv lazily initialize libavfilter graphs. This means the filters
> > > > > > are initialized with the actual output format, instead of whatever
> > > > > > libavformat reports.
> > > > > > 
> > > > > > It's a prerequisite to making hardware decoding support saner, as
> > > > > > hardware decoders will output a different pixfmt than the software
> > > > > > format reported by libavformat. This can be seen on ffmpeg_qsv.c,
> > > > > > which doesn't lose any functionality, even though half of the code
> > > > > > is removed.
> > > > > > 
> > > > > > There are some differences in how ffmpeg.c and avconv.c filter-flow
> > > > > > works. Also, avconv.c doesn't have sub2video. Relatively intrusive
> > > > > > changes were required.
> > > > > > 
> > > > > > The status of cuvid is unknown, but work in progress.
> > > > > > 
> > > > > > Anton Khirnov (4):
> > > > > >   ffmpeg: do packet ts rescaling in write_packet()
> > > > > >   ffmpeg: init filtergraphs only after we have a frame on each input
> > > > > >   ffmpeg: move flushing the queued frames to configure_filtergraph()
> > > > > >   ffmpeg: restructure sending EOF to filters
> > > > > > 
> > > > > > Timo Rothenpieler (3):
> > > > > >   ffmpeg_cuvid: adapt for recent filter graph initialization changes
> > > > > >   avcodec/cuvid: add format mismatch debug logs
> > > > > >   avcodec/cuvid: update hw_frames_ctx reference after get_format call
> > > > > > 
> > > > > > wm4 (2):
> > > > > >   ffmpeg: make sure packets put into the muxing FIFO are refcounted
> > > > > >   ffmpeg: fix printing of filter input/output names  
> > > > > 
> > > > > This patchset breaks
> > > > > ./ffmpeg -i Voting_Machine.wmv test.avi
> > > > > 
> > > > > http://data.onas.ru/fun-clips/Voting_Machine.wmv
> > > > > 
> > > > > didnt bisect which patch causes it  
> > > > 
> > > > heres another example:
> > > > 
> > > > ./ffmpeg -i ~/tickets/4329/bogus_video.mp4 -vframes 5  -vf crop=720:404  out.mov
> > > > ./ffplay out.mov
> > > > before this patchset out.mov had an audio stream  
> > > 
> > > sample seems to be here:
> > > http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket4329/
> > > 
> > > [...]
> > > 
> > > 
> > 
> > Most of these should be fixed, new patches:
> > https://github.com/wm4/FFmpeg/commits/filter-merge
> 
> already reported on IRC:
> this breaks:
> ./ffmpeg -i ~/videos/matrixbench_mpeg2.mpg -vf scale=80x60  small.mpg && ./ffmpeg -i small.mpg  -vframes 3 -metadata compilation="1"  blah.m4a
> 
> 
> Also please repost the patchset or changed patches to the ML
> I think this needs more testing, its a large patchset

a new one

./ffmpeg -i tickets//3080/flossless_cut.avi -f null -
(with 2c7a77304a7c4f8f7670866db319613771fcde13)

seems to never exit on its own


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170214/7d277c16/attachment.sig>


More information about the ffmpeg-devel mailing list