[FFmpeg-devel] [PATCH 7/7] avfilter/formats: Cosmetics

2020-08-14 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavfilter/formats.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/libavfilter/formats.c b/libavfilter/formats.c index 4ac690ea8a..71060f5f50 100644 --- a/libavfilter/formats.c +++ b/libavfilter/formats.c @@ -74,10 +7

[FFmpeg-devel] [PATCH 5/7] avfilter/formats: Factor checking for mergeability out of ff_merge_*

2020-08-14 Thread Andreas Rheinhardt
The callers of the ff_merge_*() functions fall into two categories with quite different needs: One caller is can_merge_formats() which only wants to test for mergeability without it merging anything. In order to do so, it duplicates the lists it intends to test and resets their owners so that they

[FFmpeg-devel] [PATCH 6/7] avfilter/formats: Always keep longer references list when merging lists

2020-08-14 Thread Andreas Rheinhardt
This means that one only needs to update the shorter list of references. Signed-off-by: Andreas Rheinhardt --- I doubt that this optimizations is worth the additional complexity. I have just added it for you to decide. libavfilter/formats.c | 24 +--- 1 file changed, 17 inse

[FFmpeg-devel] [PATCH 1/7] avfilter/formats: Make check for buffer overflow redundant

2020-08-14 Thread Andreas Rheinhardt
and remove the redundant check. This check for whether the allocated buffer is sufficient has been added in commit 1cbf7fb4345a3e5b7791d483241bf4759bde4ece (merging commit 5775a1832c4165e6acc1d307004b38701bb463f4). It is not sufficient to detect invalid input lists (namely lists with duplicates);

[FFmpeg-devel] [PATCH 2/7] avfilter/formats: Avoid allocations when merging formats and samplerates

2020-08-14 Thread Andreas Rheinhardt
This is the analogue of cfc65520324ae640299bd321ef88ae76dcee6f78 for formats and samplerates; in contrast to said commit, one can avoid allocating a new array for formats as well (the complications of the generic channel layouts made this impossible for channel layouts). This commit also starts to

[FFmpeg-devel] [PATCH 3/7] avfilter/formats: Resize potentially oversized arrays

