[FFmpeg-devel] [PATCH] avcodec/packet: initialize payload size to zero

2019-04-02 Thread Andriy Gelman
From: Andriy Gelman --- libavcodec/avpacket.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/avpacket.c b/libavcodec/avpacket.c index 8f0603df78..6ae092ed31 100644 --- a/libavcodec/avpacket.c +++ b/libavcodec/avpacket.c @@ -44,6 +44,7 @@ FF_ENABLE_DEPRECATION_WARNINGS pkt

[FFmpeg-devel] [PATCH] avformat/utils: Use av_packet_move_ref for packet ownership transfer

2019-04-04 Thread Andriy Gelman
From: Andriy Gelman This commit replaces packet assignment operator with av_packet_move_ref when there is a packet ownership transfer. --- Aims to address a TODO in libavformat/utils.c about using av_packet_mov_ref instead of assignment operator. libavformat/utils.c | 28

[FFmpeg-devel] [PATCH v2] avformat/utils: Use av_packet_move_ref for packet ownership transfer

2019-04-08 Thread Andriy Gelman
From: Andriy Gelman This commit replaces packet assignment operator with av_packet_move_ref when there is a packet ownership transfer. --- Michael, the update patch now has correct behavior for ticket 4221. libavformat/utils.c | 30 -- 1 file changed, 16 insertions

[FFmpeg-devel] Concatenate NAL units in av_write_frame call

2019-04-10 Thread Andriy Gelman
Hello, I've been working with HEVC + mpegts at low bitrates 10-20kbps. At these bitrates, mpegts can create a substantial overhead due to the 188 byte frame size (i.e. a NAL unit of 200 bytes will get rounded up to 376 bytes). I've tried to get around this by concatenating multiple NAL units

Re: [FFmpeg-devel] [PATCH v2] avformat/utils: Use av_packet_move_ref for packet ownership transfer

2019-04-19 Thread Andriy Gelman
On Mon, 08. Apr 14:03, Andriy Gelman wrote: > From: Andriy Gelman > > This commit replaces packet assignment operator with av_packet_move_ref > when there is a packet ownership transfer. > --- > Michael, the update patch now has correct behavior for ticket 4221. > >

[FFmpeg-devel] [PATCH v3] avformat/utils: Use av_packet_move_ref for packet ownership transfer

2019-04-21 Thread Andriy Gelman
From: Andriy Gelman During AVPacket assignment, it is currently not clear when the lhs takes ownership of the packet. This commit replaces assignment with an explicit av_packet_move_ref call when there is an ownership transfer to clear the distinction. --- libavformat/utils.c | 30

[FFmpeg-devel] [PATCH] libavformat/utils: Interpolate missing timestamps in H264 and HEVC when no b-frames observed.

2019-05-11 Thread Andriy Gelman
From: Andriy Gelman Fixes Ticket #7895. Currently, timestamp interpolation is disabled by default in H264 and HEVC. This creates playback issues when the demuxer does not output a valid timestamp. This patch allows interpolation when no b-frames have been observed during decoding, which fixes

Re: [FFmpeg-devel] [PATCH] libavformat/utils: Interpolate missing timestamps in H264 and HEVC when no b-frames observed.

2019-05-13 Thread Andriy Gelman
Hello, On Mon, 13. May 12:04, Michael Niedermayer wrote: > On Sat, May 11, 2019 at 11:20:51PM -0400, Andriy Gelman wrote: > > From: Andriy Gelman > > > > Fixes Ticket #7895. > > > > Currently, timestamp interpolation is disabled by default in H264 and > &

[FFmpeg-devel] [PATCH v2 2/2] tests/fate: Add fate test for timestamp interpolation

2019-05-14 Thread Andriy Gelman
From: Andriy Gelman This test verifies that missing timestamps from mpegts are interpolated for a HEVC stream. --- tests/fate/mpegts.mak | 7 ++ tests/ref/fate/mpegts-probe-interptime | 154 + 2 files changed, 161 insertions(+) create mode 100644

[FFmpeg-devel] [PATCH v2 1/2] libavformat/utils: Interpolate missing timestamps in H264 and HEVC when no b-frames observed

2019-05-14 Thread Andriy Gelman
From: Andriy Gelman Fixes Ticket #7895. Currently, timestamp interpolation is disabled by default in H264 and HEVC. This creates playback issues when the demuxer does not output a valid timestamp. This patch allows interpolation when no b-frames have been observed during decoding, which fixes

Re: [FFmpeg-devel] [PATCH v2 2/2] tests/fate: Add fate test for timestamp interpolation

2019-05-14 Thread Andriy Gelman
Hello, On Tue, 14. May 17:54, Andriy Gelman wrote: > From: Andriy Gelman > > This test verifies that missing timestamps from mpegts are interpolated > for a HEVC stream. > --- > tests/fate/mpegts.mak | 7 ++ > tests/ref/fate/mpegts-p

