[FFmpeg-devel] [PATCH] Add FATE test for the number of seeks performed when probing fragmented MP4

[Tomas Härdin]

ons 2024-12-11 klockan 15:41 +0100 skrev Tomas Härdin:
> ons 2024-12-11 klockan 15:24 +0100 skrev Tomas Härdin:
> > Hi
> > 
> > This patchset addresses the issues Spotify has been having reading
> > fragmented MP4 over HTTP. The issue is that mov.c does hundreds of
> > seeks unless -use_mfra_for is set to something other than auto.
> > However, defaulting -use_mfra_for to either pts or dts breaks FATE.
> > I'm
> > not sure what to do there, so this patch always reads mfra if the
> > file
> > is fragmented, which stops the entire file being parsed when
> > probing.
> 
> Oh and this of course also needs a FATE test. I didn't bother writing
> that yet before I get some feedback. But the idea I have is to check
> the number of seeks performed and require that it not be more than 4
> in
> this case. I assume we have access to sed on all systems FATE runs on

Here's a FATE test checking the number of seeks performed

/Tomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-FATE-test-for-the-number-of-seeks-performed-when.patch
Type: text/x-patch
Size: 1526 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20241213/a0550c19/attachment.bin>