2020-08-14 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavfilter/formats.c | 4 1 file changed, 4 insertions(+) diff --git a/libavfilter/formats.c b/libavfilter/formats.c index c370f7f91c..4efbcbebfe 100644 --- a/libavfilter/formats.c +++ b/libavfilter/formats.c @@ -60,6 +60,7 @@ do {

[FFmpeg-devel] [PATCH 4/7] avfilter/formats: Avoid code duplication when merging samplerates

2020-08-14 Thread Andreas Rheinhardt
by adapting the MERGE_FORMATS() so that only one instance of the MERGE_REF() macro needs to exist in ff_merge_samplerates(). Signed-off-by: Andreas Rheinhardt --- libavfilter/formats.c | 26 ++ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/libavfilter/fo

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: parse for AC-3 descriptor 6A

2020-08-14 Thread lance . lmwang
On Fri, Aug 14, 2020 at 10:08:06PM +0200, Marton Balint wrote: > > > On Fri, 14 Aug 2020, lance.lmw...@gmail.com wrote: > > > From: Limin Wang > > I don't think this patch fits into libavformat. I am not a fan of dumping > the descriptor data with AV_LOG_DEBUG, because libavformat is not a str

Re: [FFmpeg-devel] [PATCH] avformat/mpeg: Check avio_read() return value in get_pts()

2020-08-14 Thread Thierry Foucu
On Fri, Aug 14, 2020, 4:08 PM Michael Niedermayer wrote: > Found-by: Thierry Foucu > Fixes: Use-of-uninitialized-value > Signed-off-by: Michael Niedermayer > --- > libavformat/mpeg.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/libavformat/mpeg.c b/libavformat/m

[FFmpeg-devel] [PATCH] avformat/mpeg: Check avio_read() return value in get_pts()

2020-08-14 Thread Michael Niedermayer
Found-by: Thierry Foucu Fixes: Use-of-uninitialized-value Signed-off-by: Michael Niedermayer --- libavformat/mpeg.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c index 265b2bd1ad..a5e17925ce 100644 --- a/libavformat/mpeg.c +++ b/l

Re: [FFmpeg-devel] [PATCH] libavformat/mpeg.c: Initialize the buffer uses to read the PTS.

2020-08-14 Thread Michael Niedermayer
On Thu, Aug 13, 2020 at 02:21:55PM -0700, Thierry Foucu wrote: > Fixed an Use-of-uninitialized-value > --- > libavformat/mpeg.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c > index 265b2bd1ad..15a768e6e8 100644 > --- a/libavforma

[FFmpeg-devel] [PATCH v5 3/3] [RFC] Update documentation and configure script for FLIF

2020-08-14 Thread Anamitra Ghorui
Signed-off-by: Anamitra Ghorui --- Changelog| 3 ++- configure| 1 + doc/general.texi | 2 ++ 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/Changelog b/Changelog index 1efc768387..77b8f46a19 100644 --- a/Changelog +++ b/Changelog @@ -14,7 +14,8 @@ version : - ADPC

[FFmpeg-devel] [PATCH v5 2/3] [RFC] lavfmt: add FLIF demuxing support

2020-08-14 Thread Anamitra Ghorui
This patch removes a redundant class member mentioned in v3, and skips iCCP metadata segments. Signed-off-by: Anamitra Ghorui --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/flifdec.c| 431 +++ libavformat/version.h

Re: [FFmpeg-devel] [PATCH v4 1/3] [RFC] lavc: add FLIF decoding support

2020-08-14 Thread Anamitra Ghorui
Patches need to be self contained. Didn't notice that before. Will repost. Regards, Anamitra ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-

[FFmpeg-devel] [PATCH v4 2/3] [RFC] lavfmt: add FLIF demuxing support

2020-08-14 Thread Anamitra Ghorui
This patch removes a redundant class member mentioned in v3, and skips iCCP metadata segments. Signed-off-by: Anamitra Ghorui --- Changelog| 3 +- configure| 250 --- doc/general.texi | 2 + libavformat/Makefile | 1 + libavf

[FFmpeg-devel] [PATCH v4 3/3] [RFC] lavc: add codec entries for FLIF

2020-08-14 Thread Anamitra Ghorui
Signed-off-by: Anamitra Ghorui --- libavcodec/codec_desc.c | 7 +++ libavcodec/codec_id.h | 1 + 2 files changed, 8 insertions(+) diff --git a/libavcodec/codec_desc.c b/libavcodec/codec_desc.c index 0ae6aee63b..11acb91b76 100644 --- a/libavcodec/codec_desc.c +++ b/libavcodec/codec_desc.c @

Re: [FFmpeg-devel] [PATCH v4] avformat/mxfdec: Read video range from PictureDescriptor

2020-08-14 Thread Harry Mallon
> On 14 Aug 2020, at 11:53, Tomas Härdin wrote: > > tor 2020-08-13 klockan 22:21 +0200 skrev Marton Balint: >> >> On Thu, 13 Aug 2020, Tomas Härdin wrote: >> >>> tor 2020-08-13 klockan 11:04 +0100 skrev Harry Mallon: Here is an updated patch (now hopefully going with correct email heade

[FFmpeg-devel] [PATCH 3/3] avformat/hls: Use av_init_pkt() directly

2020-08-14 Thread Andreas Rheinhardt
and remove reset_packet(). The packet's data pointer is already zeroed, so the only thing that reset_packet() does that av_init_pkt() doesn't is redundant. Signed-off-by: Andreas Rheinhardt --- libavformat/hls.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/lib

[FFmpeg-devel] [PATCH 2/3] avformat/hls: Remove redundant resetting of AVPacket

2020-08-14 Thread Andreas Rheinhardt
av_read_frame() already returns blank packets on error. Signed-off-by: Andreas Rheinhardt --- libavformat/hls.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index 4a3e0d6842..8217c5ede4 100644 --- a/libavformat/hls.c +++ b/libavformat/hls.c @@ -2162,7

[FFmpeg-devel] [PATCH] avformat/hls: Fix memleak when url is empty

2020-08-14 Thread Andreas Rheinhardt
Fixes Coverity ID 1465888. Signed-off-by: Andreas Rheinhardt --- Why does this code actually not check the return value of ff_make_absolute_url()? libavformat/hls.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index 84f0a5f323..4a3

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mpegtsenc: support DVB 6A descriptor for AC-3

2020-08-14 Thread Marton Balint
On Fri, 14 Aug 2020, lance.lmw...@gmail.com wrote: From: Limin Wang Signed-off-by: Limin Wang --- Sorry, fix for the patch order for fate test libavformat/mpegts.h| 16 +++ libavformat/mpegtsenc.c | 76 + I think the ac3_parser de

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: parse for AC-3 descriptor 6A

2020-08-14 Thread Marton Balint
On Fri, 14 Aug 2020, lance.lmw...@gmail.com wrote: From: Limin Wang I don't think this patch fits into libavformat. I am not a fan of dumping the descriptor data with AV_LOG_DEBUG, because libavformat is not a stream analyzer. Also I don't see much benefit of parsing the whole descriptor

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Derek Buitenhuis
On 14/08/2020 20:13, Paul B Mahol wrote: >> Resending because I accidentally replied to James instead of the list. >> Woops. >> >> I guess it was not clear to me this is not the initial thread, since it is >> not >> a v2 patch, and no other thread is titled this, or seems to include SIMD? >> Perhap

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Paul B Mahol
On 8/14/20, Derek Buitenhuis wrote: > On 14/08/2020 19:01, James Almer wrote: >> On the very first email from this thread, he said "Please review and/or >> benchmark, especially .asm file". He did not state his benchmarks were >> irrelevant at first, but he did ask others for theirs. > > Resending

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add disable_all_auto_conversion_filters option.

2020-08-14 Thread Mark Thompson
On 14/08/2020 18:14, Nicolas George wrote: Signed-off-by: Nicolas George --- doc/ffmpeg.texi | 5 + fftools/ffmpeg.h| 1 + fftools/ffmpeg_filter.c | 2 ++ fftools/ffmpeg_opt.c| 3 +++ 4 files changed, 11 insertions(+) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi

Re: [FFmpeg-devel] [PATCH 2/2] fate: disable automatic conversions on most filter tests.

2020-08-14 Thread Mark Thompson
On 14/08/2020 18:14, Nicolas George wrote: Explicitly insert the scale or aresample filter where it would have been inserted by the negotiation. If a conversion is needed in a test, we want to know about it. If the negotiation changes and makes new conversion necessary, we want to know about it

Re: [FFmpeg-devel] [PATCH v2 2/3] vaapi_encode: Rewrite slice/tile support

2020-08-14 Thread Mark Thompson
On 14/08/2020 10:30, Linjie Fu wrote: On Fri, Aug 14, 2020 at 5:51 AM Mark Thompson wrote: This precalculates all of the information we will need to define slice and tile positions at init time rather than rebuilding some of it with every slice. The control of tiles is generalised to match sl

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Derek Buitenhuis
On 14/08/2020 19:01, James Almer wrote: > On the very first email from this thread, he said "Please review and/or > benchmark, especially .asm file". He did not state his benchmarks were > irrelevant at first, but he did ask others for theirs. Resending because I accidentally replied to James inst

Re: [FFmpeg-devel] [PATCH] avutil/opt: Restore NULL input handling to set_string_video_rate()

2020-08-14 Thread Andreas Rheinhardt
Michael Niedermayer: > On Thu, Aug 13, 2020 at 12:41:26PM +0100, Jack Haughton wrote: >> Commit a500b975 removed NULL input handling from this function, >> moving the check higher up the call tree in one branch. However, >> there is another call to set_string_video_rate() which may pass >> NULL, an

Re: [FFmpeg-devel] [PATCH, RFC] lavc/hevcdec: add invalid for skip_frame to skip invalid nalus before IRAP

2020-08-14 Thread Mark Thompson
On 14/08/2020 11:35, Linjie Fu wrote: Add "-skip_frame invalid" option to allow user to request decoder to skip invalid nalus before an IRAP. This would benefit decoding pipeline of bitstreams who didn't start from an IRAP frame. NULL pointer pointing to missing reference may lead to unexpected

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread James Almer
On 8/14/2020 2:44 PM, Derek Buitenhuis wrote: > On 14/08/2020 18:36, Paul B Mahol wrote: >> You are not being helpful at all. > > It's called having basic standards for commit messages. > >> I clearly asked for testers to give their benchmarks as mine are >> little relevant. > > You did not. Fee

Re: [FFmpeg-devel] [PATCH] avutil/opt: Restore NULL input handling to set_string_video_rate()

2020-08-14 Thread Michael Niedermayer
On Thu, Aug 13, 2020 at 12:41:26PM +0100, Jack Haughton wrote: > Commit a500b975 removed NULL input handling from this function, > moving the check higher up the call tree in one branch. However, > there is another call to set_string_video_rate() which may pass > NULL, and future users of the funct

Re: [FFmpeg-devel] [PATCH v4 2/3] [RFC] lavfmt: add FLIF demuxing support

2020-08-14 Thread Nicolas George
Anamitra Ghorui (12020-08-14): > This patch removes a redundant class member mentioned in v3, and > skips iCCP metadata segments. > > Signed-off-by: Anamitra Ghorui > --- > Changelog| 3 +- > configure| 250 --- Looks like you have a lot of

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Derek Buitenhuis
On 14/08/2020 18:36, Paul B Mahol wrote: > You are not being helpful at all. It's called having basic standards for commit messages. > I clearly asked for testers to give their benchmarks as mine are > little relevant. You did not. Feel free to point out where you did, because I cannnot find it.

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Paul B Mahol
On 8/14/20, Derek Buitenhuis wrote: > On 14/08/2020 16:37, Paul B Mahol wrote: >>> I would expect any SIMD patch to include benchmarks showing it >>> is actually faster. >> It is faster, but I have only Celeron CPU to test. > > I can't tell if you're trolling or not, but that doesn't > change the

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Paul B Mahol
On 8/14/20, Derek Buitenhuis wrote: > On 14/08/2020 16:37, Paul B Mahol wrote: >>> I would expect any SIMD patch to include benchmarks showing it >>> is actually faster. >> It is faster, but I have only Celeron CPU to test. > > I can't tell if you're trolling or not, but that doesn't > change the

Re: [FFmpeg-devel] [PATCH] swresample/rematrix: handle 22.2 as a 9 channel layout

2020-08-14 Thread Michael Niedermayer
On Thu, Aug 13, 2020 at 01:26:58AM +0300, Jan Ekström wrote: > This is as far as 22.2 follows the same channel order as > WaveFormatExtensible's channel mask (and the AV_CH_* defines). > > After LFE2 the side channels would follow, but that offset of > one stops us from utilizing them without furt

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mlvdec: Check for existence of AVIOContext before using it

2020-08-14 Thread Andreas Rheinhardt
Andreas Rheinhardt: > The mlv demuxer supports input split into multiple files; if invalid > data is encountered when parsing one of the subsequent files, that file > is closed. But at this point some index entries belonging to this file > might already have been added. In this case, the read_packe

[FFmpeg-devel] [PATCH 2/2] fate: disable automatic conversions on most filter tests.

2020-08-14 Thread Nicolas George
Explicitly insert the scale or aresample filter where it would have been inserted by the negotiation. If a conversion is needed in a test, we want to know about it. If the negotiation changes and makes new conversion necessary, we want to know about it even more. Signed-off-by: Nicolas George --

[FFmpeg-devel] [PATCH 1/2] ffmpeg: add disable_all_auto_conversion_filters option.

2020-08-14 Thread Nicolas George
Signed-off-by: Nicolas George --- doc/ffmpeg.texi | 5 + fftools/ffmpeg.h| 1 + fftools/ffmpeg_filter.c | 2 ++ fftools/ffmpeg_opt.c| 3 +++ 4 files changed, 11 insertions(+) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi index 267ddfe8b5..1603431d56 100644 --- a/doc/ffmp

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Derek Buitenhuis
On 14/08/2020 16:37, Paul B Mahol wrote: >> I would expect any SIMD patch to include benchmarks showing it >> is actually faster. > It is faster, but I have only Celeron CPU to test. I can't tell if you're trolling or not, but that doesn't change the standards... - Derek _

Re: [FFmpeg-devel] [PATCH 1/5] avcodec: split off A53 Closed Caption parsing code into its own file

2020-08-14 Thread James Almer
On 8/10/2020 12:49 PM, James Almer wrote: > Signed-off-by: James Almer > --- > configure | 1 + > libavcodec/Makefile | 1 + > libavcodec/atsc_a53.c | 76 +++ > libavcodec/atsc_a53.h | 39 ++ > 4 files changed, 117 insert

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Paul B Mahol
On 8/14/20, Derek Buitenhuis wrote: >>> Updated patch attached. >>> >> >> Sorry, wrong patch. > > I would expect any SIMD patch to include benchmarks showing it > is actually faster. It is faster, but I have only Celeron CPU to test. > > - Derek > ___

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/utils: calculate frame number of HEVC if the framerate > 30FPS

2020-08-14 Thread lance . lmwang
On Fri, Jul 17, 2020 at 09:26:38PM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang > > --- > libavcodec/internal.h | 3 ++- > libavcodec/nvenc.c| 2 +- > libavcodec/utils.c| 13 - > 3 files changed, 15 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/inter

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegtsenc: support DVB 6A descriptor for AC-3

2020-08-14 Thread lance . lmwang
On Fri, Aug 14, 2020 at 04:33:03PM +0200, Andreas Rheinhardt wrote: > lance.lmw...@gmail.com: > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > libavformat/mpegts.h| 16 +++ > > libavformat/mpegtsenc.c | 76 > > + > >

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegtsenc: support DVB 6A descriptor for AC-3

2020-08-14 Thread Andreas Rheinhardt
lance.lmw...@gmail.com: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavformat/mpegts.h| 16 +++ > libavformat/mpegtsenc.c | 76 > + > 2 files changed, 92 insertions(+) > > diff --git a/libavformat/mpegts.h b/libavforma

[FFmpeg-devel] [PATCH 2/2] avformat/mpegts: parse for AC-3 descriptor 6A

2020-08-14 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/mpegts.c | 53 ++-- 1 file changed, 47 insertions(+), 6 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index f71f18a5..72cc72a 100644 --- a/libavformat/mpegts.c +++

[FFmpeg-devel] [PATCH 1/2] avformat/mpegtsenc: support DVB 6A descriptor for AC-3

2020-08-14 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- Sorry, fix for the patch order for fate test libavformat/mpegts.h| 16 +++ libavformat/mpegtsenc.c | 76 + 2 files changed, 92 insertions(+) diff --git a/libavformat/mpegts.h b/libavform

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Lynne
Aug 13, 2020, 18:23 by one...@gmail.com: > Hi, > > patch attached. > > Please review and/or benchmark, especially .asm file. > I took a look. Its just the horizontal pass of an inverse 2-6 idwt with clipping. The code is so simple I wasn't able to find any obvious ways to improve it, except perh

[FFmpeg-devel] [PATCH] avformat/av1dec: inline obu_read_data() and obu_prefetch() into obu_get_packet()

2020-08-14 Thread James Almer
They don't really help making the demuxer more readable. Signed-off-by: James Almer --- libavformat/av1dec.c | 56 +--- 1 file changed, 21 insertions(+), 35 deletions(-) diff --git a/libavformat/av1dec.c b/libavformat/av1dec.c index 0693e40ac1..c06f5303f5

[FFmpeg-devel] [PATCH 2/2] avformat/mpegtsenc: support DVB 6A descriptor for AC-3

2020-08-14 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/mpegts.h| 16 +++ libavformat/mpegtsenc.c | 76 + 2 files changed, 92 insertions(+) diff --git a/libavformat/mpegts.h b/libavformat/mpegts.h index fe10b38..951aa61 100644 ---

[FFmpeg-devel] [PATCH 1/2] avformat/mpegts: parse for AC-3 descriptor 6A

2020-08-14 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/mpegts.c | 53 ++-- 1 file changed, 47 insertions(+), 6 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index f71f18a5..72cc72a 100644 --- a/libavformat/mpegts.c +++

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Derek Buitenhuis
>> Updated patch attached. >> > > Sorry, wrong patch. I would expect any SIMD patch to include benchmarks showing it is actually faster. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To

Re: [FFmpeg-devel] [PATCH V2] dnn_backend_openvino.c: parse options in openvino backend

2020-08-14 Thread Alexander Strasser
Am 14. August 2020 14:07:23 MESZ schrieb "Xu, Guangxin" : > > >> -Original Message- >> From: ffmpeg-devel On Behalf Of >Guo, >> Yejun >> Sent: Friday, August 14, 2020 9:50 AM >> To: ffmpeg-devel@ffmpeg.org >> Subject: [FFmpeg-devel] [PATCH V2] dnn_backend_openvino.c: parse >options >> in

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Paul B Mahol
On 8/13/20, Paul B Mahol wrote: > Hi, > > patch attached. > > Please review and/or benchmark, especially .asm file. > Updated patch attached. 0001-avcodec-cfhd-add-x86-SIMD.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmp

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Paul B Mahol
On 8/14/20, Paul B Mahol wrote: > On 8/13/20, Paul B Mahol wrote: >> Hi, >> >> patch attached. >> >> Please review and/or benchmark, especially .asm file. >> > > Updated patch attached. > Sorry, wrong patch. New patch attached. 0001-avcodec-cfhd-add-x86-SIMD.patch Description: Binary data ___

Re: [FFmpeg-devel] [PATCH V2] dnn_backend_openvino.c: parse options in openvino backend

2020-08-14 Thread Xu, Guangxin
> -Original Message- > From: ffmpeg-devel On Behalf Of Guo, > Yejun > Sent: Friday, August 14, 2020 9:50 AM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH V2] dnn_backend_openvino.c: parse options > in openvino backend > > Signed-off-by: Guo, Yejun > --- > libavfilter/

Re: [FFmpeg-devel] [PATCH] dnn_backend_native_layer_mathbinary: add floormod support

2020-08-14 Thread Xu, Guangxin
> -Original Message- > From: ffmpeg-devel On Behalf Of > Mingyu Yin > Sent: Friday, August 14, 2020 6:11 PM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH] dnn_backend_native_layer_mathbinary: add > floormod support > > Signed-off-by: Mingyu Yin > --- > .../dnn/dnn_bac

