Re: [FFmpeg-devel] [PATCH 2/2] avformat/hls: reduce default max reload to 100

[Michael Niedermayer <michael@niedermayer.cc>]

[-- Attachment #1.1: Type: text/plain, Size: 1221 bytes --]

On Thu, May 11, 2023 at 07:36:04PM +0200, Paul B Mahol wrote:
> On Wed, May 10, 2023 at 11:58 PM Michael Niedermayer <michael@niedermayer.cc>
> wrote:
> 
> > The 1000 did result in a apparent never ending reload loop
> >
> >
> How so? Somewhere overflow happens causing infinity?

no, it was lack of patience
each reload before the patches takes 50seconds with the testcase
thats with one url which returns 55 bytes
i didnt benchmark it but i would expect 1000 reloads to take
about 14 hours
after the bugfix it should be a bit less than 2 hours
and with just 100 reloads its maybe 10minutes

Why is the reload taking so long ?
The RFC says this:
"If the client reloads a Playlist file and finds that it has not
   changed, then it MUST wait for a period of one-half the target
   duration before retrying."
   
Given that i think a default of 100 is actually still too much
and something like 3 is better, so i will locally change the default
to 3   

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
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".