[FFmpeg-devel] [PATCH] avformat/movenc: let avpriv_ac3_parse_header() allocate the AC3HeaderInfo struct

2017-10-30 Thread James Almer
This removes sizeof(AC3HeaderInfo) from the ABI. Signed-off-by: James Almer --- libavformat/movenc.c | 72 +++- 1 file changed, 43 insertions(+), 29 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index a920eb7c8f..cc3fc19d9b

[FFmpeg-devel] [PATCH] avcodec/dca: return standard error codes in avpriv_dca_parse_core_frame_header()

2017-10-30 Thread James Almer
This prevents making the DCAParseError enum part of the ABI. Signed-off-by: James Almer --- libavcodec/dca.c | 11 --- libavcodec/dca.h | 12 ++-- 2 files changed, 18 insertions(+), 5 deletions(-) diff --git a/libavcodec/dca.c b/libavcodec/dca.c index 942fe6c3c9..799c414259 1006

Re: [FFmpeg-devel] [PATCH 6/8] x86inc: reduce difference to x264 upstream

2017-10-30 Thread Michael Niedermayer
On Mon, Oct 30, 2017 at 02:08:33PM +0100, James Darnley wrote: > These changes were commited to x264 in b568a256 "Experimental nasm > support" > --- > libavutil/x86/x86inc.asm | 16 ++-- > 1 file changed, 14 insertions(+), 2 deletions(-) breaks build: libavcodec/x86/rv34dsp.asm:211:

[FFmpeg-devel] [PATCH 2/2] lavc/vaapi_encode_h264: correct bit_rate_scale setting.

2017-10-30 Thread Jun Zhao
From d1e105057e93e7c2788d6d684292db9008fbf3ac Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Tue, 31 Oct 2017 10:19:08 +0800 Subject: [PATCH 2/2] lavc/vaapi_encode_h264: correct bit_rate_scale setting. As H264 Spec 2012 E.2.2, bit_rate_scale means the max input bit rate. Signed-off-by: Jun Zhao

[FFmpeg-devel] [PATCH 1/2] lavc/vaapi_encode: correct the HRD buffer size calculate.

2017-10-30 Thread Jun Zhao
From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Tue, 31 Oct 2017 10:13:42 +0800 Subject: [PATCH 1/2] lavc/vaapi_encode: correct the HRD buffer size calculate. when rc_buffer_size didn't setting, always use the max bit rate per second as HRD buffer size

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-30 Thread Michael Niedermayer
On Thu, Oct 19, 2017 at 11:46:47AM -0700, Nikolas Bowe wrote: > Found via fuzzing. > /tmp/poc is a 1 MB mpegts file generated via fuzzing, where 1 packet has many > NALUs > Before this change: > $ /usr/bin/time -f "\t%M Max Resident Set Size (Kb)" ./ffprobe /tmp/poc > 2>&1 | tail -n 1 >

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-30 Thread Michael Niedermayer
On Thu, Oct 19, 2017 at 11:46:47AM -0700, Nikolas Bowe wrote: > Found via fuzzing. > /tmp/poc is a 1 MB mpegts file generated via fuzzing, where 1 packet has many > NALUs > Before this change: > $ /usr/bin/time -f "\t%M Max Resident Set Size (Kb)" ./ffprobe /tmp/poc > 2>&1 | tail -n 1 >

[FFmpeg-devel] [PATCH]lavc/dnxhddata: Do not print frame rates with supported profiles