Re: [FFmpeg-devel] [PATCH v4] avformat/mxfdec: Read video range from PictureDescriptor

2020-08-14 Thread Tomas Härdin
tor 2020-08-13 klockan 22:21 +0200 skrev Marton Balint: > > On Thu, 13 Aug 2020, Tomas Härdin wrote: > > > tor 2020-08-13 klockan 11:04 +0100 skrev Harry Mallon: > > > Here is an updated patch (now hopefully going with correct email headers). > > > > It would be nice if in the future you either

[FFmpeg-devel] [PATCH, RFC] lavc/hevcdec: add invalid for skip_frame to skip invalid nalus before IRAP

2020-08-14 Thread Linjie Fu
Add "-skip_frame invalid" option to allow user to request decoder to skip invalid nalus before an IRAP. This would benefit decoding pipeline of bitstreams who didn't start from an IRAP frame. NULL pointer pointing to missing reference may lead to unexpected hang issues[1] in sub-level like hardwar

[FFmpeg-devel] [PATCH] dnn_backend_native_layer_mathbinary: add floormod support

2020-08-14 Thread Mingyu Yin
Signed-off-by: Mingyu Yin --- .../dnn/dnn_backend_native_layer_mathbinary.c | 17 + .../dnn/dnn_backend_native_layer_mathbinary.h | 1 + tests/dnn/dnn-layer-mathbinary-test.c | 5 + tools/python/convert_from_tensorflow.py | 2 +- tools/python/convert_h

