[FFmpeg-devel] [PATCH 1/1] avcodec/exr: use lookuptable for alpha if there is no trc_func

2020-11-08 Thread mindmark
From: Mark Reid --- libavcodec/exr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/exr.c b/libavcodec/exr.c index cf7824402a..e907c5c464 100644 --- a/libavcodec/exr.c +++ b/libavcodec/exr.c @@ -1203,7 +1203,7 @@ static int decode_block(AVCodecContext *avctx,

Re: [FFmpeg-devel] [PATCH] lavc/qsvenc: allows the SDK runtime to choose LowPower/non-LowPower modes

2020-11-08 Thread Xiang, Haihao
@Zhong, @Mark Could you take a look at this patch when you get a chance? Thanks Haihao > The SDK supports LowPower and non-LowPower modes, but some features are > available only under one of the two modes. Currently non-LowPower mode > is always chosen in FFmpeg if the mode is not set

Re: [FFmpeg-devel] [PATCH] lavc/qsvenc: add support for Screen Content Coding (SCC) extension

2020-11-08 Thread Xiang, Haihao
@Zhong, @Mark Could you take a look at this patch when you get a chance? Thanks Haihao > > > This patch adds support for SCC encoding on ICL+ platform > > Sample pipeline: > ffmpeg -init_hw_device qsv=qsv:hw -f lavfi -i testsrc -vf \ > "format=nv12,hwupload=extra_hw_frames=64,format=qsv"

[FFmpeg-devel] [PATCH 1/1] avfilter/vf_lut3d: fix sanitizef INF handling

2020-11-08 Thread mindmark
From: Mark Reid --- libavfilter/vf_lut3d.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_lut3d.c b/libavfilter/vf_lut3d.c index 988f6c8b55..172d6df0c8 100644 --- a/libavfilter/vf_lut3d.c +++ b/libavfilter/vf_lut3d.c @@ -107,7 +107,7 @@ typedef struct

Re: [FFmpeg-devel] [PATCH 1/4] avformat/nutenc: don't use header_count to store different variables

2020-11-08 Thread Andriy Gelman
On Mon, 09. Nov 00:04, Andreas Rheinhardt wrote: > Andriy Gelman: > > From: Andriy Gelman > > > > Currently, header_count is used to store both the elision header count > > and the header repetition count (number of times headers have been written > > to output). Fix this by using a separate

Re: [FFmpeg-devel] [PATCH 12/12] avformat/nutenc: Fix repeating headers

2020-11-08 Thread Michael Niedermayer
On Thu, May 07, 2020 at 06:05:01PM +0200, Andreas Rheinhardt wrote: > Michael Niedermayer: > > On Tue, May 05, 2020 at 04:16:57PM +0200, Andreas Rheinhardt wrote: [...] > > > >> I am also unsure why the size difference between successive headers > >> grows exponentially. > > > > I think i dont

Re: [FFmpeg-devel] [PATCH 11/12] avformat/nutenc: Don't segfault when chapters are added during muxing

2020-11-08 Thread Michael Niedermayer
On Thu, May 07, 2020 at 05:49:06PM +0200, Andreas Rheinhardt wrote: > Michael Niedermayer: > > On Tue, May 05, 2020 at 04:16:56PM +0200, Andreas Rheinhardt wrote: > >> When writing the header, the NUT muxer allocates an array with as many > >> entries as there are chapters containing information

Re: [FFmpeg-devel] [PATCH 08/10] avformat/nutenc: Cosmetics

2020-11-08 Thread Michael Niedermayer
On Mon, May 04, 2020 at 08:22:48PM +0200, Andreas Rheinhardt wrote: > Mainly reindentation. > > Signed-off-by: Andreas Rheinhardt > --- > libavformat/nutenc.c | 42 ++ > 1 file changed, 22 insertions(+), 20 deletions(-) probably ok [...] -- Michael

[FFmpeg-devel] [PATCH] avfilter: add speechnorm filter

2020-11-08 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- doc/filters.texi| 57 + libavfilter/Makefile| 1 + libavfilter/af_speechnorm.c | 441 libavfilter/allfilters.c| 1 + 4 files changed, 500 insertions(+) create mode 100644

Re: [FFmpeg-devel] [PATCH 1/4] avformat/nutenc: don't use header_count to store different variables

2020-11-08 Thread Andreas Rheinhardt
Andriy Gelman: > From: Andriy Gelman > > Currently, header_count is used to store both the elision header count > and the header repetition count (number of times headers have been written > to output). Fix this by using a separate variable to store repetition > count. > > Signed-off-by: Andriy

Re: [FFmpeg-devel] [PATCH 4/5] avformat/lvfdec: Check stream_index before use

2020-11-08 Thread Paul B Mahol
LGTM On Sun, Nov 8, 2020 at 12:18 AM Michael Niedermayer wrote: > Fixes: assertion failure > Fixes: > 26905/clusterfuzz-testcase-minimized-ffmpeg_dem_LVF_fuzzer-5724267599364096.fuzz > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg >

Re: [FFmpeg-devel] [PATCH 1/5] lavc/amfenc: HWConfig for amf encoder.

2020-11-08 Thread Mark Thompson
On 28/10/2020 23:28, Dmitrii Ovchinnikov wrote: This looks right, but can you clarify the intent behind the hw config cases? (Do you have an ffmpeg command line or other example which is improved by them?) Hi! This ticket describes the problem. https://trac.ffmpeg.org/ticket/8953#ticket I

[FFmpeg-devel] [PATCH 2/4] avformat/nutenc: check return of write_headers()

2020-11-08 Thread Andriy Gelman
From: Andriy Gelman Signed-off-by: Andriy Gelman --- libavformat/nutenc.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavformat/nutenc.c b/libavformat/nutenc.c index 87adef6f7e..0646f72af2 100644 --- a/libavformat/nutenc.c +++ b/libavformat/nutenc.c @@ -988,8

[FFmpeg-devel] [PATCH 3/4] avformat/nutenc: add option to periodically insert global headers

2020-11-08 Thread Andriy Gelman
From: Andriy Gelman Signed-off-by: Andriy Gelman --- doc/muxers.texi | 6 ++ libavformat/nut.h| 3 ++- libavformat/nutenc.c | 10 ++ 3 files changed, 18 insertions(+), 1 deletion(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index 813b4678f4..ace236fca6 100644 ---

[FFmpeg-devel] [PATCH 1/4] avformat/nutenc: don't use header_count to store different variables

2020-11-08 Thread Andriy Gelman
From: Andriy Gelman Currently, header_count is used to store both the elision header count and the header repetition count (number of times headers have been written to output). Fix this by using a separate variable to store repetition count. Signed-off-by: Andriy Gelman --- libavformat/nut.h

[FFmpeg-devel] [PATCH 4/4] avformat/nutenc: cosmetics

2020-11-08 Thread Andriy Gelman
From: Andriy Gelman Realign from previous commit and fix typo in comment. --- libavformat/nutenc.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavformat/nutenc.c b/libavformat/nutenc.c index 5a775a92fe..0323984556 100644 --- a/libavformat/nutenc.c +++

[FFmpeg-devel] [PATCH] libavformat/movenc.c: Fix avg_frame_rate inversion on timecode track writing

2020-11-08 Thread Ivan Baykalov
When mov_create_timecode_track function is executed, track->st->avg_frame_rate becomes inverted due to track->st->avg_frame_rate = av_inv_q(rate); , where "rate" is frame per second here. I didn't find any obvious negative effect of this operation, but this may lead to some misunderstanding.

Re: [FFmpeg-devel] [PATCH v4] Unbreak av_malloc_max(0) API/ABI

2020-11-08 Thread Michael Niedermayer
On Sun, Nov 08, 2020 at 11:35:43AM +0100, Andreas Rheinhardt wrote: > Michael Niedermayer: > > On Fri, Oct 16, 2020 at 10:57:22AM +0200, Joakim Tjernlund wrote: > >> From https://bugs.chromium.org/p/chromium/issues/detail?id=1095962 > >> > >> This seems to be caused by

Re: [FFmpeg-devel] [PATCH v4] Unbreak av_malloc_max(0) API/ABI

2020-11-08 Thread Andreas Rheinhardt
Michael Niedermayer: > On Fri, Oct 16, 2020 at 10:57:22AM +0200, Joakim Tjernlund wrote: >> From https://bugs.chromium.org/p/chromium/issues/detail?id=1095962 >> >> This seems to be caused by the custom handling of "av_max_alloc(0)" in >> Chromium's ffmpeg fork to mean

Re: [FFmpeg-devel] [PATCH v4] Unbreak av_malloc_max(0) API/ABI

2020-11-08 Thread Michael Niedermayer
On Fri, Oct 16, 2020 at 10:57:22AM +0200, Joakim Tjernlund wrote: > From https://bugs.chromium.org/p/chromium/issues/detail?id=1095962 > > This seems to be caused by the custom handling of "av_max_alloc(0)" in > Chromium's ffmpeg fork to mean unlimited (added in [1]).

[FFmpeg-devel] [PATCH v2] avformat/dhav: also support ZLAV packets

2020-11-08 Thread Michael Keeley
Some DVRs (e.g. D7008FH made by iSmart Cities (Zhuhai) Limited) output the same format .dav file, but use ZLAV/zlav tags to delimit the packets instead of DHAV/dhav. Signed-off-by: Michael Keeley --- Changelog | 1 + libavformat/dhav.c| 31 ++-