2017-10-30 Thread Carl Eugen Hoyos
Hi! Attached patch is meant to fix ticket #4815: Nobody can maintain the list of frame-rates, the current output primarily leads to confusion instead of helping users. Please comment, Carl Eugen From ab2371ace24d4e2ed99b4f5beb2676433cf15e88 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: T

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-30 Thread Michael Niedermayer
Hi On Mon, Oct 23, 2017 at 04:43:55PM -0700, Niki Bowe wrote: > On Thu, Oct 19, 2017 at 3:39 PM, Carl Eugen Hoyos > wrote: > > > 2017-10-19 20:46 GMT+02:00 Nikolas Bowe : > > > Found via fuzzing. > > > /tmp/poc is a 1 MB mpegts file generated via fuzzing, where 1 packet has > > many NALUs > > >

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-30 Thread Kieran Kunhya
On Tue, 31 Oct 2017 at 01:42 Kieran Kunhya wrote: > On Tue, 24 Oct 2017 at 00:56 Carl Eugen Hoyos wrote: > >> 2017-10-24 1:43 GMT+02:00 Niki Bowe : >> > On Thu, Oct 19, 2017 at 3:39 PM, Carl Eugen Hoyos >> > wrote: >> >> >> Does the patch have a measurable speed impact? >> >> >> > Is there a st

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-30 Thread Kieran Kunhya
On Tue, 24 Oct 2017 at 00:56 Carl Eugen Hoyos wrote: > 2017-10-24 1:43 GMT+02:00 Niki Bowe : > > On Thu, Oct 19, 2017 at 3:39 PM, Carl Eugen Hoyos > > wrote: > > >> Does the patch have a measurable speed impact? > >> > > Is there a standard set of benchmarks I can run? > > > > For typical videos

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-30 Thread Marton Balint
On Mon, 30 Oct 2017, Mironov, Mikhail wrote: [...] I still think this would be much better off using the send_frame()/receive_packet() API. Even if your API doesn't expose any information about the queue length, you only need to hold a single input frame transiently to get around that (the u

Re: [FFmpeg-devel] On in-tree external headers

2017-10-30 Thread Stephen Hutchinson
On 10/30/2017 4:16 PM, Jan Ekstrom wrote: On Mon, Oct 30, 2017 at 9:51 PM, Mark Thompson wrote: PPS: The position stated above would imply removing the avisynth headers. Can anyone who uses it comment on what would be required for that? Avisynth headers are available from the Avisynth SDK

Re: [FFmpeg-devel] [PATCH 4/8] avutil: add alignment needed for AVX-512

2017-10-30 Thread James Almer
On 10/30/2017 10:08 AM, James Darnley wrote: > --- > libavutil/x86/cpu.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavutil/x86/cpu.c b/libavutil/x86/cpu.c > index b22a950b79..c56f2a8754 100644 > --- a/libavutil/x86/cpu.c > +++ b/libavutil/x86/cpu.c > @@ -247,6 +247,8 @@ size_t

Re: [FFmpeg-devel] [PATCH] lavfi/testsrc2: fix hang with very small sizes.

2017-10-30 Thread Martin Vignali
2017-10-30 10:14 GMT+01:00 Nicolas George : > Move a subtraction to the other side of the equal to avoid overflows. > > Signed-off-by: Nicolas George > --- > libavfilter/vsrc_testsrc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavfilter/vsrc_testsrc.c b/libav

Re: [FFmpeg-devel] [PATCH 0/8] AVX-512 support (v.1)

2017-10-30 Thread Martin Vignali
Hello, > > The second is the v210enc function, it passes checkasm but > > it is not any faster. It is there to show that all the previous > > commits work correctly, namely: configure checks, cpuid > > detection, x86inc changes, checkasm. > > >

Re: [FFmpeg-devel] [RFC]lswr/rematrix: Support s32p

2017-10-30 Thread Carl Eugen Hoyos
2017-10-30 9:47 GMT+01:00 Muhammad Faiz : > On Sun, Oct 29, 2017 at 3:55 AM, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes a random testcase for ticket #6785 here but I >> don't know if this is the correct fix. >> >> Please review, Carl Eugen >> >> From a93b9309d74f5eadece371ee1e682d26

[FFmpeg-devel] [PATCH]lavf/mxfenc: Support 60fps output

