Re: [FFmpeg-devel] [PATCH] avcodec/libopusenc: add option to set inband FEC

2020-09-07 Thread Gyan Doshi
Plan to push in a day. On 06-09-2020 12:27 pm, Gyan Doshi wrote: --- doc/encoders.texi | 5 + libavcodec/libopusenc.c | 9 + 2 files changed, 14 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 69bf742c2d..a48b15eb1b 100644 --- a/doc/encoders.texi

Re: [FFmpeg-devel] [PATCH 4/5] avcodec: add "Argonaut Games Video" descriptor

2020-09-07 Thread Zane van Iperen
On Mon, 7 Sep 2020 20:15:28 -0300 "James Almer" wrote: > > You could change the demuxer you submitted to simply skip the video > track and demux only the audio, which afaik can be decoded by libavcodec. > If this Argo video codec was supported by some other container then it > would make sense

[FFmpeg-devel] [PATCH v2] qsv: add ${includedir}/mfx to the search path for old versions of libmfx

2020-09-07 Thread Haihao Xiang
${includedir}/mfx has been added to Cflags in libmfx.pc in new versions of libmfx. We may add ${includedir}/mfx to the search path for old versions of libmfx so that we may inlcude mfxfoo.h instead of mfx/mfxfoo.h If your libmfx comes without pkg-config support, this patch requires a small change

Re: [FFmpeg-devel] [PATCH v5 2/2] dnn_backend_native_layer_conv2d.c:Add mutithread function

2020-09-07 Thread Guo, Yejun
> -Original Message- > From: ffmpeg-devel On Behalf Of > xuju...@sjtu.edu.cn > Sent: 2020年9月6日 20:29 > To: ffmpeg-devel@ffmpeg.org > Cc: xuju...@sjtu.edu.cn > Subject: [FFmpeg-devel] [PATCH v5 2/2] > dnn_backend_native_layer_conv2d.c:Add mutithread function > > From: Xu Jun > > Use

Re: [FFmpeg-devel] [PATCH 4/5] avcodec: add "Argonaut Games Video" descriptor

2020-09-07 Thread James Almer
On 9/7/2020 8:10 PM, Zane van Iperen wrote: > On Mon, 7 Sep 2020 16:33:58 +0200 > "Paul B Mahol" wrote: > >> >> On Mon, Sep 07, 2020 at 01:14:26PM +, Zane van Iperen wrote: >>> Signed-off-by: Zane van Iperen >>> --- >>> libavcodec/codec_desc.c | 7 +++ >>> libavcodec/codec_id.h | 1 +

Re: [FFmpeg-devel] [PATCH 4/5] avcodec: add "Argonaut Games Video" descriptor

2020-09-07 Thread Zane van Iperen
On Mon, 7 Sep 2020 16:33:58 +0200 "Paul B Mahol" wrote: > > On Mon, Sep 07, 2020 at 01:14:26PM +, Zane van Iperen wrote: > > Signed-off-by: Zane van Iperen > > --- > > libavcodec/codec_desc.c | 7 +++ > > libavcodec/codec_id.h | 1 + > > libavcodec/version.h| 2 +- > > 3 files

Re: [FFmpeg-devel] [PATCH 5/5] avformat: add Argonaut Games BRP demuxer

2020-09-07 Thread Zane van Iperen
On Mon, 7 Sep 2020 16:35:47 +0200 "Paul B Mahol" wrote: > > + > > +fail: > > +/* TODO: Remove once AVFMT_HEADER_CLEANUP lands. */ > > Huh, what is this one comment about? > I believe Andreas is updating all the demuxers so their read_close() function can safely be called if read_header()

Re: [FFmpeg-devel] [PATCH 1/1] avcodec/exr: add support data windows larger or outside display window

2020-09-07 Thread Paul B Mahol
On Mon, Sep 07, 2020 at 02:54:35PM -0700, mindm...@gmail.com wrote: > From: Mark Reid > > Hi, > The following patch adds exr support for data windows that are larger or > outside the display window. > This adds support for both scanline and tiled formats. > > Here are the added exr files for

Re: [FFmpeg-devel] [PATCH 2/2] tools/target_dec_fuzzer: Adjust threshold for WMV3IMAGE

2020-09-07 Thread Michael Niedermayer
On Sun, Sep 06, 2020 at 03:26:17PM +0200, Michael Niedermayer wrote: > Fixes: Timeout (1131sec -> 1sec) > Fixes: > 24727/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_WMV3IMAGE_fuzzer-5754167793287168 > > Found-by: continuous fuzzing process >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/ffwavesynth: Fix integer overflow in wavesynth_synth_sample / WS_SINE

