Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Moritz Barsnick
On Tue, Oct 06, 2015 at 22:52:28 -0400, Ganesh Ajjanagadde wrote: > ping: the reason I persist in this is because (long-term) the only 2 > compilers where we can reasonably reach a near "warning is regression" > state and benefit from it are clang and gcc. BTW, are the "remaining" (meaning: more

[FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-07 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/vf_w3fdif.c | 139 +++ libavfilter/w3fdif.h | 61 +++ libavfilter/x86/Makefile | 2 + libavfilter/x86/vf_w3fdif.asm| 230 +++

Re: [FFmpeg-devel] [PATCHv2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 3:34 PM, Michael Niedermayer wrote: > On Wed, Oct 07, 2015 at 12:29:35PM -0400, Ganesh Ajjanagadde wrote: >> Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. >> The concept of av_uninit is inherently useful though. This

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Moritz Barsnick
On Wed, Oct 07, 2015 at 17:47:44 -0400, Ganesh Ajjanagadde wrote: > Since I am currently in the warning suppression business, I can help > out at least in some basic checking. Unfortunately, I believe the icc > is not freely licensed, which means I can't test this stuff. I'll get in touch "when I

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Moritz Barsnick
On Wed, Oct 07, 2015 at 16:07:53 -0400, Ronald S. Bultje wrote: > > BTW, are the "remaining" (meaning: more than from gcc) warnings emitted > > by the Intel C(++) Compiler redundant or false? The icc configuration > > should then probably silence them. > > I doubt anyone has ever investigated

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 5:35 PM, Moritz Barsnick wrote: > On Wed, Oct 07, 2015 at 16:07:53 -0400, Ronald S. Bultje wrote: >> > BTW, are the "remaining" (meaning: more than from gcc) warnings emitted >> > by the Intel C(++) Compiler redundant or false? The icc configuration >> >

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-07 Thread James Almer
On 10/7/2015 5:27 PM, Paul B Mahol wrote: > +cglobal w3fdif_simple_high, 5, 10, 8, 0, work_line, in_lines_cur0, > in_lines_adj0, coef, linesize [...] > +cglobal w3fdif_complex_high, 5, 14, 8, 0, work_line, in_lines_cur0, > in_lines_adj0, coef, linesize All the values in coeff_hf fit in words,

Re: [FFmpeg-devel] [PATCH 1/3] dnxhddec: better support for 4:4:4

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 08:08:39PM +0200, Christophe Gisquet wrote: > 2015-10-07 10:32 GMT+02:00 Christophe Gisquet : > >> this would allow ctx->bit_depth to be set but without initializing the > >> dsp contexts > >> subsequent runs would also skip init due to > >> "

Re: [FFmpeg-devel] [PATCH 2/3] dnxhddata: introduce and use MBAFF flag

2015-10-07 Thread Michael Niedermayer
On Mon, Oct 05, 2015 at 08:44:45PM +0200, Christophe Gisquet wrote: > MBAFF-like handling of interlaced content in CID 1260 is different from > the other CIDs, and in particular doesn't use the same syntax. > --- > libavcodec/dnxhddata.c | 2 +- > libavcodec/dnxhddata.h | 3 ++- > 2 files

[FFmpeg-devel] [PATCH] vp9: add 10/12bpp sse2 SIMD versions of iadst8x8.

2015-10-07 Thread Ronald S. Bultje
--- libavcodec/x86/vp9dsp_init_16bpp_template.c | 4 +- libavcodec/x86/vp9itxfm_16bpp.asm | 211 ++-- 2 files changed, 202 insertions(+), 13 deletions(-) diff --git a/libavcodec/x86/vp9dsp_init_16bpp_template.c b/libavcodec/x86/vp9dsp_init_16bpp_template.c

Re: [FFmpeg-devel] [PATCH 3/3 v3] h264: Run VLC init under pthread_once

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 12:28:51PM -0400, Derek Buitenhuis wrote: > This makes the h.264 decoder threadsafe to initialize. can you explain why it is not threadsafe currently ? (assuming "done = 1" would be moved to the end of ff_h264_decode_init_vlc()) [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCHv3] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. The concept of av_uninit is inherently useful though. This patch silences a bunch of warnings on clang e.g http://fate.ffmpeg.org/log.cgi?time=20150918181527=compile=x86_64-darwin-clang-polly-vectorize-stripmine-3.7.

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ronald S. Bultje
Hi, On Wed, Oct 7, 2015 at 3:46 PM, Moritz Barsnick wrote: > On Tue, Oct 06, 2015 at 22:52:28 -0400, Ganesh Ajjanagadde wrote: > > ping: the reason I persist in this is because (long-term) the only 2 > > compilers where we can reasonably reach a near "warning is regression" >

Re: [FFmpeg-devel] [PATCH 3/3 v3] h264: Run VLC init under pthread_once

2015-10-07 Thread Hendrik Leppkes
On Wed, Oct 7, 2015 at 10:18 PM, Michael Niedermayer wrote: > On Wed, Oct 07, 2015 at 12:28:51PM -0400, Derek Buitenhuis wrote: >> This makes the h.264 decoder threadsafe to initialize. > > can you explain why it is not threadsafe currently ? > (assuming "done = 1" would

Re: [FFmpeg-devel] [PATCH 3/3] dnxhdenc: mark CID 1260 encoder experimental

2015-10-07 Thread Michael Niedermayer
On Mon, Oct 05, 2015 at 08:44:46PM +0200, Christophe Gisquet wrote: > The MBAFF handling recently introduced on the decoder side shows that > the encoder does not support it correctly. Therefore, make the related > profile experimental. > > Furthermore, current encoder logic treats it as unable

Re: [FFmpeg-devel] [PATCH 1/3] dnxhddec: better support for 4:4:4

2015-10-07 Thread Christophe Gisquet
Hi, > this would allow ctx->bit_depth to be set but without initializing the > dsp contexts > subsequent runs would also skip init due to > " if (ctx->bit_depth != old_bit_depth) {" Actually, other changes could trigger the issue, like 4:2:2 10 bits -> 4:4:4 8 bits (unknown, unsupported) ->

[FFmpeg-devel] [PATCH] avfilter/delogo: Set default band to 1

2015-10-07 Thread Jean Delvare
The original interpolation algorithm behaved poorly on the borders and did not even guarantee continuity at the borders. For this reason, a second interpolation/blending pass was required on the borders to make them seamless. However, since the interpolation algorithm was improved in June 2013,

Re: [FFmpeg-devel] [PATCHv3] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 8:21 PM, Michael Niedermayer wrote: > On Wed, Oct 07, 2015 at 06:22:08PM -0400, Ganesh Ajjanagadde wrote: >> Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. >> The concept of av_uninit is inherently useful though. This

Re: [FFmpeg-devel] [PATCH 3/3 v3] h264: Run VLC init under pthread_once

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 10:46:50PM +0200, Hendrik Leppkes wrote: > On Wed, Oct 7, 2015 at 10:18 PM, Michael Niedermayer > wrote: > > On Wed, Oct 07, 2015 at 12:28:51PM -0400, Derek Buitenhuis wrote: > >> This makes the h.264 decoder threadsafe to initialize. > > > > can

Re: [FFmpeg-devel] [PATCHv3] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ronald S. Bultje
Hi, On Wed, Oct 7, 2015 at 6:22 PM, Ganesh Ajjanagadde wrote: > Commit 6dac8c8327 disabled av_uninit for clang, due to some useless > warnings. > The concept of av_uninit is inherently useful though. This patch silences a > bunch of warnings on clang e.g > >

Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration

2015-10-07 Thread highgod0401
From: Timo Rothenpieler Date: 2015-09-30 19:27 To: FFmpeg development discussions and patches Subject: Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration >> ping once again >> >> Hi > Could you describe how to verify it, and how can I test it? > > Thanks > Best

[FFmpeg-devel] [PATCH] fix b frame n_quant_offset for nvenc

2015-10-07 Thread Agatha Hu
--- libavcodec/nvenc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index 18bcd96..0e6ef43 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -742,8 +742,6 @@ static av_cold int nvenc_encode_init(AVCodecContext *avctx)

[FFmpeg-devel] [PATCH 3/6] ffmpeg: use lavf API for applying bitstream filters

2015-10-07 Thread Rodger Combs
--- ffmpeg.c | 46 -- ffmpeg.h | 1 - ffmpeg_opt.c | 6 +- 3 files changed, 9 insertions(+), 44 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index e31a2c6..b9cde79 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -682,47 +682,10 @@ static void

[FFmpeg-devel] [PATCH 5/6] lavf: add API to append a bsf to a stream's list

2015-10-07 Thread Rodger Combs
--- libavformat/avformat.h | 8 libavformat/utils.c| 19 +++ 2 files changed, 27 insertions(+) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index f3c8260..20759e3 100644 --- a/libavformat/avformat.h +++ b/libavformat/avformat.h @@ -2758,6 +2758,14 @@

[FFmpeg-devel] [PATCH 2/6] lavf: add API to apply a list of bsfs to a packet

2015-10-07 Thread Rodger Combs
--- libavformat/avformat.h | 8 libavformat/utils.c| 48 2 files changed, 56 insertions(+) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index e2a27d4..5226b0a 100644 --- a/libavformat/avformat.h +++

Re: [FFmpeg-devel] [PATCH 0/9] Initial support for DNxHR, v2

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 08:11:21PM +0200, Christophe Gisquet wrote: > 2015-10-05 14:03 GMT+02:00 Christophe Gisquet : > > Hi, > > > > 2015-10-05 13:18 GMT+02:00 Michael Niedermayer : > >> do you have any testcases you can share for these ? > >

Re: [FFmpeg-devel] [PATCHv3] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 06:22:08PM -0400, Ganesh Ajjanagadde wrote: > Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. > The concept of av_uninit is inherently useful though. This patch silences a > bunch of warnings on clang e.g >

Re: [FFmpeg-devel] [PATCHv3] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 8:44 PM, Ronald S. Bultje wrote: > Hi, > > On Wed, Oct 7, 2015 at 6:22 PM, Ganesh Ajjanagadde > wrote: >> >> Commit 6dac8c8327 disabled av_uninit for clang, due to some useless >> warnings. >> The concept of av_uninit is

[FFmpeg-devel] [PATCH] doc/faq: use https instead of http

2015-10-07 Thread Ganesh Ajjanagadde
Change to https for FFmpeg websites. Signed-off-by: Ganesh Ajjanagadde --- doc/faq.texi | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/doc/faq.texi b/doc/faq.texi index 5fe716b..ef111c7 100644 --- a/doc/faq.texi +++ b/doc/faq.texi

[FFmpeg-devel] [PATCH 1/6] lavc: move bitstream filter args to the bsf ctx

2015-10-07 Thread Rodger Combs
--- libavcodec/avcodec.h | 1 + libavcodec/bitstream_filter.c | 5 +++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index ff70d25..ce42e57 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -5026,6 +5026,7 @@

[FFmpeg-devel] [PATCH 6/6] lavf/matroska: add automatic bitstream filtering

2015-10-07 Thread Rodger Combs
--- libavformat/matroskaenc.c | 43 +++ 1 file changed, 35 insertions(+), 8 deletions(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 1fb39fe..81e859f 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@

[FFmpeg-devel] [PATCH 4/6] lavf: add automatic bitstream filtering

2015-10-07 Thread Rodger Combs
This solves the problem discussed in https://ffmpeg.org/pipermail/ffmpeg-devel/2015-September/179238.html by allowing AVCodec::write_header to be delayed until after packets have been run through required bitstream filters in order to generate global extradata. It also provides a mechanism by

Re: [FFmpeg-devel] [PATCH 0/9] Initial support for DNxHR, v2

2015-10-07 Thread Christophe Gisquet
Hi, 2015-10-08 1:38 GMT+02:00 Michael Niedermayer : >> So I have made a fate test on 1 frame for each sequence (total around 1.8MB). >> >> Is it ok? > > ok Here you go. Hendrik has kindly volunteered to upload the 2 samples for me. -- Christophe From

Re: [FFmpeg-devel] [PATCHv2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 12:29:35PM -0400, Ganesh Ajjanagadde wrote: > Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. > The concept of av_uninit is inherently useful though. This patch silences a > bunch of warnings on clang e.g >

Re: [FFmpeg-devel] [PATCHv2] avcodec/libx264: silence -Waddress

2015-10-07 Thread Ronald S. Bultje
Hi, On Tue, Oct 6, 2015 at 10:48 PM, Ganesh Ajjanagadde wrote: > This patch moves the pointer validity check outside the macro, > and silences the -Waddress observed with GCC 5.2. > > Note that this changes the error message slightly, from: > "bad option..." to "Error

Re: [FFmpeg-devel] [PATCH] RFC: Automatic bitstream filtering

2015-10-07 Thread Michael Niedermayer
On Tue, Oct 06, 2015 at 08:07:08PM -0500, Rodger Combs wrote: > This solves the problem discussed in > https://ffmpeg.org/pipermail/ffmpeg-devel/2015-September/179238.html > by allowing AVCodec::write_header to be delayed until after packets have been > run through required bitstream filters in

Re: [FFmpeg-devel] [PATCH] RFC: Automatic bitstream filtering

2015-10-07 Thread wm4
On Tue, 6 Oct 2015 20:07:08 -0500 Rodger Combs wrote: > This solves the problem discussed in > https://ffmpeg.org/pipermail/ffmpeg-devel/2015-September/179238.html > by allowing AVCodec::write_header to be delayed until after packets have been > run through required

Re: [FFmpeg-devel] [PATCH] avfilter/delogo: Set default band to 1

2015-10-07 Thread Jean Delvare
Hi Stefano, On Wed, 7 Oct 2015 11:21:45 +0200, Stefano Sabatini wrote: > On date Wednesday 2015-10-07 08:54:59 +0200, Jean Delvare encoded: > > The original interpolation algorithm behaved poorly on the borders and > > did not even guarantee continuity at the borders. For this reason, a > >

Re: [FFmpeg-devel] [PATCH] avfilter/delogo: Set default band to 1

2015-10-07 Thread Stefano Sabatini
On date Wednesday 2015-10-07 08:54:59 +0200, Jean Delvare encoded: > The original interpolation algorithm behaved poorly on the borders and > did not even guarantee continuity at the borders. For this reason, a > second interpolation/blending pass was required on the borders to make > them

Re: [FFmpeg-devel] [PATCH] avfilter/x86/vf_blend.asm: add hardmix and phoenix sse2 SIMD

2015-10-07 Thread Ronald S. Bultje
Hi, On Wed, Oct 7, 2015 at 5:38 AM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/x86/vf_blend.asm| 62 > + > libavfilter/x86/vf_blend_init.c | 14 ++ > 2 files changed, 76

[FFmpeg-devel] [PATCH] avfilter/x86/vf_blend.asm: add hardmix and phoenix sse2 SIMD

2015-10-07 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/x86/vf_blend.asm| 62 + libavfilter/x86/vf_blend_init.c | 14 ++ 2 files changed, 76 insertions(+) diff --git a/libavfilter/x86/vf_blend.asm b/libavfilter/x86/vf_blend.asm index

Re: [FFmpeg-devel] [PATCH] doc/developer: use https instead of http

2015-10-07 Thread Michael Niedermayer
On Tue, Oct 06, 2015 at 11:15:49PM -0400, Ganesh Ajjanagadde wrote: > Change to https for FFmpeg websites. > > Signed-off-by: Ganesh Ajjanagadde > --- > doc/developer.texi | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) applied thanks [...] --

Re: [FFmpeg-devel] [PATCH v2 0/8] libkvazaar improvements

2015-10-07 Thread Arttu Ylä-Outinen
On 2015-09-29 16:29, Arttu Ylä-Outinen wrote: These patches fix some problems in libkvazaar encoder and update it to work with the latest git version of Kvazaar. Most notable changes are setting pts, dts and keyframe flag on the output packets and fixing the calculation of the framerate. v2:

[FFmpeg-devel] [PATCH 1/2] avcodec/mips: build fix for MSA

2015-10-07 Thread shivraj.patil
From: Shivraj Patil Modified sps and pps access from old HEVCContext(s) structure to newly introduced HEVCParamSets(ps) Signed-off-by: Shivraj Patil --- libavcodec/mips/hevcpred_msa.c | 282 1 file

Re: [FFmpeg-devel] [PATCHv2] avfilter/buffersrc: add av_warn_unused_result attributes

2015-10-07 Thread Clément Bœsch
On Wed, Oct 07, 2015 at 09:31:49AM -0400, Ganesh Ajjanagadde wrote: > On Wed, Oct 7, 2015 at 8:59 AM, Clément Bœsch wrote: > > On Tue, Oct 06, 2015 at 06:53:47PM -0400, Ganesh Ajjanagadde wrote: > >> This adds av_warn_unused_result whenever it is relevant. > >> > >> Signed-off-by:

Re: [FFmpeg-devel] [PATCH] avfilter/delogo: Set default band to 1

2015-10-07 Thread Stefano Sabatini
On date Wednesday 2015-10-07 13:57:46 +0200, Jean Delvare encoded: > Hi Stefano, > > On Wed, 7 Oct 2015 11:21:45 +0200, Stefano Sabatini wrote: > > On date Wednesday 2015-10-07 08:54:59 +0200, Jean Delvare encoded: > > > The original interpolation algorithm behaved poorly on the borders and > > >

Re: [FFmpeg-devel] [PATCH] avfilter/delogo: Set default band to 1

2015-10-07 Thread Jean Delvare
On Wed, 7 Oct 2015 14:19:58 +0200, Stefano Sabatini wrote: > > > LGTM. BTW, if you want to drop the band option, you could ifdef it so > > > that it will be dropt at the next lavfi major bump. > > > > Good idea. Something like this? > > > > --- > > libavfilter/vf_delogo.c |6 ++ > > 1

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Michael Niedermayer
On Sat, Sep 19, 2015 at 01:00:17AM -0400, Ganesh Ajjanagadde wrote: > Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. > The concept of av_uninit is inherently useful though. This patch silences a > bunch of warnings on clang e.g >

[FFmpeg-devel] [PATCH v2] avfilter/delogo: Set default band to 1

2015-10-07 Thread Jean Delvare
The original interpolation algorithm behaved poorly on the borders and did not even guarantee continuity at the borders. For this reason, a second interpolation/blending pass was required on the borders to make them seamless. However, since the interpolation algorithm was improved in June 2013,

Re: [FFmpeg-devel] [PATCHv2] avfilter/buffersrc: add av_warn_unused_result attributes

2015-10-07 Thread Clément Bœsch
On Tue, Oct 06, 2015 at 06:53:47PM -0400, Ganesh Ajjanagadde wrote: > This adds av_warn_unused_result whenever it is relevant. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/buffersrc.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 8:56 AM, Michael Niedermayer wrote: > On Sat, Sep 19, 2015 at 01:00:17AM -0400, Ganesh Ajjanagadde wrote: >> Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. >> The concept of av_uninit is inherently useful though. This

Re: [FFmpeg-devel] [PATCH] avcodec/alac: also use a temp buffer for 24bit samples

2015-10-07 Thread wm4
On Tue, 6 Oct 2015 20:13:58 -0300 James Almer wrote: > On 10/6/2015 5:33 PM, Paul B Mahol wrote: > > you set aligned number of samples before calling get_buffer and after > > that changes > > frame->nb_samples to actual number of samples. > > > > Alternatively IIRC default 16

[FFmpeg-devel] [PATCH 2/2] avcodec/mips: build fix for MSA 64bit

2015-10-07 Thread shivraj.patil
From: Shivraj Patil Modified datatype of function argument (pitch from int32_t to ptrdiff_t) Signed-off-by: Shivraj Patil --- libavcodec/mips/vp9_lpf_msa.c | 42 - 1 file changed, 21 insertions(+),

Re: [FFmpeg-devel] [PATCH v2 0/8] libkvazaar improvements

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 01:33:58PM +0300, Arttu Ylä-Outinen wrote: > On 2015-09-29 16:29, Arttu Ylä-Outinen wrote: > >These patches fix some problems in libkvazaar encoder and update it to work > >with the latest git version of Kvazaar. Most notable changes are setting > >pts, dts and keyframe

Re: [FFmpeg-devel] [PATCHv2] avcodec/libx264: silence -Waddress

2015-10-07 Thread Ronald S. Bultje
Hi, On Tue, Oct 6, 2015 at 10:48 PM, Ganesh Ajjanagadde wrote: > This patch moves the pointer validity check outside the macro, > and silences the -Waddress observed with GCC 5.2. > > Note that this changes the error message slightly, from: > "bad option..." to "Error

Re: [FFmpeg-devel] [PATCHv2] avfilter/buffersrc: add av_warn_unused_result attributes

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 8:59 AM, Clément Bœsch wrote: > On Tue, Oct 06, 2015 at 06:53:47PM -0400, Ganesh Ajjanagadde wrote: >> This adds av_warn_unused_result whenever it is relevant. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/buffersrc.h |

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/mips: build fix for MSA 64bit

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 06:21:59PM +0530, shivraj.pa...@imgtec.com wrote: > From: Shivraj Patil > > Modified datatype of function argument (pitch from int32_t to ptrdiff_t) > > Signed-off-by: Shivraj Patil > --- >

[FFmpeg-devel] [PATCH] libavcodec/qsv.c: Re-design session control and internal allocation

2015-10-07 Thread Ivan Uskov
Hello All, The attached patch represents new design for qsv session control and internal allocation. All qsv modules now uses instance of AVQSVContext so now session allocates by external application and session allocates internally by ffmpeg itself handles by unified way. For the case of

Re: [FFmpeg-devel] [PATCH] vp9: add 12bpp sse2 versions of iadst4.

2015-10-07 Thread Henrik Gramner
On Wed, Oct 7, 2015 at 3:59 AM, Ronald S. Bultje wrote: > diff --git a/libavcodec/x86/vp9itxfm_16bpp.asm > b/libavcodec/x86/vp9itxfm_16bpp.asm > +%macro IADST4_12BPP_1D 0 > +pandm4, m0, [pd_3fff] > +pandm5, m1, [pd_3fff] > +psrad

Re: [FFmpeg-devel] [PATCH] libavcodec/qsv.c: Re-design session control and internal allocation

2015-10-07 Thread Hendrik Leppkes
On Wed, Oct 7, 2015 at 4:41 PM, Ivan Uskov wrote: > Hello All, > > The attached patch represents new design for qsv session control and internal > allocation. All qsv modules now uses instance of AVQSVContext so now session > allocates by external application and session

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mips: build fix for MSA

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 06:20:53PM +0530, shivraj.pa...@imgtec.com wrote: > From: Shivraj Patil > > Modified sps and pps access from old HEVCContext(s) structure to newly > introduced HEVCParamSets(ps) > > Signed-off-by: Shivraj Patil > ---

Re: [FFmpeg-devel] [PATCH 01/15] lavfi/af_aresample: remove looping on request_frame().

2015-10-07 Thread Nicolas George
Le tridi 13 vendémiaire, an CCXXIV, Michael Niedermayer a écrit : > whole patchset should be ok Thanks. Warnings fixed (sorry about that) and series pushed. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] w32pthreads: add pthread_once emulation

2015-10-07 Thread Hendrik Leppkes
On Wed, Oct 7, 2015 at 7:06 PM, Matt Oliver wrote: > On 8 October 2015 at 03:29, Hendrik Leppkes wrote: > >> On Wed, Oct 7, 2015 at 6:23 PM, Matt Oliver wrote: >> > On 6 October 2015 at 21:36, Hendrik Leppkes

[FFmpeg-devel] [PATCH 1/3] Revert "cabac: Allow hardcoding CABAC table."

2015-10-07 Thread Derek Buitenhuis
This becomes unuseful in the following commit. This reverts commit 092d1977cc7146f20c8db2155e7d648afb300de7. Signed-off-by: Derek Buitenhuis --- libavcodec/Makefile | 4 +- libavcodec/cabac.c | 74 -

Re: [FFmpeg-devel] [PATCH 3/3] h264: Run VLC init under pthread_once

2015-10-07 Thread wm4
On Wed, 7 Oct 2015 11:39:47 -0400 Derek Buitenhuis wrote: > This makes the h.264 decoder threadsafe to initialize. > > Signed-off-by: Derek Buitenhuis > --- > libavcodec/h264.c | 7 ++- > libavcodec/h264.h | 10 ++ > 2

Re: [FFmpeg-devel] [PATCH 3/3] h264: Run VLC init under pthread_once

2015-10-07 Thread Derek Buitenhuis
On 10/7/2015 5:05 PM, Matt Oliver wrote: > Couldnt all the above if/includes be simplified by just using a single > include libavutil/thread.h (which has all the above checks in it. So, I included "thread.h", which is from avcodec, and it failed. I think it's pretty non-obvious I need the

Re: [FFmpeg-devel] [PATCH] w32pthreads: add pthread_once emulation

2015-10-07 Thread Matt Oliver
On 6 October 2015 at 21:36, Hendrik Leppkes wrote: > The emulation uses native InitOnce* APIs on Windows Vista+, and a > lock-free/allocation-free approach using atomics and spinning for Windows > XP. > --- > > This is in preparation to use pthread_once for global static

[FFmpeg-devel] [PATCH 3/3 v2] h264: Run VLC init under pthread_once

2015-10-07 Thread Derek Buitenhuis
This makes the h.264 decoder threadsafe to initialize. Signed-off-by: Derek Buitenhuis --- libavcodec/h264.c | 9 - libavcodec/h264.h | 7 +++ 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index

[FFmpeg-devel] [PATCH 2/3] cabac: Make cabac starts hardcoded

2015-10-07 Thread Derek Buitenhuis
From: Anton Khirnov There's not much reason to generate such a small table at runtime. Signed-off-by: Derek Buitenhuis --- libavcodec/cabac.c | 207 ++- libavcodec/cabac.h | 3 +-

[FFmpeg-devel] [PATCH] vp9: add 10/12bpp sse2 SIMD for idct_idct_8x8.

2015-10-07 Thread Ronald S. Bultje
--- libavcodec/x86/vp9dsp_init_16bpp_template.c | 4 +- libavcodec/x86/vp9itxfm_16bpp.asm | 299 +--- 2 files changed, 234 insertions(+), 69 deletions(-) diff --git a/libavcodec/x86/vp9dsp_init_16bpp_template.c b/libavcodec/x86/vp9dsp_init_16bpp_template.c

Re: [FFmpeg-devel] [PATCH] winrt: multithreading support

2015-10-07 Thread Matt Oliver
On 2 October 2015 at 05:14, Hendrik Leppkes wrote: > On Thu, Oct 1, 2015 at 9:05 PM, wm4 wrote: > > On Fri, 2 Oct 2015 02:58:52 +0800 > > Wang Bin wrote: > > > >> From b8b5ad2d6510778111c2a03ae5cfbe727ee6 Mon Sep 17 00:00:00

Re: [FFmpeg-devel] [PATCH 3/3] h264: Run VLC init under pthread_once

2015-10-07 Thread Matt Oliver
On 8 October 2015 at 02:39, Derek Buitenhuis wrote: > This makes the h.264 decoder threadsafe to initialize. > > Signed-off-by: Derek Buitenhuis > --- > libavcodec/h264.c | 7 ++- > libavcodec/h264.h | 10 ++ > 2 files

Re: [FFmpeg-devel] [PATCH] libavcodec/qsv.c: Re-design session control and internal allocation

2015-10-07 Thread Ivan Uskov
Hello Hendrik, Wednesday, October 7, 2015, 5:58:25 PM, you wrote: HL> On Wed, Oct 7, 2015 at 4:41 PM, Ivan Uskov wrote: HL> Global static variables are not acceptable, sorry. HL> You'll have to find another way to solve your problem, but global HL> state is not the way

[FFmpeg-devel] [PATCH 0/3] h264: Threadsafe initialization

2015-10-07 Thread Derek Buitenhuis
Towards a glorious non-locking future. Obviously requires Hendrik's phread_once patch. Anton Khirnov (1): cabac: Make cabac starts hardcoded Derek Buitenhuis (2): Revert "cabac: Allow hardcoding CABAC table." h264: Run VLC init under pthread_once libavcodec/Makefile | 4 +-

Re: [FFmpeg-devel] [PATCH 3/3] h264: Run VLC init under pthread_once

2015-10-07 Thread Derek Buitenhuis
On 10/7/2015 4:39 PM, Derek Buitenhuis wrote: > +static pthread_once_t ff_h264_vlc_init = PTHREAD_ONCE_INIT; I've added av_unused here locally. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mips: build fix for MSA

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 06:20:53PM +0530, shivraj.pa...@imgtec.com wrote: > From: Shivraj Patil > > Modified sps and pps access from old HEVCContext(s) structure to newly > introduced HEVCParamSets(ps) > > Signed-off-by: Shivraj Patil > ---

Re: [FFmpeg-devel] [PATCH 1/3] dnxhddec: better support for 4:4:4

2015-10-07 Thread Christophe Gisquet
2015-10-07 10:32 GMT+02:00 Christophe Gisquet : >> this would allow ctx->bit_depth to be set but without initializing the >> dsp contexts >> subsequent runs would also skip init due to >> " if (ctx->bit_depth != old_bit_depth) {" > > Actually, other changes could

Re: [FFmpeg-devel] [PATCH 0/9] Initial support for DNxHR, v2

2015-10-07 Thread Christophe Gisquet
2015-10-05 14:03 GMT+02:00 Christophe Gisquet : > Hi, > > 2015-10-05 13:18 GMT+02:00 Michael Niedermayer : >> do you have any testcases you can share for these ? > > I was provided some but that I assume not redistributable. I asked the > same

Re: [FFmpeg-devel] [PATCHv2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 12:29 PM, Ganesh Ajjanagadde wrote: > Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. > The concept of av_uninit is inherently useful though. This patch silences a > bunch of warnings on clang e.g >

Re: [FFmpeg-devel] [PATCH 3/3 v2] h264: Run VLC init under pthread_once

2015-10-07 Thread wm4
On Wed, 7 Oct 2015 12:24:32 -0400 Derek Buitenhuis wrote: > This makes the h.264 decoder threadsafe to initialize. > > Signed-off-by: Derek Buitenhuis > --- > libavcodec/h264.c | 9 - > libavcodec/h264.h | 7 +++ > 2 files

Re: [FFmpeg-devel] [PATCH 3/3 v2] h264: Run VLC init under pthread_once

2015-10-07 Thread Derek Buitenhuis
On 10/7/2015 5:24 PM, Derek Buitenhuis wrote: > This makes the h.264 decoder threadsafe to initialize. > > Signed-off-by: Derek Buitenhuis > --- > libavcodec/h264.c | 9 - > libavcodec/h264.h | 7 +++ > 2 files changed, 15 insertions(+), 1 deletion(-)

Re: [FFmpeg-devel] [PATCH] w32pthreads: add pthread_once emulation

2015-10-07 Thread Matt Oliver
On 8 October 2015 at 03:29, Hendrik Leppkes wrote: > On Wed, Oct 7, 2015 at 6:23 PM, Matt Oliver wrote: > > On 6 October 2015 at 21:36, Hendrik Leppkes wrote: > > > >> The emulation uses native InitOnce* APIs on Windows Vista+,

Re: [FFmpeg-devel] [PATCH 3/3] h264: Run VLC init under pthread_once

2015-10-07 Thread Hendrik Leppkes
On Wed, Oct 7, 2015 at 6:10 PM, Derek Buitenhuis wrote: > On 10/7/2015 4:57 PM, wm4 wrote: >> A static variable in a header file? This doesn't look right. You should >> define it in h264.c instead. > > [17:03] <+wm4> Daemon404: what exactly is the point of declaring

[FFmpeg-devel] [PATCHv2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. The concept of av_uninit is inherently useful though. This patch silences a bunch of warnings on clang e.g http://fate.ffmpeg.org/log.cgi?time=20150918181527=compile=x86_64-darwin-clang-polly-vectorize-stripmine-3.7.

[FFmpeg-devel] [PATCH 3/3 v3] h264: Run VLC init under pthread_once

2015-10-07 Thread Derek Buitenhuis
This makes the h.264 decoder threadsafe to initialize. Signed-off-by: Derek Buitenhuis --- libavcodec/h264.c | 9 - libavcodec/h264.h | 1 + 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index

Re: [FFmpeg-devel] [PATCH] libavcodec/qsv.c: Re-design session control and internal allocation

2015-10-07 Thread Ivan Uskov
Hello wm4, Wednesday, October 7, 2015, 7:40:45 PM, you wrote: w> There's no automagic way to get this done. w> Hardware accelerators like vaapi and vdpau need the same thing. These w> have special APIs to set an API context on the decoder. Some APIs use w> hwaccel_context, vdpau uses