From: Michael Niedermayer <michael@niedermayer.cc> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH v2] avformat/hls: Partially revert "reduce default max reload to 3" Date: Fri, 28 Feb 2025 00:03:07 +0100 Message-ID: <20250227230307.GK4991@pb2> (raw) In-Reply-To: <pull.51.v2.ffstaging.FFmpeg.1740674216207.ffmpegagent@gmail.com> [-- Attachment #1.1: Type: text/plain, Size: 1904 bytes --] On Thu, Feb 27, 2025 at 04:36:56PM +0000, softworkz wrote: > From: softworkz <softworkz@hotmail.com> > > (setting to 100 as a reasonable compromise) > > The change has caused regressions for many users and consumers. > Playlist reloads only happen when a playlist doesn't indicate that it > has ended (via #EXT-X-ENDLIST), which means that the addition of future > segments is still expected. > It is well possible that an HLS server is temporarily unable to serve > further segments but resumes after some time, either indicating a > discontinuity or even by fully catching up. > With a segment length of 3s, a max_reload value of 1000 corresponds to > a duration of 50 minutes which appears to be a reasonable default. > --- > avformat/hls: Partially revert "reduce default max reload to 3" > > This change has caused regressions for many users and consumers. > Playlist reloads only happen when a playlist doesn't indicate that it > has ended (via #EXT-X-ENDLIST), which means that the addition of future > segments is still expected. It is well possible that an HLS server is > temporarily unable to serve further segments but resumes after some > time, either indicating a discontinuity or even by fully catching up. > With a segment length of 3s, a max_reload value of 1000 corresponds to a > duration of 50 minutes which appears to be a reasonable default. > > V2 > > * reduced from 1000 to 100 (halfway) according to Michael's calcularion > :-) [...] > libavformat/hls.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) will probably apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Some people wanted to paint the bikeshed green, some blue and some pink. People argued and fought, when they finally agreed, only rust was left. [-- 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".
prev parent reply other threads:[~2025-02-27 23:03 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2025-01-19 8:52 [FFmpeg-devel] [PATCH] avformat/hls: Revert " softworkz 2025-01-19 9:27 ` Soft Works 2025-02-07 1:38 ` Soft Works 2025-02-07 2:24 ` Steven Liu 2025-02-07 2:39 ` Soft Works 2025-02-26 0:38 ` Michael Niedermayer 2025-02-26 23:38 ` Soft Works 2025-02-27 22:58 ` Michael Niedermayer 2025-02-28 1:39 ` Soft Works 2025-02-27 16:36 ` [FFmpeg-devel] [PATCH v2] avformat/hls: Partially revert " softworkz 2025-02-27 23:03 ` Michael Niedermayer [this message]
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20250227230307.GK4991@pb2 \ --to=michael@niedermayer.cc \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git