2020-09-07 Thread Michael Niedermayer
On Sun, Sep 06, 2020 at 07:38:07PM +0200, Nicolas George wrote: > Michael Niedermayer (12020-09-06): > > Fixes: signed integer overflow: -1429092 * -32596 cannot be represented in > > type 'int' > > Fixes: > >

Re: [FFmpeg-devel] [PATCH] avformat/electronicarts: Check if there are any streams

2020-09-07 Thread Michael Niedermayer
On Sun, Sep 06, 2020 at 08:50:02PM -0300, James Almer wrote: > On 9/6/2020 7:57 PM, Michael Niedermayer wrote: > > Fixes: Assertion failure (invalid stream index) > > Fixes: > > 25120/clusterfuzz-testcase-minimized-ffmpeg_DEMUXER_fuzzer-6565251898933248 > > > > Found-by: continuous fuzzing

[FFmpeg-devel] [PATCH 1/1] avcodec/exr: add support data windows larger or outside display window

2020-09-07 Thread mindmark
From: Mark Reid Hi, The following patch adds exr support for data windows that are larger or outside the display window. This adds support for both scanline and tiled formats. Here are the added exr files for the fate tests. Could somebody upload them to fate for me?

Re: [FFmpeg-devel] [PATCH] avformat/rtsp: fix infinite loop with udp transport

2020-09-07 Thread Andriy Gelman
On Sun, 30. Aug 11:50, Zhao Zhili wrote: > > > > On Aug 30, 2020, at 1:45 AM, Andriy Gelman wrote: > > > > Hi Zhao, > > > > On Tue, 25. Aug 01:17, Zhao Zhili wrote: > >> Ping again. > >> > >>> On Aug 17, 2020, at 8:05 AM, zhilizhao wrote: > >>> > >>> Please help review the patch, thanks! >

[FFmpeg-devel] [PATCH 2/2 v3] avcodec/decode: use a packet list to store packet properties

2020-09-07 Thread James Almer
Keeping only the latest packet fed to the decoder works only for decoders that return a frame immediately after every consumed packet. Decoders that consume several packets before they return a frame will fill said frame with properties taken from the last consumed packet instead of the earliest.

Re: [FFmpeg-devel] [PATCH 5/5] avformat: add Argonaut Games BRP demuxer

2020-09-07 Thread Paul B Mahol
On Mon, Sep 07, 2020 at 01:14:34PM +, Zane van Iperen wrote: > Used in FMVs for FX Fighter and Croc. Supports BVID and BASF streams, > requests samples for anything else. > > Due to the way BASF streams are contained in the file, only one is > supported. I have yet to see a BRP file with

Re: [FFmpeg-devel] [PATCH 4/5] avcodec: add "Argonaut Games Video" descriptor

2020-09-07 Thread Paul B Mahol
On Mon, Sep 07, 2020 at 01:14:26PM +, Zane van Iperen wrote: > Signed-off-by: Zane van Iperen > --- > libavcodec/codec_desc.c | 7 +++ > libavcodec/codec_id.h | 1 + > libavcodec/version.h| 2 +- > 3 files changed, 9 insertions(+), 1 deletion(-) > If there is no actual video

[FFmpeg-devel] [PATCH 1/2 v3] avcodec/packet: move AVPacketList definition and function helpers over from libavformat

2020-09-07 Thread James Almer
And replace the flags parameter with a function callback that can be used to copy the contents of the packet (e.g, av_packet_ref and av_packet_copy_props). Signed-off-by: James Almer --- This version achieves the same effect as v2 while being cleaner. I could leave the flags parameter, too, for

[FFmpeg-devel] [PATCH] avformat/utils: Avoid duplicating extradata from extract_extradata BSF

2020-09-07 Thread Andreas Rheinhardt
Instead move the extradata contained in packet side-data to its destination. This is possible because the side data already has zeroed padding. Notice that the check for FF_MAX_EXTRADATA_SIZE has been dropped, because said constant is from libavcodec/internal.h. If libavcodec wanted to enforce

[FFmpeg-devel] [PATCH 1/3] avcodec/extract_extradata: Consolidate zeroing extradata padding

2020-09-07 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/extract_extradata_bsf.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/libavcodec/extract_extradata_bsf.c b/libavcodec/extract_extradata_bsf.c index 1fead74b57..6e80076522 100644 --- a/libavcodec/extract_extradata_bsf.c