Re: [FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-15 Thread Andriy Gelman
On Tue, 14. May 22:36, Jun Li wrote: > Fix #6945 > Current implementaion for autorotate works fine for stream > level rotataion but no support for frame level operation > and frame flip. This patch is for adding flip support and > per frame operations. > --- > fftools/cmdutils.c | 9 ++--- >

Re: [FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-15 Thread Andriy Gelman
On Tue, 14. May 22:36, Jun Li wrote: > Fix #6945 > Current implementaion for autorotate works fine for stream > level rotataion but no support for frame level operation > and frame flip. This patch is for adding flip support and > per frame operations. > --- > fftools/cmdutils.c | 9 ++--- >

Re: [FFmpeg-devel] [PATCH v2 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-15 Thread Andriy Gelman
On Wed, 15. May 11:57, Jun Li wrote: > On Wed, May 15, 2019 at 11:45 AM Andriy Gelman > wrote: > > > On Tue, 14. May 22:36, Jun Li wrote: > > > Fix #6945 > > > Current implementaion for autorotate works fine for stream > > > level rotataion but no s

Re: [FFmpeg-devel] [PATCH v2 1/2] libavformat/utils: Interpolate missing timestamps in H264 and HEVC when no b-frames observed

2019-05-15 Thread Andriy Gelman
On Thu, 16. May 00:43, Michael Niedermayer wrote: > On Tue, May 14, 2019 at 05:54:21PM -0400, Andriy Gelman wrote: > > From: Andriy Gelman > > > > Fixes Ticket #7895. > > > > Currently, timestamp interpolation is disabled by default in H264 and > > HEVC.

Re: [FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-16 Thread Andriy Gelman
Hi Jun, On Thu, 16. May 00:12, Jun Li wrote: > Fix #6945 > Current implementaion for autorotate works fine for stream > level rotataion but no support for frame level operation > and frame flip. This patch is for adding flip support and > per frame operations. > --- > fftools/cmdutils.c |

Re: [FFmpeg-devel] [PATCH v3 2/2] fftools/ffmpeg: add support for per frame rotation and flip

2019-05-16 Thread Andriy Gelman
On Thu, 16. May 18:28, Jun Li wrote: > On Thu, May 16, 2019 at 12:54 PM Andriy Gelman > wrote: > > > Hi Jun, > > > > On Thu, 16. May 00:12, Jun Li wrote: > > > Fix #6945 > > > Current implementaion for autorotate works fine for stream > &g

Re: [FFmpeg-devel] [PATCH v3] avformat/utils: Use av_packet_move_ref for packet ownership transfer

2019-05-19 Thread Andriy Gelman
On Sun, 21. Apr 10:14, Andriy Gelman wrote: > From: Andriy Gelman > > During AVPacket assignment, it is currently not clear when the lhs takes > ownership of the packet. This commit replaces assignment with an > explicit av_packet_move_ref call when there is an ownership transfer

Re: [FFmpeg-devel] [PATCH] libavformat/mpegtsenc: new interlaced mux mode

2019-06-11 Thread Andriy Gelman
Hello, On Mon, 10. Jun 17:29, Andreas Håkon wrote: > Hi, > > Here is a list of comments on this patch: > (Note: I use for all the tests the file > https://samples.ffmpeg.org/HDTV/bshi01.tp) > > - By default the current behavior is selected. You can verify that this > patch doesn’t alter the ori

Re: [FFmpeg-devel] [PATCH v2 1/2] libavformat/utils: Interpolate missing timestamps in H264 and HEVC when no b-frames observed

2019-06-15 Thread Andriy Gelman
Michael, On Thu, 16. May 00:43, Michael Niedermayer wrote: > On Tue, May 14, 2019 at 05:54:21PM -0400, Andriy Gelman wrote: > > From: Andriy Gelman > > > > Fixes Ticket #7895. > > > > Currently, timestamp interpolation is disabled by default in H264 and > &

Re: [FFmpeg-devel] [PATCH 5/5] startcode: Filter out non-startcodes earlier

2019-06-30 Thread Andriy Gelman
Andreas, On Sun, 09. Jun 13:00, Andreas Rheinhardt wrote: > Up until now, the bitmasks used to initially find out when one needs > to take a closer look and search for startcodes were rather primitive: > If a block (of four or eight bytes, depending on the system) contained a > zero, it was treat

[FFmpeg-devel] [PATCH] avformat/rawenc: avoid divide by 0 when writing adx trailer

2019-07-13 Thread Andriy Gelman
From: Andriy Gelman Fixes #7985 This patch checks that the number of audio channels is not zero when writing the total sample count in adx header. --- libavformat/rawenc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/rawenc.c b/libavformat/rawenc.c index 993d232b70

Re: [FFmpeg-devel] [PATCH] avformat/rawenc: avoid divide by 0 when writing adx trailer

2019-07-13 Thread Andriy Gelman
On Sun, 14. Jul 00:40, Carl Eugen Hoyos wrote: > > > Am 13.07.2019 um 22:50 schrieb Andriy Gelman : > > > > From: Andriy Gelman > > > > Fixes #7985 > > Did you check the patch mentioned there? Sorry, I missed that you sent a patch. I applied

Re: [FFmpeg-devel] [PATCH] avformat/rawenc: avoid divide by 0 when writing adx trailer

2019-07-13 Thread Andriy Gelman
On Sat, 13. Jul 23:10, Andriy Gelman wrote: > On Sun, 14. Jul 00:40, Carl Eugen Hoyos wrote: > > > > > Am 13.07.2019 um 22:50 schrieb Andriy Gelman : > > > > > > From: Andriy Gelman > > > > > > Fixes #7985 > > > > Did you che

[FFmpeg-devel] [PATCH] hevc_mp4toannexb: Do not duplicate parameter sets

2019-07-20 Thread Andriy Gelman
From: Andriy Gelman Fixes #7799 Currently, the mp4toannexb filter always inserts extradata at the start of each IRAP unit. This can lead to duplication of parameter sets if the demuxed packet from mdat atom already contains a version of the parameters. As in ticket #7799 this can also lead to

Re: [FFmpeg-devel] [PATCH] libavformat/mpegtsenc: fix incorrect PCR with multiple programs

2019-07-27 Thread Andriy Gelman
Andreas, On Mon, 22. Jul 12:22, Andreas Håkon wrote: > Hi, > > Based on the discussion of my previous patch > https://patchwork.ffmpeg.org/patch/13487/ > Here I publish a first part of the patch that only addresses the PCR problem. Thanks for splitting the patch set. > > This supersedes too:

Re: [FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v3]

2019-07-28 Thread Andriy Gelman
Andreas, On Sun, 28. Jul 19:07, Andreas Håkon wrote: > Hi, > > This last version fixes the small bug discovered by Michael Niedermayer: > https://patchwork.ffmpeg.org/patch/14099/ > > This version is finally clean. > > Regards. > A.H. > > --- > From 08565b81aa2b6d75043e5e984da143480891c3b0 Mo

[FFmpeg-devel] [PATCH] libavformat: Add ZeroMQ as a protocol option

2019-07-30 Thread Andriy Gelman
-f mpegts zmq:tcp://127.0.0.1: Multiple clients can then connect with: $ ./ffplay zmq:tcp://127.0.0.1: I would be happy to maintain the code. Thanks, Andriy >From 366f705945f9b2c40158730ec18ac9259bca2695 Mon Sep 17 00:00:00 2001 From: Andriy Gelman Date: Tue, 30 Jul 2019 14:39:32

[FFmpeg-devel] [PATCH] lavfi/zmq: Avoid mem copy past the end of input buffer

2019-07-30 Thread Andriy Gelman
From: Andriy Gelman --- libavfilter/f_zmq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/f_zmq.c b/libavfilter/f_zmq.c index 89da5bef06..744c721305 100644 --- a/libavfilter/f_zmq.c +++ b/libavfilter/f_zmq.c @@ -139,7 +139,7 @@ static int recv_msg

Re: [FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v4]

2019-07-31 Thread Andriy Gelman
tests pointed by Michael Niedermayer (Sample_cut.ts) and > Andriy Gelman (day_flight.mpg). > > I hope this time the patch will be accepted. > Regards. > A.H. > > --- > From 8381febd0e881cfcd53583b0ccdd7eb2c580e422 Mon Sep 17 00:00:00 2001 > From: Andreas Hakon > D

[FFmpeg-devel] [PATCH 2/2] avformat/mpegtsenc: Remove redundant goto after malloc fail

2019-07-31 Thread Andriy Gelman
From: Andriy Gelman --- libavformat/mpegtsenc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index fc0ea225c6..1541a7a073 100644 --- a/libavformat/mpegtsenc.c +++ b/libavformat/mpegtsenc.c @@ -846,10 +846,8 @@ static

[FFmpeg-devel] [PATCH 1/2] lavfi/zmq: Avoid mem copy past the end of input buffer

2019-07-31 Thread Andriy Gelman
From: Andriy Gelman --- libavfilter/f_zmq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/f_zmq.c b/libavfilter/f_zmq.c index 89da5bef06..744c721305 100644 --- a/libavfilter/f_zmq.c +++ b/libavfilter/f_zmq.c @@ -139,7 +139,7 @@ static int recv_msg

Re: [FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v4]

2019-08-01 Thread Andriy Gelman
Andreas, On Thu, 01. Aug 08:24, Andreas Håkon wrote: > Hi Andriy, > > > ‐‐‐ Original Message ‐‐‐ > On Thursday, 1 de August de 2019 0:23, Andriy Gelman > wrote: > > > > +for (i = 0; i < ts->nb_services; i++) { > > > +serv

Re: [FFmpeg-devel] [PATCH 1/4] avformat/mpegtsenc: fix incorrect PCR selection with multiple programs

2019-08-02 Thread Andriy Gelman
On Sat, 03. Aug 00:37, Marton Balint wrote: > The MPEG-TS muxer had a serious bug related to the use of multiple programs: > in that case, the PCR pid selection was incomplete for all services except > one. > This patch solves this problem and selects a stream to become PCR for each > service, pre

Re: [FFmpeg-devel] [PATCHv2 1/4] avformat/mpegtsenc: fix incorrect PCR selection with multiple programs

2019-08-03 Thread Andriy Gelman
On Sat, 03. Aug 10:19, Marton Balint wrote: > The MPEG-TS muxer had a serious bug related to the use of multiple programs: > in that case, the PCR pid selection was incomplete for all services except > one. > This patch solves this problem and selects a stream to become PCR for each > service, pre

[FFmpeg-devel] [PATCH v2] libavformat: Add ZeroMQ as a protocol option

2019-08-04 Thread Andriy Gelman
Changes in v2: 1. Replaced zmq_poll with zmq_msg_recv. 2. Remove user timeout option as zmq_msg_recv(.., .., ZMQ_DONTWAIT) is a non-blocking call. 3. Updated docs. Andriy >From 53e6e00d30c9fbf5127eea9d377686d37e981c0c Mon Sep 17 00:00:00 2001 From: Andriy Gelman Date: Tue, 30

Re: [FFmpeg-devel] [PATCH v2] libavformat: Add ZeroMQ as a protocol option

2019-08-06 Thread Andriy Gelman
On Sun, 04. Aug 14:36, Andriy Gelman wrote: > Changes in v2: > 1. Replaced zmq_poll with zmq_msg_recv. > 2. Remove user timeout option as zmq_msg_recv(.., .., ZMQ_DONTWAIT) is a > non-blocking call. > 3. Updated docs. > > Andriy > > From 53e6e00d30c9fbf51

[FFmpeg-devel] [PATCH] tools/zmqsend: Avoid mem copy past the end of input buffer

2019-08-08 Thread Andriy Gelman
From: Andriy Gelman This patch avoids a read past the end of the input buffer in memcpy since the size of the received zmq message is recv_buf_size - 1. --- tools/zmqsend.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/zmqsend.c b/tools/zmqsend.c index 7bd7fe4199

Re: [FFmpeg-devel] [PATCH] hevc_mp4toannexb: Do not duplicate parameter sets

2019-08-12 Thread Andriy Gelman
Andreas, On Sun, 21. Jul 10:47, Andreas Rheinhardt wrote: > Andriy Gelman: > > From: Andriy Gelman > > > > Fixes #7799 > > > > Currently, the mp4toannexb filter always inserts extradata at the start > > of each IRAP unit. This can lead to duplication of p

[FFmpeg-devel] [PATCH v2] hevc_mp4toannexb: Insert correct parameter sets before IRAP

2019-08-18 Thread Andriy Gelman
From: Andriy Gelman Fixes #7799 Currently, the mp4toannexb filter always inserts the same extradata at the start of the first IRAP unit. As in ticket #7799, this can lead to decoding errors if modified parameter sets are signalled in-band. This commit keeps track of the vps/sps/pps parameter

Re: [FFmpeg-devel] [PATCH] hevc_mp4toannexb: Do not duplicate parameter sets

2019-08-18 Thread Andriy Gelman
Andreas, On Tue, 13. Aug 06:24, Andreas Rheinhardt wrote: > Andriy Gelman: > > Andreas, > > > > On Sun, 21. Jul 10:47, Andreas Rheinhardt wrote: > >> Andriy Gelman: > >>> From: Andriy Gelman > >>> > >>> Fixes #7799 > >&g

[FFmpeg-devel] [PATCH v3] libavformat: Add ZeroMQ as a protocol option

2019-08-19 Thread Andriy Gelman
Minor changes in v3: 1. Removed tab character from as per feedback 2. Removed unused timeout variable from ZMQContext Andriy >From 66c11c12fcfa8a7fbb3c8c09d23c017992229a99 Mon Sep 17 00:00:00 2001 From: Andriy Gelman Date: Tue, 30 Jul 2019 14:39:32 -0400 Subject: [PATCH] libavformat:

Re: [FFmpeg-devel] [PATCH v2] hevc_mp4toannexb: Insert correct parameter sets before IRAP

2019-08-19 Thread Andriy Gelman
On Tue, 20. Aug 01:01, Michael Niedermayer wrote: > On Sun, Aug 18, 2019 at 10:37:57PM -0400, Andriy Gelman wrote: > [...] > > diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak > > index 559c3898bc..46f130f4d1 100644 > > --- a/tests/fate/hevc.mak > > +++ b/tes

Re: [FFmpeg-devel] [PATCH v2] hevc_mp4toannexb: Insert correct parameter sets before IRAP

2019-08-20 Thread Andriy Gelman
Andreas, On Tue, 20. Aug 07:54, Andreas Rheinhardt wrote: > Hello, > > I have not looked at the *PS and the SEI stuff yet, but here is > already my review of the general code. > > Andriy Gelman: > > From: Andriy Gelman > > > > Fixes #7799 > > &g

Re: [FFmpeg-devel] [PATCH v3] libavformat: Add ZeroMQ as a protocol option

2019-08-23 Thread Andriy Gelman
On Mon, 19. Aug 17:28, Andriy Gelman wrote: > Minor changes in v3: > 1. Removed tab character from as per feedback > 2. Removed unused timeout variable from ZMQContext > > Andriy > From 66c11c12fcfa8a7fbb3c8c09d23c017992229a99 Mon Sep 17 00:00:00 2001 > From: Andriy Ge

Re: [FFmpeg-devel] [PATCH v2 5/8] avformat/utils: Fix memleaks II

2019-08-24 Thread Andriy Gelman
Andreas, On Mon, 19. Aug 23:56, Andreas Rheinhardt wrote: > Up until now, avformat_find_stream_info had a potential for memleaks: > When everything is fine, it read packets and (depending upon whether > AVFMT_FLAG_NOBUFFER was set) put them in a packet list or unreferenced > them when they were no

Re: [FFmpeg-devel] [PATCH v2 7/8] avformat/utils: Remove unnecessary packet copies

2019-08-28 Thread Andriy Gelman
On Mon, 19. Aug 23:56, Andreas Rheinhardt wrote: > Up until now, read_frame_internal in avformat/utils.c uses a spare > packet on the stack that serves no real purpose: At no point in this > function is there a need for another packet besides the packet destined > for output: > 1. If the packet doe

Re: [FFmpeg-devel] [PATCH v3] libavformat: Add ZeroMQ as a protocol option

2019-08-28 Thread Andriy Gelman
Marton, Thanks for reviewing this patch. On Sat, 24. Aug 19:33, Marton Balint wrote: > > > On Fri, 23 Aug 2019, Andriy Gelman wrote: > > > On Mon, 19. Aug 17:28, Andriy Gelman wrote: > > > Minor changes in v3: 1. Removed tab character from as per feedback >

Re: [FFmpeg-devel] [PATCH v3] libavformat: Add ZeroMQ as a protocol option

2019-08-28 Thread Andriy Gelman
On Thu, 29. Aug 02:28, Marton Balint wrote: > > > On Wed, 28 Aug 2019, Andriy Gelman wrote: > > > > > > +h->is_streamed = 1; > > > > > + > > > > > +av_strstart(uri, "zmq:", &uri); > > > > >

[FFmpeg-devel] [PATCH v4] Add ZeroMQ as protocol option

2019-08-29 Thread Andriy Gelman
rom ca5aa5149b97152b15457b2f40c753c80e68e3de Mon Sep 17 00:00:00 2001 From: Andriy Gelman Date: Tue, 30 Jul 2019 14:39:32 -0400 Subject: [PATCH] libavformat: Add ZeroMQ as a protocol option Currently multiple clients are only supported by using a multicast destination address. An alternative is to stream t

[FFmpeg-devel] [PATCH] avcodec/mpeg: Initialize quarter_sample parameter from previous thread.

2018-12-12 Thread Andriy Gelman
Fixes #7410. The value of sub-pixel precision for me/mc can change during an Intra frame. In multi-threaded decoding this change is not propagated to other frame threads causing decoding artifacts. This patch initializes the sub-pixel precision parameter from previous thread, which fixes the iss

Re: [FFmpeg-devel] [PATCH] lavc/v4l2_context: fix compile warning for incompatible pointer type

2020-02-27 Thread Andriy Gelman
On Fri, 28. Feb 00:14, Linjie Fu wrote: > Signed-off-by: Linjie Fu > --- > libavcodec/v4l2_context.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/v4l2_context.c b/libavcodec/v4l2_context.c > index 95a2bfa..8110bbb 100644 > --- a/libavcodec/v4l2_context.c >

[FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-02-28 Thread Andriy Gelman
From: Andriy Gelman Supports connecting to a RabbitMQ broker via AMQP version 0-9-1. Signed-off-by: Andriy Gelman --- Changes in v2: - Addressed comments from Marton - Updated documentation Compilation notes: - Requires librabbitmq-dev package (on ubuntu). - The pkg-config

[FFmpeg-devel] [PATCH v2 2/2] avformat/libzmq: Make default pkt_size value consistent with amqp

2020-02-28 Thread Andriy Gelman
From: Andriy Gelman Signed-off-by: Andriy Gelman --- libavformat/libzmq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/libzmq.c b/libavformat/libzmq.c index 8c8b294c921..1b0d8638dbf 100644 --- a/libavformat/libzmq.c +++ b/libavformat/libzmq.c @@ -40,7 +40,7

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-02-29 Thread Andriy Gelman
On Sat, 29. Feb 11:21, Paul B Mahol wrote: > I think this was already rejected? Thilo had a question on broker performance, which I believe addressed: http://ffmpeg.org/pipermail/ffmpeg-devel/2020-February/257483.html j-b had concerns about how it fits into libavformat: http://ffmpeg.org/pipermai

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/encode: restructure the core encoding code

2020-02-29 Thread Andriy Gelman
On Thu, 27. Feb 15:02, James Almer wrote: > This commit follows the same logic as 061a0c14bb, but for the encode API: The > new public encoding API will no longer be a wrapper around the old deprecated > one, and the internal API used by the encoders now consists of a single > receive_packet() call

Re: [FFmpeg-devel] [PATCH] avformat/udp: support w32pthreads compat

2020-03-02 Thread Andriy Gelman
On Mon, 02. Mar 13:38, phunkyfish wrote: > --- > compat/w32pthreads.h | 10 ++ > libavformat/udp.c| 8 +++- > 2 files changed, 17 insertions(+), 1 deletion(-) > > diff --git a/compat/w32pthreads.h b/compat/w32pthreads.h > index 7df33b7da4..64cd40cda4 100644 > --- a/compat/w32pthr

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-03-02 Thread Andriy Gelman
On Mon, 02. Mar 11:41, Anton Khirnov wrote: > Quoting Marton Balint (2020-02-29 17:09:58) > > > > > > On Sat, 29 Feb 2020, Paul B Mahol wrote: > > > > > I think this was already rejected? > > > > jb questioned if this belongs to libavformat, and timo asked how well the > > message brokers hand

[FFmpeg-devel] [PATCH] avcodec/avcodec: Fix typos

2020-03-04 Thread Andriy Gelman
From: Andriy Gelman Signed-off-by: Andriy Gelman --- libavcodec/avcodec.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 5a0fc3405c5..8cda2422efa 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -5037,7

[FFmpeg-devel] [PATCH] avcodec/v4l2_m2m_enc: Adapt to the new internal encode API

2020-03-07 Thread Andriy Gelman
From: Andriy Gelman Should be squashed with: http://ffmpeg.org/pipermail/ffmpeg-devel/2020-February/257735.html Signed-off-by: Andriy Gelman --- libavcodec/v4l2_m2m.c | 9 - libavcodec/v4l2_m2m.h | 6 +- libavcodec/v4l2_m2m_dec.c | 2 +- libavcodec/v4l2_m2m_enc.c | 17

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2_m2m_enc: Adapt to the new internal encode API

2020-03-07 Thread Andriy Gelman
On Sat, 07. Mar 23:23, James Almer wrote: > On 3/7/2020 9:23 PM, Andriy Gelman wrote: > > From: Andriy Gelman > > > > Should be squashed with: > > http://ffmpeg.org/pipermail/ffmpeg-devel/2020-February/257735.html > > > > Signed-off-by: Andriy Gelman >

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2_m2m_enc: Adapt to the new internal encode API

2020-03-07 Thread Andriy Gelman
On Sat, 07. Mar 23:13, James Almer wrote: > On 3/7/2020 9:23 PM, Andriy Gelman wrote: > > From: Andriy Gelman > > > > Should be squashed with: > > http://ffmpeg.org/pipermail/ffmpeg-devel/2020-February/257735.html > > Thank you! Only three remain now :) > &

[FFmpeg-devel] [PATCH] lavc/v4l2_m2m_dec: Init reserved bytes to zero before ioctl call

2020-03-08 Thread Andriy Gelman
From: Andriy Gelman struct v4l2_selection contains reserved bytes which should be set to zero before the ioctl call. Fixes valgrind error: Syscall param ioctl(VKI_V4L2_S_SELECTION) points to uninitialised byte(s) Signed-off-by: Andriy Gelman --- libavcodec/v4l2_m2m_dec.c | 2 +- 1 file

[FFmpeg-devel] [PATCH v3 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-03-08 Thread Andriy Gelman
From: Andriy Gelman Supports connecting to a RabbitMQ broker via AMQP version 0-9-1. Signed-off-by: Andriy Gelman --- Changes in v3: - Fixed a bug when parsing username that contained a url encoded ":" - Fixed documentation error and improved logging Changes in v2: -

[FFmpeg-devel] [PATCH v3 2/2] avformat/libzmq: Make default pkt_size value consistent with amqp

2020-03-08 Thread Andriy Gelman
From: Andriy Gelman Signed-off-by: Andriy Gelman --- doc/protocols.texi | 2 +- libavformat/libzmq.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/protocols.texi b/doc/protocols.texi index 27df4759b63..e510019f2d8 100644 --- a/doc/protocols.texi +++ b/doc

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-03-08 Thread Andriy Gelman
On Sun, 08. Mar 12:25, Marton Balint wrote: > > > Subject: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 > > protocol support > > > > From: Andriy Gelman > > > > Supports connecting to a RabbitMQ broker via AMQP version 0-9-1

Re: [FFmpeg-devel] [PATCH v3 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-03-08 Thread Andriy Gelman
On Sun, 08. Mar 16:33, Andriy Gelman wrote: > From: Andriy Gelman > > Supports connecting to a RabbitMQ broker via AMQP version 0-9-1. > > Signed-off-by: Andriy Gelman > --- > > Changes in v3: > - Fixed a bug when parsing username that contained a ur

[FFmpeg-devel] [PATCH v4 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-03-08 Thread Andriy Gelman
From: Andriy Gelman Supports connecting to a RabbitMQ broker via AMQP version 0-9-1. Signed-off-by: Andriy Gelman --- Changes in v4: - Fixed leak from user_decoded/password_decoded when amqp_new_connection() fails Changes in v3: - Fixed a bug when parsing username that

[FFmpeg-devel] [PATCH 2/2] avcodec/v4l2_m2m{enc, dec}: Fix leaks when init fails

2020-03-09 Thread Andriy Gelman
From: Andriy Gelman v4l2_m2m_{enc,dec} doesn't call its close function when initialization fails because FF_CODEC_CAP_INIT_CLEANUP is not set. This causes a couple possible leaks, which are fixed in the commit. Signed-off-by: Andriy Gelman --- libavcodec/v4l2_m2m_dec.

[FFmpeg-devel] [PATCH 1/2] avcodec/v4l2_m2m_enc: Fix leak when device init fails

2020-03-09 Thread Andriy Gelman
From: Colin NG Fixes ticket #8285 Signed-off-by: Andriy Gelman --- This was sent to the mailing list by Colin NG some time ago, but was never applied. libavcodec/v4l2_m2m_enc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/v4l2_m2m_enc.c b/libavcodec/v4l2_m2m_enc.c index

[FFmpeg-devel] [PATCH v2] avcodec/v4l2_m2m_enc: Adapt to the new internal encode API

2020-03-09 Thread Andriy Gelman
From: Andriy Gelman Should be squashed with: http://ffmpeg.org/pipermail/ffmpeg-devel/2020-February/257735.html Signed-off-by: Andriy Gelman --- libavcodec/v4l2_m2m.c | 8 libavcodec/v4l2_m2m.h | 3 +++ libavcodec/v4l2_m2m_enc.c | 15 ++- 3 files changed, 25

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/v4l2_m2m: Avoid using intermediate buffer

2020-03-10 Thread Andriy Gelman
evertheless led to a compiler warning > > "‘strncpy’ specified bound depends on the length of the source argument > > [-Wstringop-overflow=]" in GCC 9.2. strlen() was unnecessary anyway. > > > > Reviewed-by: Andriy Gelman > > Signed-off-by: Andreas Rheinhardt

Re: [FFmpeg-devel] [PATCH] avcodec/avcodec: Fix typos

2020-03-11 Thread Andriy Gelman
On Thu, 05. Mar 00:02, Andriy Gelman wrote: > From: Andriy Gelman > > Signed-off-by: Andriy Gelman > --- > libavcodec/avcodec.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index 5a0f

Re: [FFmpeg-devel] [PATCH] lavc/v4l2_m2m_dec: Init reserved bytes to zero before ioctl call

2020-03-12 Thread Andriy Gelman
On Sun, 08. Mar 11:49, Andriy Gelman wrote: > From: Andriy Gelman > > struct v4l2_selection contains reserved bytes which should be set to > zero before the ioctl call. > > Fixes valgrind error: > Syscall param ioctl(VKI_V4L2_S_SELECTION) points to uninitialised byte(

Re: [FFmpeg-devel] [PATCH v4 1/3] avcodec/v4l2_m2m_enc: Reduce log verbosity for some params

2020-03-14 Thread Andriy Gelman
On Sun, 23. Feb 13:47, Andriy Gelman wrote: > From: Andriy Gelman > > Currently the user gets unhelpful warnings when some default parameters > are not supported by the device. The verbosity of these log messages has > been changed to AV_LOG_DEBUG. > > Signed-

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2_m2m: fix setting the frame rate

2020-03-16 Thread Andriy Gelman
On Mon, 16. Mar 10:00, Ming Qian wrote: > v4l2 set the frame rate through frame intervals, > not set frame rate directly. > the frame rate and frame intervals are reciprocal. > so in libavdevice/v4l2.c we can see the following code: > tpf->numerator = framerate_q.den; > tpf->denominat

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2_buffers: don't prevent enqueue capture buffer to driver

2020-03-16 Thread Andriy Gelman
On Mon, 16. Mar 10:02, Ming Qian wrote: > the draining is set when the output port is finished, > but it doesn't mean the capture port is finished. > especially for decoder, there may be a stream buffer to store several > frames. > so the decoder still need capture buffer even if the draining is se

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2_m2m: handle the v4l2 eos event

2020-03-16 Thread Andriy Gelman
On Mon, 16. Mar 10:03, Ming Qian wrote: > when the last frame of capture is dequeueed, > driver may send this V4L2_EVENT_EOS event, > if this event is received, then we can set the capture port done Please add to your commit message (or something similar depending on what you tested): "Without th

[FFmpeg-devel] [PATCH] doc/general: Fix entry for AMQP

2020-03-19 Thread Andriy Gelman
From: Andriy Gelman Signed-off-by: Andriy Gelman --- doc/general.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/general.texi b/doc/general.texi index 14878f4dcf3..752618a00b4 100644 --- a/doc/general.texi +++ b/doc/general.texi @@ -1336,7 +1336,7 @@ performance on

Re: [FFmpeg-devel] [PATCH] lavc/v4l2_m2m_dec: Init reserved bytes to zero before ioctl call

2020-03-26 Thread Andriy Gelman
On Thu, 12. Mar 22:40, Andriy Gelman wrote: > On Sun, 08. Mar 11:49, Andriy Gelman wrote: > > From: Andriy Gelman > > > > struct v4l2_selection contains reserved bytes which should be set to > > zero before the ioctl call. > > > > Fixes

Re: [FFmpeg-devel] [PATCH v2] avcodec/v4l2_m2m: fix setting the frame rate

2020-03-29 Thread Andriy Gelman
On Tue, 17. Mar 19:29, Ming Qian wrote: > v4l2 set the frame rate through frame intervals, > not set frame rate directly. > the frame rate and frame intervals are reciprocal. > so in libavdevice/v4l2.c we can see the following code: > tpf->numerator = framerate_q.den; > tpf->denominat

Re: [FFmpeg-devel] [PATCH v3] avcodec/v4l2_m2m: handle the v4l2 eos event

2020-03-29 Thread Andriy Gelman
On Tue, 17. Mar 19:39, Ming Qian wrote: > when the last frame of capture is dequeueed, > driver may send this V4L2_EVENT_EOS event, > if this event is received, then we can set the capture port done s/dequeueed/dequeued Also I'd change the last line to: "If this event is received, then the captur

Re: [FFmpeg-devel] [PATCH v2] avcodec/v4l2_m2m: fix setting the frame rate

2020-03-31 Thread Andriy Gelman
On Sun, 29. Mar 10:53, Andriy Gelman wrote: > On Tue, 17. Mar 19:29, Ming Qian wrote: > > v4l2 set the frame rate through frame intervals, > > not set frame rate directly. > > the frame rate and frame intervals are reciprocal. > > so in libavdevice/v4l2.c we can see the

Re: [FFmpeg-devel] [PATCH] lavc/v4l2_m2m_dec: Init reserved bytes to zero before ioctl call

2020-03-31 Thread Andriy Gelman
On Wed, 01. Apr 00:38, Michael Niedermayer wrote: > On Sun, Mar 08, 2020 at 11:49:47AM -0400, Andriy Gelman wrote: > > From: Andriy Gelman > > > > struct v4l2_selection contains reserved bytes which should be set to > > zero before the ioctl call. > > > >

[FFmpeg-devel] [PATCH] avcodec/v4l2_m2m_enc: Add option to remove ivf container

2020-04-04 Thread Andriy Gelman
From: Andriy Gelman The dequeued packets from vp8 (s5p-mfc) encoder are output in ivf format which breaks the stream when the packets are muxed in avformat. This commit adds an option to remove the container and thus support the encoder. Signed-off-by: Andriy Gelman --- libavcodec/v4l2_m2m.h

Re: [FFmpeg-devel] [PATCH]lavfi/telecine: Mark telecined frames as interlaced

2020-04-04 Thread Andriy Gelman
On Sun, 05. Apr 02:05, Carl Eugen Hoyos wrote: > Am So., 5. Apr. 2020 um 01:02 Uhr schrieb Carl Eugen Hoyos > : > > > > Am Sa., 4. Apr. 2020 um 00:44 Uhr schrieb Carl Eugen Hoyos > > : > > > > > > Am Sa., 4. Apr. 2020 um 00:40 Uhr schrieb James Almer : > > > > > > > > On 4/3/2020 6:37 PM, Carl Eu

Re: [FFmpeg-devel] [PATCH]lavfi/telecine: Mark telecined frames as interlaced

2020-04-05 Thread Andriy Gelman
On Sun, 05. Apr 10:34, Carl Eugen Hoyos wrote: > Am So., 5. Apr. 2020 um 03:30 Uhr schrieb Andriy Gelman > : > > > > On Sun, 05. Apr 02:05, Carl Eugen Hoyos wrote: > > > Am So., 5. Apr. 2020 um 01:02 Uhr schrieb Carl Eugen Hoyos > > > : > > > > &

Re: [FFmpeg-devel] [PATCH]lavf/chromaprint: Silence compilation warnings

2020-04-05 Thread Andriy Gelman
lgtm with a couple of minor comments On Sun, 05. Apr 10:49, Carl Eugen Hoyos wrote: > Am Di., 13. Aug. 2019 um 12:45 Uhr schrieb Carl Eugen Hoyos > : > > > Attached patch fixes several compilation warnings when building with > > chromapring. > > I will push this patch if there are no objections.

[FFmpeg-devel] [PATCH 2/2] avcodec/v4l2_context: Log warning when all capture buffers are in userspace

2020-04-05 Thread Andriy Gelman
From: Andriy Gelman v4l2_m2m uses device memory mapped buffers to store dequeued frames/packets (reference counted by AVBufferRef). When the reference count drops to zero, the buffer ownership is returned back to the device, so that they can re-filled with frames/packets. There are some cases

[FFmpeg-devel] [PATCH 1/2] avcodec/v4l2_context: Use av_freep

2020-04-05 Thread Andriy Gelman
From: Andriy Gelman Signed-off-by: Andriy Gelman --- libavcodec/v4l2_context.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavcodec/v4l2_context.c b/libavcodec/v4l2_context.c index 8110bbb555..31af10d28e 100644 --- a/libavcodec/v4l2_context.c +++ b/libavcodec

Re: [FFmpeg-devel] [PATCH v4] avcodec/v4l2_m2m: handle the v4l2 eos event

2020-04-08 Thread Andriy Gelman
On Wed, 01. Apr 10:38, Ming Qian wrote: > when the last frame of capture is dequeueed, > driver may send this V4L2_EVENT_EOS event, > If this event is received, then the capture buffers have been flushed and > avcodec_receive_packet()/avcodec_receive_frame() can return AVERROR_EOF. > Otherwise, the

Re: [FFmpeg-devel] 回复: [EXT] Re: [PATCH] avcodec/v4l2_buffers: don't prevent enqueue capture buffer to driver

2020-04-08 Thread Andriy Gelman
, and this patch would prevent it (although it's not needed if an eos event is received). Perhaps Mark/Aman could also comment? Thanks, Andriy > > ________ > 发件人: ffmpeg-devel 代表 Andriy Gelman > > 发送时间: 2020年3月16日 20:07 > 收件人: FFmpeg development discus

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/v4l2_context: Use av_freep

2020-04-08 Thread Andriy Gelman
On Mon, 06. Apr 09:37, Andreas Rheinhardt wrote: > Andriy Gelman: > > From: Andriy Gelman > > > > Signed-off-by: Andriy Gelman > > --- > > libavcodec/v4l2_context.c | 6 ++ > > 1 file changed, 2 insertions(+), 4 deletions(-) > > > >

Re: [FFmpeg-devel] 回复: [EXT] Re: [PATCH] avcodec/v4l2_buffers: don't prevent enqueue capture buffer to driver

2020-04-08 Thread Andriy Gelman
On Thu, 09. Apr 02:14, Ming Qian wrote: > Did you try increasing the -num_capture_buffers option? It may solve your > problem. > 1. We can't increase the num_capture_buffers indefinitely. > 2. There is a ring buffer in driver, so the number of frames who is stored in > the ring buffer may be a la

Re: [FFmpeg-devel] [PATCH v4] avcodec/v4l2_m2m: handle the v4l2 eos event

2020-04-11 Thread Andriy Gelman
On Wed, 08. Apr 20:51, Andriy Gelman wrote: > On Wed, 01. Apr 10:38, Ming Qian wrote: > > when the last frame of capture is dequeueed, > > driver may send this V4L2_EVENT_EOS event, > > If this event is received, then the capture buffers have been flushed and >

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2_m2m_enc: Add option to remove ivf container

2020-04-11 Thread Andriy Gelman
On Sat, 11. Apr 15:56, Mark Thompson wrote: > On 04/04/2020 21:26, Andriy Gelman wrote: > > From: Andriy Gelman > > > > The dequeued packets from vp8 (s5p-mfc) encoder are output in ivf format > > which breaks the stream when the packets are muxed in avformat. This c

Re: [FFmpeg-devel] [PATCH] avcodec/v4l2_m2m_enc: Add option to remove ivf container

2020-04-11 Thread Andriy Gelman
On Sat, 11. Apr 18:56, Jan Ekström wrote: > On Sat, Apr 11, 2020 at 6:43 PM Andriy Gelman wrote: > > > > On Sat, 11. Apr 15:56, Mark Thompson wrote: > > > On 04/04/2020 21:26, Andriy Gelman wrote: > > > > From: Andriy Gelman > > > > > > &

Re: [FFmpeg-devel] [PATCH v1] fftools: fix hwaccles option dump redundancy

2020-04-13 Thread Andriy Gelman
On Mon, 13. Apr 20:02, Jun Zhao wrote: > From: Jun Zhao > > when enable the QSV in FFmpeg, used the cmd "ffmpeg -hwaccels" always > dump redundancy acceleration methods for QSV like: > > Hardware acceleration methods: > vaapi > qsv > drm > opencl > qsv > > Signed-off-by: Jun Zhao > --- > ffto

Re: [FFmpeg-devel] [PATCH] avformat/movenc: stop guessing colr atom values

2020-04-13 Thread Andriy Gelman
On Mon, 13. Apr 10:29, Michael Bradshaw wrote: > On Tue, Mar 31, 2020 at 6:23 AM Derek Buitenhuis > wrote: > > > I agree strongly with Michael's points. > > > I've pushed the patch to master. seems to break fate make fate-vsynth1-dnxhd-1080i-colr TESTvsynth1-dnxhd-1080i-colr --- ./tests/

Re: [FFmpeg-devel] [PATCH v4] avcodec/v4l2_m2m: handle the v4l2 eos event

2020-04-14 Thread Andriy Gelman
On Wed, 08. Apr 20:51, Andriy Gelman wrote: > On Wed, 01. Apr 10:38, Ming Qian wrote: > > when the last frame of capture is dequeueed, > > driver may send this V4L2_EVENT_EOS event, > > If this event is received, then the capture buffers have been flushed and >

  1   2   3   4   5   6   >