2017-10-30 Thread Carl Eugen Hoyos
Hi! Kasper Folman has reported that FFmpeg does not write 60fps mxf output and has provided a 60fps mxf file on -users. Attached patch allows 60fps output. Please comment, Carl Eugen From bb5b071db9a8977bd1a7dea579b4d244e17ca3a3 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Tue, 31 Oct

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-30 Thread Mironov, Mikhail
> >>> + > >>> +int ff_amf_encode_frame(AVCodecContext *avctx, AVPacket *pkt, > >>> +const AVFrame *frame, int *got_packet) { > >>> +int ret = 0; > >>> +AMF_RESULT res = AMF_OK; > >>> +AmfContext *ctx = avctx->priv_data; > >>> +AMFSurface

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-30 Thread Mironov, Mikhail
Mikhail > -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Carl Eugen Hoyos > Sent: October 30, 2017 6:19 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding f

[FFmpeg-devel] [PATCH]lavf/latmenc: Error out for invalid codecs

2017-10-30 Thread Carl Eugen Hoyos
Hi! Attached patch makes sure the loas muxer does not try to write anything but aac and latm. Please comment, Carl Eugen From 9f8f39b402f77b53613a395129f96feee5e873ba Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 30 Oct 2017 23:49:29 +0100 Subject: [PATCH] lavf/latmenc: Error out fo

Re: [FFmpeg-devel] [PATCH 0/8] AVX-512 support (v.1)

2017-10-30 Thread Carl Eugen Hoyos
2017-10-30 14:08 GMT+01:00 James Darnley : > There are 2 commits here that I don't intend to be applied (now). > The first is the alignment increase reported by avutil. Why is that bad? > The second is the v210enc function, it passes checkasm but > it is not any faster. It is there to show tha

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-30 Thread Carl Eugen Hoyos
2017-10-30 23:35 GMT+01:00 Mark Thompson : > On 30/10/17 21:30, Mironov, Mikhail wrote: +static void AMF_CDECL_CALL AMFTraceWriter_Write(AMFTraceWriter >>> *pThis, +const wchar_t *scope, const wchar_t *message) +{ +AmfTraceWriter *tracer = (AmfTraceWriter*)pThis; +

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-30 Thread Mark Thompson
On 30/10/17 21:30, Mironov, Mikhail wrote: >>> +static void AMF_CDECL_CALL AMFTraceWriter_Write(AMFTraceWriter >> *pThis, >>> +const wchar_t *scope, const wchar_t *message) >>> +{ >>> +AmfTraceWriter *tracer = (AmfTraceWriter*)pThis; >>> +av_log(tracer->avctx, AV_LOG_DEBUG, "%ls: %ls",

[FFmpeg-devel] [PATCH 1/2] avcodec/xan: Improve overlapping check

2017-10-30 Thread Michael Niedermayer
Fixes: memcpy-param-overlap Fixes: 3612/clusterfuzz-testcase-minimized-6393461273001984 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/xan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(

[FFmpeg-devel] [PATCH 2/2] avcodec/h264idct_template: Fix integer overflows in ff_h264_idct8_add()

2017-10-30 Thread Michael Niedermayer
Fixes: runtime error: signed integer overflow: 924846844 + 1457520640 cannot be represented in type 'int' Fixes: 3416/clusterfuzz-testcase-minimized-6125587682820096 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niederm

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-30 Thread Carl Eugen Hoyos
2017-10-30 18:56 GMT+01:00 mmironov : [...] > +const enum AVPixelFormat ff_amf_pix_fmts[] = { > +AV_PIX_FMT_NV12, > +AV_PIX_FMT_0RGB32, > +AV_PIX_FMT_0BGR32, > +AV_PIX_FMT_YUV420P, > +AV_PIX_FMT_D3D11, > +AV_PIX_FMT_NONE > +}; > + > +typedef struct FormatMap { > +enum

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-30 Thread Mironov, Mikhail
> > +static void AMF_CDECL_CALL AMFTraceWriter_Write(AMFTraceWriter > *pThis, > > +const wchar_t *scope, const wchar_t *message) > > +{ > > +AmfTraceWriter *tracer = (AmfTraceWriter*)pThis; > > +av_log(tracer->avctx, AV_LOG_DEBUG, "%ls: %ls", scope, message); > > Does the message neces

Re: [FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-30 Thread Thomas Köppe
Please find below the actual patch: == Variables used in inline assembly need to be marked with attribute((used)). Static constants already were, via the define of DECLARE_ASM_CONST. But DECLARE_ALIGNED does not add this attribute, and some of the variables defined with it are const on

Re: [FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-30 Thread Thomas Köppe
(I must have forgotten to rebase first!) On 30 October 2017 at 20:14, Thomas Köppe wrote: > It should be under __GNUC__ -- something went wrong with the patch! I'll > make a new one. Sorry about that! > > On 30 October 2017 at 19:31, James Almer wrote: > >> On 10/30/2017 4:17 PM, Thomas Köppe w

Re: [FFmpeg-devel] On in-tree external headers

2017-10-30 Thread Jan Ekstrom
On Mon, Oct 30, 2017 at 9:51 PM, Mark Thompson wrote: > PPS: > > The position stated above would imply removing the avisynth headers. Can > anyone who uses it comment on what would be required for that? Avisynth headers are available from the Avisynth SDK that comes with each installer of Avisy

Re: [FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-30 Thread Thomas Köppe
It should be under __GNUC__ -- something went wrong with the patch! I'll make a new one. Sorry about that! On 30 October 2017 at 19:31, James Almer wrote: > On 10/30/2017 4:17 PM, Thomas Köppe wrote: > > Variables used in inline assembly need to be marked with > attribute((used)). > > Static con

[FFmpeg-devel] On in-tree external headers

2017-10-30 Thread Mark Thompson
Hi all, The recent submission of AMD AMF patches including a builtin header prompted me to think further about what external API headers should actually be included in the tree. For the AMD headers (like V4L2 previously), it seems entirely silly to include them - they are available upstream i

Re: [FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-30 Thread James Almer
On 10/30/2017 4:17 PM, Thomas Köppe wrote: > Variables used in inline assembly need to be marked with attribute((used)). > Static constants already were, via the define of DECLARE_ASM_CONST. > But DECLARE_ALIGNED does not add this attribute, and some of the variables > defined with it are const onl

[FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-30 Thread Thomas Köppe
Variables used in inline assembly need to be marked with attribute((used)). Static constants already were, via the define of DECLARE_ASM_CONST. But DECLARE_ALIGNED does not add this attribute, and some of the variables defined with it are const only used in inline assembly, and therefore appeared d

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-10-30 Thread Mark Thompson
On 30/10/17 17:56, mmironov wrote: > From 9337cb69176bc15aaaf74186cb3468f106236f04 Mon Sep 17 00:00:00 2001 > From: mmironov > Date: Fri, 27 Oct 2017 13:03:15 -0400 > Subject: [PATCH] Added: HW accelerated H.264 and HEVC encoding for AMD GPUs > based on AMF SDK > > Signed-off-by: mmironov > ---

Re: [FFmpeg-devel] fix minor leak in id3v2 parsing

2017-10-30 Thread Fredrik Hubinette
Sorry, it was an old patch that I had delayed to submit for various reasons. Glad it's fixed. /Hubbe On Thu, Oct 26, 2017 at 9:57 AM, James Almer wrote: > On 10/24/2017 8:09 PM, Moritz Barsnick wrote: > > On Tue, Oct 24, 2017 at 19:38:58 -0300, James Almer wrote: > >>> Subject: [PATCH] Fi

Re: [FFmpeg-devel] [PATCH 1/8] configure: test whether x86 assembler supports AVX-512

2017-10-30 Thread James Darnley
On 2017-10-30 18:40, James Almer wrote: > On 10/30/2017 10:08 AM, James Darnley wrote: >> @@ -5669,6 +5672,7 @@ EOF >> elf*) enabled debug && append X86ASMFLAGS $x86asm_debug ;; >> esac >> >> +check_x86asm "vmovdqa32 [eax]{k1}{z}, zmm0"|| disable >> avx512_exter

Re: [FFmpeg-devel] [PATCH 1/8] configure: test whether x86 assembler supports AVX-512

2017-10-30 Thread James Darnley
On 2017-10-30 17:53, Hendrik Leppkes wrote: > On Mon, Oct 30, 2017 at 2:08 PM, James Darnley wrote: >> --- >> configure | 5 + >> 1 file changed, 5 insertions(+) >> >> diff --git a/configure b/configure >> index 2f259996ae..e172fe8571 100755 >> --- a/configure >> +++ b/configure >> @@ -406,6

Re: [FFmpeg-devel] [PATCH 8/8] avcodec/v210enc: add AVX-512 10-bit line pack function

2017-10-30 Thread James Darnley
On 2017-10-30 17:14, Henrik Gramner wrote: > On Mon, Oct 30, 2017 at 2:08 PM, James Darnley wrote: >> +INIT_YMM avx512 > > ymm? Yes. I haven't written a correct one using zmm regs yet. I will ask some questions about it, possibly very soon. If I don't get a longer version to work, I could jus

Re: [FFmpeg-devel] order T-shirts

2017-10-30 Thread Thilo Borgmann
Am 30.10.17 um 03:52 schrieb Steven Liu: > 2017-10-30 10:12 GMT+08:00 Lou Logan : >> On Sat, Oct 28, 2017, at 03:41 AM, Thilo Borgmann wrote: >>> Both designs come for 12€ each! Lou's design doesn't have a printing on >>> the neck. I'd choosen the green for our logo by their color chart - it >>> lo

Re: [FFmpeg-devel] [PATCH] avcodec/avcodec.h: remove doxy from the old bsf API functions

2017-10-30 Thread James Almer
On 10/30/2017 12:20 PM, Derek Buitenhuis wrote: > On 10/29/2017 7:20 PM, James Almer wrote: >> Make it clear that these are deprecated and the new API should be >> used instead. >> >> As a side effect, this slightly reduces differences with libav. >> >> Signed-off-by: James Almer >> --- >> libavc

Re: [FFmpeg-devel] [PATCH 1/8] configure: test whether x86 assembler supports AVX-512

2017-10-30 Thread James Almer
On 10/30/2017 10:08 AM, James Darnley wrote: > --- > configure | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/configure b/configure > index 2f259996ae..e172fe8571 100755 > --- a/configure > +++ b/configure > @@ -406,6 +406,7 @@ Optimization options (experts only): >--disable-fm

Re: [FFmpeg-devel] [PATCH 1/8] configure: test whether x86 assembler supports AVX-512

2017-10-30 Thread Hendrik Leppkes
On Mon, Oct 30, 2017 at 2:08 PM, James Darnley wrote: > --- > configure | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/configure b/configure > index 2f259996ae..e172fe8571 100755 > --- a/configure > +++ b/configure > @@ -406,6 +406,7 @@ Optimization options (experts only): >--d

Re: [FFmpeg-devel] [PATCH 8/8] avcodec/v210enc: add AVX-512 10-bit line pack function

2017-10-30 Thread Henrik Gramner
On Mon, Oct 30, 2017 at 2:08 PM, James Darnley wrote: > +INIT_YMM avx512 ymm? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/avcodec.h: remove doxy from the old bsf API functions

2017-10-30 Thread Derek Buitenhuis
On 10/29/2017 7:20 PM, James Almer wrote: > Make it clear that these are deprecated and the new API should be > used instead. > > As a side effect, this slightly reduces differences with libav. > > Signed-off-by: James Almer > --- > libavcodec/avcodec.h | 70 > +++--

[FFmpeg-devel] [PATCH 7/8] checkasm: support for AVX-512 functions

2017-10-30 Thread James Darnley
--- tests/checkasm/checkasm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c index b8b0e32dbd..9fb1438bdb 100644 --- a/tests/checkasm/checkasm.c +++ b/tests/checkasm/checkasm.c @@ -192,6 +192,7 @@ static const struct { { "FMA3", "

[FFmpeg-devel] [PATCH 1/8] configure: test whether x86 assembler supports AVX-512

2017-10-30 Thread James Darnley
--- configure | 5 + 1 file changed, 5 insertions(+) diff --git a/configure b/configure index 2f259996ae..e172fe8571 100755 --- a/configure +++ b/configure @@ -406,6 +406,7 @@ Optimization options (experts only): --disable-fma3 disable FMA3 optimizations --disable-fma4

[FFmpeg-devel] [PATCH 6/8] x86inc: reduce difference to x264 upstream

2017-10-30 Thread James Darnley
These changes were commited to x264 in b568a256 "Experimental nasm support" --- libavutil/x86/x86inc.asm | 16 ++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index f3dd2b788a..10e4603a65 100644 --- a/libavutil/x86

[FFmpeg-devel] [PATCH 4/8] avutil: add alignment needed for AVX-512

2017-10-30 Thread James Darnley
--- libavutil/x86/cpu.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavutil/x86/cpu.c b/libavutil/x86/cpu.c index b22a950b79..c56f2a8754 100644 --- a/libavutil/x86/cpu.c +++ b/libavutil/x86/cpu.c @@ -247,6 +247,8 @@ size_t ff_get_cpu_max_align_x86(void) { int flags = av_get_cpu_f

[FFmpeg-devel] [PATCH 8/8] avcodec/v210enc: add AVX-512 10-bit line pack function

2017-10-30 Thread James Darnley
--- libavcodec/x86/v210enc.asm| 5 + libavcodec/x86/v210enc_init.c | 7 +++ 2 files changed, 12 insertions(+) diff --git a/libavcodec/x86/v210enc.asm b/libavcodec/x86/v210enc.asm index 965f2bea3c..5068af27f8 100644 --- a/libavcodec/x86/v210enc.asm +++ b/libavcodec/x86/v210enc.asm @@ -

[FFmpeg-devel] [PATCH 3/8] avutil: detect when AVX-512 is available

2017-10-30 Thread James Darnley
--- libavutil/x86/cpu.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavutil/x86/cpu.c b/libavutil/x86/cpu.c index f33088c8c7..b22a950b79 100644 --- a/libavutil/x86/cpu.c +++ b/libavutil/x86/cpu.c @@ -143,6 +143,7 @@ int ff_get_cpu_flags_x86(void) #endif /* HAVE_SSE */ }

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/aacdec_fixed: Fix integer overflow in predict()

2017-10-30 Thread Michael Niedermayer
On Fri, Oct 27, 2017 at 02:23:20AM +0200, Michael Niedermayer wrote: > Fixes: runtime error: signed integer overflow: -2110708110 + -82837504 cannot > be represented in type 'int' > Fixes: 3547/clusterfuzz-testcase-minimized-6009386439802880 > > Found-by: continuous fuzzing process > https://git

[FFmpeg-devel] [PATCH 5/8] x86inc: AVX-512 support

2017-10-30 Thread James Darnley
From: Henrik Gramner AVX-512 consists of a plethora of different extensions, but in order to keep things a bit more manageable we group together the following extensions under a single baseline cpu flag which should cover SKL-X and future CPUs: * AVX-512 Foundation (F) * AVX-512 Conflict Detect

[FFmpeg-devel] [PATCH 2/8] avutil: add AVX-512 flags

2017-10-30 Thread James Darnley
--- libavutil/cpu.c | 6 +- libavutil/cpu.h | 1 + libavutil/tests/cpu.c | 1 + libavutil/x86/cpu.h | 2 ++ 4 files changed, 9 insertions(+), 1 deletion(-) diff --git a/libavutil/cpu.c b/libavutil/cpu.c index c8401b8258..6548cc3042 100644 --- a/libavutil/cpu.c +++ b/libavutil/cp

[FFmpeg-devel] [PATCH 0/8] AVX-512 support (v.1)

2017-10-30 Thread James Darnley
This patch set adds support for AVX-512 functions to be written. While not immediately useful this does let people start writing them. Presently it lumps a "more manageable" set of sub-features into the overall AVX-512 flag. Just to be clear: current processors severly limit the performace when

Re: [FFmpeg-devel] [PATCH] avutil/crc: use ff_thread_once at av_crc_get_table

2017-10-30 Thread Michael Niedermayer
On Mon, Oct 30, 2017 at 02:14:35PM +0700, Muhammad Faiz wrote: > On Tue, Oct 24, 2017 at 4:31 PM, Muhammad Faiz wrote: > > Fix tsan warnings. > > > > Signed-off-by: Muhammad Faiz > > --- > > libavutil/crc.c | 49 + > > 1 file changed, 29 insertions

Re: [FFmpeg-devel] [RFC]lswr/rematrix: Support s32p

2017-10-30 Thread Michael Niedermayer
On Mon, Oct 30, 2017 at 03:47:13PM +0700, Muhammad Faiz wrote: > On Sun, Oct 29, 2017 at 3:55 AM, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch fixes a random testcase for ticket #6785 here but I > > don't know if this is the correct fix. > > > > Please review, Carl Eugen > > > > From a93

Re: [FFmpeg-devel] [PATCH 1/2] avfilter: pass correct argument to helper function

2017-10-30 Thread Nicolas George
Le sextidi 6 brumaire, an CCXXVI, Paul B Mahol a écrit : > Signed-off-by: Paul B Mahol > --- > libavfilter/avfilter.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c > index bc9e60bb85..b98b32bacb 100644 > --- a/libavfilter

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve avc avg mc 22, 11, 31, 13 and 33 msa functions

2017-10-30 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Friday, October 27, 2017 5:01 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve avc avg mc 22, 11, 31, 13

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve avc chroma avg hv mc msa functions

2017-10-30 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Friday, October 27, 2017 5:03 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve avc chroma avg hv mc msa

Re: [FFmpeg-devel] [PATCH v4 2/2] rtp: rfc4175: add handler for YCbCr-4:2:2

2017-10-30 Thread Éloi Bail
> I think you misunderstood "unpublished" for some other word. Those > specs are actually available for anyone that wants to read them (if > they are willing to pay for them). 2110-20 isn't available on any of > the usual sources for obtaining SMPTE standards documents. > - Hendrik Hi Hendrik.

Re: [FFmpeg-devel] libavfilter/vsrc_testsrc : fix hang with testsrc2 and small size

2017-10-30 Thread Nicolas George
Le septidi 7 brumaire, an CCXXVI, Martin Vignali a écrit : > Not sure i understand, can you explain a little bit more ? This was a quick guess make sure the values are positive before manipulating them instead of making far-reaching changes. As it happens, there was an even simpler fix, see the pa

[FFmpeg-devel] [PATCH] lavfi/testsrc2: fix hang with very small sizes.

2017-10-30 Thread Nicolas George
Move a subtraction to the other side of the equal to avoid overflows. Signed-off-by: Nicolas George --- libavfilter/vsrc_testsrc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vsrc_testsrc.c b/libavfilter/vsrc_testsrc.c index fe0d50aa41..a790974d14 100644 -

[FFmpeg-devel] [PATCH] avdevice: remove dangling parenthesis from 2245476e5c45

2017-10-30 Thread Ricardo Constantino
--- libavdevice/decklink_dec.cpp| 2 +- libavdevice/libndi_newtek_dec.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/decklink_dec.cpp index e786dd85dc..191547ff10 100644 --- a/libavdevice/decklink_dec.cpp +++ b/libavdevice/de

Re: [FFmpeg-devel] [RFC]lswr/rematrix: Support s32p

2017-10-30 Thread Muhammad Faiz
On Sun, Oct 29, 2017 at 3:55 AM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes a random testcase for ticket #6785 here but I > don't know if this is the correct fix. > > Please review, Carl Eugen > > From a93b9309d74f5eadece371ee1e682d266af6cd83 Mon Sep 17 00:00:00 2001 > From: Carl Eugen

Re: [FFmpeg-devel] [PATCH] avutil/crc: use ff_thread_once at av_crc_get_table

2017-10-30 Thread Muhammad Faiz
On Tue, Oct 24, 2017 at 4:31 PM, Muhammad Faiz wrote: > Fix tsan warnings. > > Signed-off-by: Muhammad Faiz > --- > libavutil/crc.c | 49 + > 1 file changed, 29 insertions(+), 20 deletions(-) Ping. My recent benchmark: old/new: 296320/ 340400 d