[FFmpeg-devel] [PATCH 3/3] avcodec/extract_extradata: Reindent after the last commit

2020-09-07 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/extract_extradata_bsf.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/libavcodec/extract_extradata_bsf.c b/libavcodec/extract_extradata_bsf.c index 38517bdcf3..32c732f52b 100644 ---

[FFmpeg-devel] [PATCH 2/3] avcodec/extract_extradata: Remove always-true check

2020-09-07 Thread Andreas Rheinhardt
If a sequence header has already been found, it is certain that the next startcode (being disjoint from the sequence header startcode) can begin at index four at the earliest. Signed-off-by: Andreas Rheinhardt --- libavcodec/extract_extradata_bsf.c | 2 -- 1 file changed, 2 deletions(-) diff

[FFmpeg-devel] [PATCH 5/5] avformat: add Argonaut Games BRP demuxer

2020-09-07 Thread Zane van Iperen
Used in FMVs for FX Fighter and Croc. Supports BVID and BASF streams, requests samples for anything else. Due to the way BASF streams are contained in the file, only one is supported. I have yet to see a BRP file with multiple. Signed-off-by: Zane van Iperen --- Changelog| 1

[FFmpeg-devel] [PATCH 4/5] avcodec: add "Argonaut Games Video" descriptor

2020-09-07 Thread Zane van Iperen
Signed-off-by: Zane van Iperen --- libavcodec/codec_desc.c | 7 +++ libavcodec/codec_id.h | 1 + libavcodec/version.h| 2 +- 3 files changed, 9 insertions(+), 1 deletion(-) diff --git a/libavcodec/codec_desc.c b/libavcodec/codec_desc.c index 9e73dcba27..f33585adaa 100644 ---

[FFmpeg-devel] [PATCH 0/5] Add demuxer for Argonaut Games BRP files

2020-09-07 Thread Zane van Iperen
Adds support for demuxing BRP files from Argonaut Games' games. Used to store the FMVs. Zane van Iperen (5): avformat/argo_asf: bail if invalid tag avformat/argo_asf: split functionality into a header avformat/argo_asf: add ASF_MIN_BUFFER_SIZE #define avcodec: add "Argonaut Games Video"

[FFmpeg-devel] [PATCH 3/5] avformat/argo_asf: add ASF_MIN_BUFFER_SIZE #define

2020-09-07 Thread Zane van Iperen
For future use by the argo_brp demuxer Signed-off-by: Zane van Iperen --- libavformat/argo_asf.c | 2 +- libavformat/argo_asf.h | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/argo_asf.c b/libavformat/argo_asf.c index ee8c85f735..c131207aa1 100644 ---

[FFmpeg-devel] [PATCH 2/5] avformat/argo_asf: split functionality into a header

