From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 029D3421BA for ; Thu, 24 Feb 2022 22:21:33 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id C356B68B347; Fri, 25 Feb 2022 00:21:31 +0200 (EET) Received: from mail-vs1-f47.google.com (mail-vs1-f47.google.com [209.85.217.47]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 9277868B334; Fri, 25 Feb 2022 00:21:20 +0200 (EET) Received: by mail-vs1-f47.google.com with SMTP id d11so3701143vsm.5; Thu, 24 Feb 2022 14:21:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=HYKVY2Znm6QYTVNT8Z0ogpgs2FJPeil6SRhdvgJWGg0=; b=CF2rNOobbLP3/8OXRPhuci5st7fv9KFufu34uBKD/LrlUiDTo/huaY+IQ5RsOJUAq7 gS1VaQe+jrUbfvdGBzbfxoZQJa9NZlBM2SSmp84ladi2BJktttzClevZZ+U/BigFDQCh Zq/yc+u1xyzMyfSJh8tEboq9EHP97KxJBot9XT8qA1pwxgB2/kKQ3SJZCvbcTbP33iSG WJw2QWPeUa5WowiTvPAiECnkCeSfGA8i6VB4qOa/kz77cuQpd63cND5XSVNEplyxQAKb wrcB4fc1yTWxvpPuxSje2Ghn16F8nDxA7MEA+bPRPasJnso8HumY2zo7b9zXNwzr+eIt ZOsw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=HYKVY2Znm6QYTVNT8Z0ogpgs2FJPeil6SRhdvgJWGg0=; b=i9TBQEpByhglvGvyvpJcy251PYcP06pXappVmIZaT0SHHK003vALezvmCdnasCzMQw cbGyTYFA0KCZ7angKG3idjLtAFLzD8SMuapE7VZOyOYkUSH5tBgLMz8Q+jE4fGLth7AU eUUUlx09AaCrDQfS97vGb8mL9bYEfAgLWWRuNV+I6Xuq71KpHqDV0YLCrVmu6ujBlxux ETrSxeHZYRMtyDh86nOQIXS6j9TxTswuCCjtNigRteIWN3GbDIJs78LUkGpuo7E6Zx8j Sz2QhjJrHNmm3AlcTwLAmtfpVuiG32nu8AwD4e1L2IgjEyLwacxhGPmhvF51jZGtxyBX +F/w== X-Gm-Message-State: AOAM533Da7taefB9Bene5dCKRl7Httjm/2AbL1rObyFJ2uFNPtGTlxyu 9rlWgYf905Bt1Ipy33DenYAdQx8erhocTzsDdU9dvs0v X-Google-Smtp-Source: ABdhPJxxMiQ6i7u93GWkvMjtb82c/rM/gZGQUNaOcnSi1FJaN0SvyGAosUzn2XeBRI6PYx3QA1WUf7BUJjIIYJogvTw= X-Received: by 2002:a05:6102:3750:b0:30f:8289:939a with SMTP id u16-20020a056102375000b0030f8289939amr2218284vst.14.1645741278938; Thu, 24 Feb 2022 14:21:18 -0800 (PST) MIME-Version: 1.0 References: <20220224203039.CD4BB41059B@natalya.videolan.org> In-Reply-To: From: Thierry Foucu Date: Thu, 24 Feb 2022 14:20:42 -0800 Message-ID: To: FFmpeg development discussions and patches X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_tpad: if there is no frame to clone return early X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Cc: FFmpeg git commit log and discussion Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Thu, Feb 24, 2022 at 2:19 PM Thierry Foucu wrote: > > > On Thu, Feb 24, 2022 at 1:50 PM Paul B Mahol wrote: > >> On Thu, Feb 24, 2022 at 10:36 PM Thierry Foucu wrote: >> >> > On Thu, Feb 24, 2022 at 1:28 PM Paul B Mahol wrote: >> > >> > > On Thu, Feb 24, 2022 at 10:12 PM Thierry Foucu >> wrote: >> > > >> > > > On Thu, Feb 24, 2022 at 12:30 PM Paul B Mahol >> > wrote: >> > > > >> > > > > ffmpeg | branch: master | Paul B Mahol | Thu >> Feb >> > 24 >> > > > > 20:32:41 2022 +0100| [3715f2f8643695940582ce040b7a052cccfb9db2] | >> > > > > committer: Paul B Mahol >> > > > > >> > > > > avfilter/vf_tpad: if there is no frame to clone return early >> > > > > >> > > > > > >> > > > > >> > > > >> > > >> > >> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=3715f2f8643695940582ce040b7a052cccfb9db2 >> > > > > --- >> > > > > >> > > > > libavfilter/vf_tpad.c | 5 +++++ >> > > > > 1 file changed, 5 insertions(+) >> > > > > >> > > > > diff --git a/libavfilter/vf_tpad.c b/libavfilter/vf_tpad.c >> > > > > index e5acece1e4..120dbcb4d3 100644 >> > > > > --- a/libavfilter/vf_tpad.c >> > > > > +++ b/libavfilter/vf_tpad.c >> > > > > @@ -148,6 +148,11 @@ static int activate(AVFilterContext *ctx) >> > > > > frame->data, frame->linesize, >> > > > > 0, 0, frame->width, frame->height); >> > > > > } else if (s->stop_mode == 1) { >> > > > > + if (!s->cache_stop) { >> > > > > + s->pad_stop = 0; >> > > > > + ff_outlink_set_status(outlink, AVERROR_EOF, >> s->pts); >> > > > > + return 0; >> > > > > + } >> > > > > frame = av_frame_clone(s->cache_stop); >> > > > > if (!frame) >> > > > > return AVERROR(ENOMEM); >> > > > > >> > > > > >> > > > The problem with this solution is that the tpad will then not do >> what >> > we >> > > > are expecting, which is padding video track, and the output file >> will >> > not >> > > > have the desired duration. >> > > > Will it not be better to just output black frame (aka stop_mode == >> 0) , >> > > > something like that? >> > > > >> > > >> > > I doubt that, clone is clone, there is no point in padding stream that >> > have >> > > no frames at all. >> > > >> > > >> > > >> > The sample I forwarded to JB has video frames. The problem was that >> there >> > was a re-init of the filter chain after a resolution change and after >> the >> > re-init, there were no frames but got frames before the re-init. >> > I will understand for a media file which has NEVER received a video >> frame >> > and in this case, this is the correct solution. >> > >> >> Hmm, so you encode output with resolution change, why not use scale filter >> as first filter and then no resolution ever change for rest of graph? >> Adding support for resolution changes to all filters is very time >> consuming >> task and I see no real benefit in doing that now. >> > > We do have the scale filter in front of the tpad filter. Here is the > filter chain we are using > > idet=1.04:1.5,yadif=0:-1:1,scale=+528:+864:flags=bicubic,setsar=1,fps=fps=16.601,tpad=stop_mode=clone:stop_duration=2808ms > > And it does crash. > > Even just fps=fps=16.601,tpad=stop_mode=clone:stop_duration=2808ms > reproduce the problem. > Maybe an option to the tpad filter to decide what should be the output when the stop_mode == 1 and cache_stop is NULL? So, in some cases, people may want a color frame, and some other will just want to exit right away? > > >> >> >> > >> > >> > > > >> > > > iff --git a/libavfilter/vf_tpad.c b/libavfilter/vf_tpad.c >> > > > index e5acece1e4..5e4062e9f2 100644 >> > > > --- a/libavfilter/vf_tpad.c >> > > > +++ b/libavfilter/vf_tpad.c >> > > > @@ -140,7 +140,7 @@ static int activate(AVFilterContext *ctx) >> > > > ff_outlink_set_status(outlink, AVERROR_EOF, s->pts); >> > > > return 0; >> > > > } >> > > > - if (s->stop_mode == 0) { >> > > > + if (s->stop_mode == 0 || !s->cache_stop) { >> > > > frame = ff_get_video_buffer(outlink, outlink->w, >> > > outlink->h); >> > > > if (!frame) >> > > > return AVERROR(ENOMEM); >> > > > >> > > > >> > > > >> > > > >> > > > > _______________________________________________ >> > > > > ffmpeg-cvslog mailing list >> > > > > ffmpeg-cvslog@ffmpeg.org >> > > > > https://ffmpeg.org/mailman/listinfo/ffmpeg-cvslog >> > > > > >> > > > > To unsubscribe, visit link above, or email >> > > > > ffmpeg-cvslog-request@ffmpeg.org with subject "unsubscribe". >> > > > > >> > > > >> > > > >> > > > -- >> > > > >> > > > Thierry Foucu >> > > > _______________________________________________ >> > > > 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". >> > > > >> > > _______________________________________________ >> > > 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". >> > > >> > >> > >> > -- >> > >> > Thierry Foucu >> > _______________________________________________ >> > 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". >> > >> _______________________________________________ >> 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". >> > > > -- > > Thierry Foucu > -- Thierry Foucu _______________________________________________ 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".