[FFmpeg-devel] [PATCH v2] avcodec/dpx: Read alternative frame rate from television header

2020-08-14 Thread Harry Mallon
Signed-off-by: Harry Mallon --- libavcodec/dpx.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c index b1833ed9ef..7e3ac0af2e 100644 --- a/libavcodec/dpx.c +++ b/libavcodec/dpx.c @@ -216,10 +216,23 @@ static int decode_frame(

Re: [FFmpeg-devel] [PATCH v2 2/3] vaapi_encode: Rewrite slice/tile support

2020-08-14 Thread Linjie Fu
On Fri, Aug 14, 2020 at 5:51 AM Mark Thompson wrote: > > This precalculates all of the information we will need to define slice > and tile positions at init time rather than rebuilding some of it with > every slice. The control of tiles is generalised to match slices, so that > arbitrary tile and

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-14 Thread Jean-Baptiste Kempf
Heya, Curiosity: Since it seems to be SSSE3, does it work on both 32bits and 64bits CPU? Best, On Thu, 13 Aug 2020, at 18:23, Paul B Mahol wrote: > Hi, > > patch attached. > > Please review and/or benchmark, especially .asm file. > > ___ > ffmpeg-d

[FFmpeg-devel] [PATCH 2/4] lavfi/buffersink: clearly document that the Params struct are unused.

2020-08-14 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/buffersink.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/buffersink.h b/libavfilter/buffersink.h index 2ec821c685..c0e6b5386f 100644 --- a/libavfilter/buffersink.h +++ b/libavfilter/buffersink.h @@ -61,7 +61,7 @

[FFmpeg-devel] [PATCH 3/4] lavfi/buffersink: add a summary documentation of the API.

2020-08-14 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/buffersink.h | 36 1 file changed, 36 insertions(+) diff --git a/libavfilter/buffersink.h b/libavfilter/buffersink.h index c0e6b5386f..65d0cca747 100644 --- a/libavfilter/buffersink.h +++ b/libavfilter/buffersink.

[FFmpeg-devel] [PATCH 4/4] ffplay: do not set redundant channel count on abuffersink.

2020-08-14 Thread Nicolas George
Signed-off-by: Nicolas George --- fftools/ffplay.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fftools/ffplay.c b/fftools/ffplay.c index d673b8049a..6c9c041e9a 100644 --- a/fftools/ffplay.c +++ b/fftools/ffplay.c @@ -2008,7 +2008,7 @@ static int configure_audio_filters(Vid

Re: [FFmpeg-devel] [PATCH v2] doc/developer: origin of tables should be documented.

2020-08-14 Thread Jean-Baptiste Kempf
On Wed, 12 Aug 2020, at 14:38, Alexander Strasser wrote: > On 2020-08-12 12:32 +0200, Jean-Baptiste Kempf wrote: > > On Wed, 12 Aug 2020, at 00:29, Alexander Strasser wrote: > > > Definitions of non-obvious data should have a short comment > > > explaining their origin. > > > > > > If t

[FFmpeg-devel] [PATCH 1/4] lavfi/buffersink: remove redundant channel layouts.

2020-08-14 Thread Nicolas George
Signed-off-by: Nicolas George --- libavfilter/buffersink.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/libavfilter/buffersink.c b/libavfilter/buffersink.c index 76a46f6678..c58daf6124 100644 --- a/libavfilter/buffersink.c +++ b/libavfilter/buffersink.c @@ -62,6 +6

[FFmpeg-devel] [PATCH] dnn/native: add log error message

2020-08-14 Thread Ting Fu
Signed-off-by: Ting Fu --- libavfilter/dnn/dnn_backend_native.c | 74 +++ libavfilter/dnn/dnn_backend_native.h | 5 ++ .../dnn/dnn_backend_native_layer_avgpool.c| 2 +- .../dnn/dnn_backend_native_layer_avgpool.h| 2 +- .../dnn/dnn_backend_native_layer_

Re: [FFmpeg-devel] [PATCH] fix memory leak in qsvenc.c

2020-08-14 Thread Linjie Fu
On Thu, Aug 13, 2020 at 10:15 AM Alex Pokotilo wrote: > > > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with

[FFmpeg-devel] [PATCH] avcodec/dpx: Read alternative frame rate from television header

2020-08-14 Thread Harry Mallon
Signed-off-by: Harry Mallon --- libavcodec/dpx.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c index b1833ed9ef..694deb27c5 100644 --- a/libavcodec/dpx.c +++ b/libavcodec/dpx.c @@ -216,10 +216,22 @@ static int decode_frame(A

Re: [FFmpeg-devel] [PATCH] lavfi: check the validity of formats lists.

2020-08-14 Thread Nicolas George
Michael Niedermayer (12020-08-14): > this > ffplay -nostats mm-short.mpg -t 1 -af volume=replaygain=track,ebur128 -nodisp > -autoexit > > gets stuck with the patch, seems not depend on the input sample Thanks for the testing. It reveals two bugs in ffplay: - On buffersink, it sets both channe

Re: [FFmpeg-devel] [PATCH] lavfi: check the validity of formats lists.

2020-08-14 Thread Michael Niedermayer
On Thu, Aug 13, 2020 at 01:20:28PM +0200, Nicolas George wrote: > Part of the code expect valid lists, in particular no duplicates. > These tests allow to catch bugs in filters (unlikely but possible) > and to give a clear message when the error comes from the user > ((a)formats) or the application

Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer copies are done before submitting them

2020-08-14 Thread Steve Lhomme
On 2020-08-13 1:01, Soft Works wrote: -Original Message- From: ffmpeg-devel On Behalf Of Steve Lhomme Sent: Wednesday, August 12, 2020 2:05 PM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer copies are done before submitting them On