2020-09-07 Thread Zane van Iperen
For future use by the argo_brp demuxer. Adds: - void ff_argo_asf_parse_file_header(ArgoASFFileHeader *hdr, const uint8_t *buf); - int ff_argo_asf_validate_file_header(AVFormatContext *s, const ArgoASFFileHeader *hdr); - void ff_argo_asf_parse_chunk_header(ArgoASFChunkHeader *hdr, const

[FFmpeg-devel] [PATCH 1/5] avformat/argo_asf: bail if invalid tag

2020-09-07 Thread Zane van Iperen
Signed-off-by: Zane van Iperen --- libavformat/argo_asf.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/argo_asf.c b/libavformat/argo_asf.c index bf9b5d0c0a..208280b4d9 100644 --- a/libavformat/argo_asf.c +++ b/libavformat/argo_asf.c @@ -144,6 +144,9 @@ static int

Re: [FFmpeg-devel] [PATCH] avcodec/hlsenc: Fix playlist not getting flushed when using custom io

2020-09-07 Thread Steven Liu
Steve Lam 于2020年9月7日周一 上午3:23写道: > > --- > libavformat/hlsenc.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c > index cb31d6aed7..fdfa907234 100644 > --- a/libavformat/hlsenc.c > +++ b/libavformat/hlsenc.c > @@ -287,6 +287,7 @@ static int

[FFmpeg-devel] [PATCH v2 3/3] avcodec/mips: Optimize function ff_h264_loop_filter_strength_msa.

2020-09-07 Thread Shiyou Yin
From: gxw Speed of decoding H264: 5.45x ==> 5.53x Signed-off-by: Shiyou Yin --- libavcodec/mips/Makefile| 3 +- libavcodec/mips/h264_deblock_msa.c | 153 libavcodec/mips/h264dsp_init_mips.c | 2 + libavcodec/mips/h264dsp_mips.h | 4

[FFmpeg-devel] [PATCH v2 1/3] avcodec/mips: Restore the initialization sequence of MSA and MMI in ff_h264chroma_init_mips.

2020-09-07 Thread Shiyou Yin
Speed of decoding H264: 5.14x ==> 5.23x (tested on 3A4000). --- libavcodec/mips/h264chroma_init_mips.c | 19 +-- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/libavcodec/mips/h264chroma_init_mips.c b/libavcodec/mips/h264chroma_init_mips.c index 6bb19d3..755cc04

[FFmpeg-devel] [PATCH v2 2/3] avcodec/mips: Refine get_cabac_inline_mips.

2020-09-07 Thread Shiyou Yin
1. Refined function get_cabac_inline_mips. 2. Optimize function get_cabac_bypass and get_cabac_bypass_sign. Speed of decoding h264: 5.23x ==> 5.45x(tested on 3A4000). --- libavcodec/mips/cabac.h | 131 +--- 1 file changed, 102 insertions(+), 29

[FFmpeg-devel] [loongson] Optimize H264 decoding.

2020-09-07 Thread Shiyou Yin
H264 decoding speed: 154fps ==> 165fps, 5.14x ==> 5.53x (tested on 3A4000) V2: Fixed a build error in [PATCH 2/3]. "Error: opcode not supported on this processor: mips32r2 (mips32r2) `dsbh $10,$10'" [PATCH 1/3] avcodec/mips: Restore the initialization sequence of MSA. [PATCH 2/3] avcodec/mips:

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mxfdec: Read Mastering Display Colour Volume from MXF

2020-09-07 Thread Tomas Härdin
mån 2020-09-07 klockan 11:40 +0200 skrev Tomas Härdin: > mån 2020-08-31 klockan 20:07 +0100 skrev Harry Mallon: > > +static const uint8_t mxf_mastering_display_primaries[]= { > > 0x06,0x0e,0x2b,0x34,0x01,0x01,0x01,0x0e,0x04,0x20,0x04,0x01,0x01,0x01,0x00,0x00 > > }; > > +static

Re: [FFmpeg-devel] [PATCH] Support HDR10+ metadata for HEVC.

2020-09-07 Thread Jan Ekström
On Sat, Jul 25, 2020 at 12:09 AM Mohammad Izadi wrote: > > On Fri, Jul 24, 2020 at 9:30 AM Andreas Rheinhardt < > andreas.rheinha...@gmail.com> wrote: > > > Mohammad Izadi: > > > From: Mohammad Izadi > > > > > > HDR10+ is dynamic metadata (A/341 Amendment - SMPTE2094-40) that needs > > to be

Re: [FFmpeg-devel] [PATCH 2/3] avformat/mxfenc: Write Mastering Display Colour Volume to MXF

2020-09-07 Thread Tomas Härdin
mån 2020-08-31 klockan 20:07 +0100 skrev Harry Mallon: > +static const MXFLocalTagPair mxf_mastering_display_local_tags[] = { > +{ 0x8201, > {0x06,0x0e,0x2b,0x34,0x01,0x01,0x01,0x0e,0x04,0x20,0x04,0x01,0x01,0x01,0x00,0x00}}, > /* Mastering Display Primaries */ > +{ 0x8202, >

Re: [FFmpeg-devel] [PATCH 3/3] avformat/mxfdec: Read Apple private Content Light Level from MXF

2020-09-07 Thread Tomas Härdin
mån 2020-08-31 klockan 20:07 +0100 skrev Harry Mallon: > * As embedded by Apple Compressor This needs a sample since it isn't part of any official spec, so that we can have a test for this. > +if (IS_KLV_KEY(uid, mxf_coll_apple_max_cll)) { > +if (!descriptor->coll) { > +

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mxfdec: Read Mastering Display Colour Volume from MXF

2020-09-07 Thread Tomas Härdin
mån 2020-08-31 klockan 20:07 +0100 skrev Harry Mallon: > +static const uint8_t mxf_mastering_display_primaries[]= { > 0x06,0x0e,0x2b,0x34,0x01,0x01,0x01,0x0e,0x04,0x20,0x04,0x01,0x01,0x01,0x00,0x00 > }; > +static const uint8_t mxf_mastering_display_white_point_chromaticity[] = {