Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH] lavfi/vf_vpp_qsv: set the right timestamp for AVERROR_EOF
@ 2023-03-17  2:38 Xiang, Haihao
  2023-03-18  0:37 ` Rogozhkin, Dmitry V
  0 siblings, 1 reply; 3+ messages in thread
From: Xiang, Haihao @ 2023-03-17  2:38 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Haihao Xiang

From: Haihao Xiang <haihao.xiang@intel.com>

Rescale the timestamp for AVERROR_EOF. This can fix tickets 10261 and
10262.

Signed-off-by: Haihao Xiang <haihao.xiang@intel.com>
---
 libavfilter/vf_vpp_qsv.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libavfilter/vf_vpp_qsv.c b/libavfilter/vf_vpp_qsv.c
index ac77f7bb4b..b4dd7a4191 100644
--- a/libavfilter/vf_vpp_qsv.c
+++ b/libavfilter/vf_vpp_qsv.c
@@ -602,6 +602,7 @@ not_ready:
     return FFERROR_NOT_READY;
 
 eof:
+    pts = av_rescale_q(pts, inlink->time_base, outlink->time_base);
     ff_outlink_set_status(outlink, status, pts);
     return 0;
 }
-- 
2.25.1

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [FFmpeg-devel] [PATCH] lavfi/vf_vpp_qsv: set the right timestamp for AVERROR_EOF
  2023-03-17  2:38 [FFmpeg-devel] [PATCH] lavfi/vf_vpp_qsv: set the right timestamp for AVERROR_EOF Xiang, Haihao
@ 2023-03-18  0:37 ` Rogozhkin, Dmitry V
  2023-03-20  2:13   ` Xiang, Haihao
  0 siblings, 1 reply; 3+ messages in thread
From: Rogozhkin, Dmitry V @ 2023-03-18  0:37 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Xiang, Haihao

On Fri, 2023-03-17 at 10:38 +0800, Xiang, Haihao wrote:
> From: Haihao Xiang <haihao.xiang@intel.com>
> 
> Rescale the timestamp for AVERROR_EOF. This can fix tickets 10261 and
> 10262.
> 
> Signed-off-by: Haihao Xiang <haihao.xiang@intel.com>
> ---
>  libavfilter/vf_vpp_qsv.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libavfilter/vf_vpp_qsv.c b/libavfilter/vf_vpp_qsv.c
> index ac77f7bb4b..b4dd7a4191 100644
> --- a/libavfilter/vf_vpp_qsv.c
> +++ b/libavfilter/vf_vpp_qsv.c
> @@ -602,6 +602,7 @@ not_ready:
>      return FFERROR_NOT_READY;
>  
>  eof:
> +    pts = av_rescale_q(pts, inlink->time_base, outlink->time_base);
>      ff_outlink_set_status(outlink, status, pts);
>      return 0;
>  }

This patch works for us and addresses issues reported in 
https://trac.ffmpeg.org/ticket/10261 and 
https://trac.ffmpeg.org/ticket/10262. Please, help to merge.
_______________________________________________
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".

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [FFmpeg-devel] [PATCH] lavfi/vf_vpp_qsv: set the right timestamp for AVERROR_EOF
  2023-03-18  0:37 ` Rogozhkin, Dmitry V
@ 2023-03-20  2:13   ` Xiang, Haihao
  0 siblings, 0 replies; 3+ messages in thread
From: Xiang, Haihao @ 2023-03-20  2:13 UTC (permalink / raw)
  To: ffmpeg-devel, Rogozhkin, Dmitry V

On Sa, 2023-03-18 at 00:37 +0000, Rogozhkin, Dmitry V wrote:
> On Fri, 2023-03-17 at 10:38 +0800, Xiang, Haihao wrote:
> > From: Haihao Xiang <haihao.xiang@intel.com>
> > 
> > Rescale the timestamp for AVERROR_EOF. This can fix tickets 10261 and
> > 10262.
> > 
> > Signed-off-by: Haihao Xiang <haihao.xiang@intel.com>
> > ---
> >  libavfilter/vf_vpp_qsv.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/libavfilter/vf_vpp_qsv.c b/libavfilter/vf_vpp_qsv.c
> > index ac77f7bb4b..b4dd7a4191 100644
> > --- a/libavfilter/vf_vpp_qsv.c
> > +++ b/libavfilter/vf_vpp_qsv.c
> > @@ -602,6 +602,7 @@ not_ready:
> >      return FFERROR_NOT_READY;
> >  
> >  eof:
> > +    pts = av_rescale_q(pts, inlink->time_base, outlink->time_base);
> >      ff_outlink_set_status(outlink, status, pts);
> >      return 0;
> >  }
> 
> This patch works for us and addresses issues reported in 
> https://trac.ffmpeg.org/ticket/10261 and 
> https://trac.ffmpeg.org/ticket/10262. Please, help to merge.

Pushed and cherry-picked to release/6.0 branch. 

Thanks
haihao


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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-03-20  2:13 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-17  2:38 [FFmpeg-devel] [PATCH] lavfi/vf_vpp_qsv: set the right timestamp for AVERROR_EOF Xiang, Haihao
2023-03-18  0:37 ` Rogozhkin, Dmitry V
2023-03-20  2:13   ` Xiang, Haihao

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