[FFmpeg-devel] [PATCH v2] avformat/hlsenc: add option hls_delete_threshold

[Steven Liu]

> On 12 Apr 2018, at 08:38, Aman Gupta <ffmpeg at tmm1.net> wrote:
> 
> From: Steven Liu <lq at chinaffmpeg.org>
> 
> When using hls_list_size with hls_flags delete_segments, currently
> hls_list_size * 2 +- segments remain on disk. With this new option,
> the amount of disk space used can be controlled by the user.
> 
> fix ticket: #7131
> 
> Signed-off-by: Steven Liu <lq at chinaffmpeg.org>
> Signed-off-by: Aman Gupta <aman at tmm1.net>
> ---
> doc/muxers.texi      | 6 ++++++
> libavformat/hlsenc.c | 9 +++++++++
> 2 files changed, 15 insertions(+)
> 
> diff --git a/doc/muxers.texi b/doc/muxers.texi
> index cb75c261c5..f288764a23 100644
> --- a/doc/muxers.texi
> +++ b/doc/muxers.texi
> @@ -513,6 +513,12 @@ Segment will be cut on the next key frame after this time has passed.
> Set the maximum number of playlist entries. If set to 0 the list file
> will contain all the segments. Default value is 5.
> 
> + at item hls_delete_threshold @var{size}
> +Set the number of unreferenced segments to keep on disk before @code{hls_flags delete_segments}
> +deletes them. Increase this to allow continue clients to download segments which
> +were recently referenced in the playlist. Default value is 1, meaning segments older than
> + at code{hls_list_size+1} will be deleted.
> +
> @item hls_ts_options @var{options_list}
> Set output format options using a :-separated list of key=value
> parameters. Values containing @code{:} special characters must be
> diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
> index 8eb84212a0..1b4f84b653 100644
> --- a/libavformat/hlsenc.c
> +++ b/libavformat/hlsenc.c
> @@ -173,6 +173,7 @@ typedef struct HLSContext {
>     float time;            // Set by a private option.
>     float init_time;       // Set by a private option.
>     int max_nb_segments;   // Set by a private option.
> +    int hls_delete_threshold; // Set by a private option.
> #if FF_API_HLS_WRAP
>     int  wrap;             // Set by a private option.
> #endif
> @@ -445,6 +446,7 @@ static int hls_delete_old_segments(AVFormatContext *s, HLSContext *hls,
>     HLSSegment *segment, *previous_segment = NULL;
>     float playlist_duration = 0.0f;
>     int ret = 0, path_size, sub_path_size;
> +    int segment_cnt = 0;
>     char *dirname = NULL, *p, *sub_path;
>     char *path = NULL;
>     AVDictionary *options = NULL;
> @@ -458,14 +460,20 @@ static int hls_delete_old_segments(AVFormatContext *s, HLSContext *hls,
>     }
> 
>     segment = vs->old_segments;
> +    segment_cnt = 0;
>     while (segment) {
>         playlist_duration -= segment->duration;
>         previous_segment = segment;
>         segment = previous_segment->next;
> +        segment_cnt++;
>         if (playlist_duration <= -previous_segment->duration) {
>             previous_segment->next = NULL;
>             break;
>         }
> +        if (segment_cnt >= hls->hls_delete_threshold) {
> +            previous_segment->next = NULL;
> +            break;
> +        }
>     }
> 
>     if (segment && !hls->use_localtime_mkdir) {
> @@ -2799,6 +2807,7 @@ static const AVOption options[] = {
>     {"hls_time",      "set segment length in seconds",           OFFSET(time),    AV_OPT_TYPE_FLOAT,  {.dbl = 2},     0, FLT_MAX, E},
>     {"hls_init_time", "set segment length in seconds at init list",           OFFSET(init_time),    AV_OPT_TYPE_FLOAT,  {.dbl = 0},     0, FLT_MAX, E},
>     {"hls_list_size", "set maximum number of playlist entries",  OFFSET(max_nb_segments),    AV_OPT_TYPE_INT,    {.i64 = 5},     0, INT_MAX, E},
> +    {"hls_delete_threshold", "set number of unreferenced segments to keep before deleting",  OFFSET(hls_delete_threshold),    AV_OPT_TYPE_INT,    {.i64 = 1},     1, INT_MAX, E},
>     {"hls_ts_options","set hls mpegts list of options for the container format used for hls", OFFSET(format_options_str), AV_OPT_TYPE_STRING, {.str = NULL},  0, 0,    E},
>     {"hls_vtt_options","set hls vtt list of options for the container format used for hls", OFFSET(vtt_format_options_str), AV_OPT_TYPE_STRING, {.str = NULL},  0, 0,    E},
> #if FF_API_HLS_WRAP
> -- 
> 2.14.2
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

will apply

Thanks
Steven