Re: [FFmpeg-devel] [PATCH] avutil/frame: Remove requirements to use accessors for AVFrame

2017-01-09 Thread wm4
On Mon, 9 Jan 2017 18:38:04 +0100 Michael Niedermayer wrote: > On Mon, Jan 09, 2017 at 10:27:09AM -0500, Ronald S. Bultje wrote: > > Hi, > > > > On Mon, Jan 9, 2017 at 8:48 AM, wm4 wrote: > > > > > On Mon, 9 Jan 2017 08:17:08 -0500 > > > "Ronald S. Bultje" wrote: > > > > > > > Hi, > > > >

Re: [FFmpeg-devel] [PATCH] lavc/cuvid: fail early if GPU can't handle given video parameters

2017-01-09 Thread wm4
On Mon, 9 Jan 2017 11:55:38 -0700 Pavel Koshevoy wrote: > On Mon, Jan 9, 2017 at 3:12 AM, wm4 wrote: > > On Mon, 2 Jan 2017 14:26:45 -0700 > > pkoshe...@gmail.com wrote: > > > >> From: Pavel Koshevoy > > >> +ret = convert_to_cuda_video_chroma_format(avctx->pix_fmt, > >> &probed_chrom

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: start_number new options

2017-01-09 Thread Steven Liu
2017-01-08 8:22 GMT+08:00 Steven Liu : > > > 2017-01-08 1:37 GMT+08:00 Bodecs Bela : > >> >> >> 2017.01.07. 0:32 keltezéssel, Steven Liu írta: >> >>> 2017-01-07 0:47 GMT+08:00 Bodecs Bela : >>> >>> 2017.01.06. 17:33 keltezéssel, Steven Liu írta: 2017-01-07 0:22 GMT+08:00 Bodecs Bela

Re: [FFmpeg-devel] [PATCH] mov: Fix spherical metadata_source field parsing.

2017-01-09 Thread Aaron Colwell
On Mon, Jan 9, 2017 at 7:27 PM James Almer wrote: > On 1/9/2017 11:47 PM, Aaron Colwell wrote: > > On Mon, Jan 9, 2017 at 6:00 PM James Almer wrote: > > > >> On 1/9/2017 3:47 PM, Aaron Colwell wrote: > >>> The attached patch fixes MOV spherical metadata parsing when the > >>> metadata_source fie

Re: [FFmpeg-devel] [PATCH] mov: Fix spherical metadata_source field parsing.

2017-01-09 Thread James Almer
On 1/9/2017 11:47 PM, Aaron Colwell wrote: > On Mon, Jan 9, 2017 at 6:00 PM James Almer wrote: > >> On 1/9/2017 3:47 PM, Aaron Colwell wrote: >>> The attached patch fixes MOV spherical metadata parsing when the >>> metadata_source field is not an empty string. >>> >>> The metadata_source field is

Re: [FFmpeg-devel] [PATCH] mov: Fix spherical metadata_source field parsing.

2017-01-09 Thread Aaron Colwell
On Mon, Jan 9, 2017 at 6:00 PM James Almer wrote: > On 1/9/2017 3:47 PM, Aaron Colwell wrote: > > The attached patch fixes MOV spherical metadata parsing when the > > metadata_source field is not an empty string. > > > > The metadata_source field is a null-terminated string, like other ISOBMFF >

Re: [FFmpeg-devel] [PATCH] mov: Fix spherical metadata_source field parsing.

2017-01-09 Thread James Almer
On 1/9/2017 3:47 PM, Aaron Colwell wrote: > The attached patch fixes MOV spherical metadata parsing when the > metadata_source field is not an empty string. > > The metadata_source field is a null-terminated string, like other ISOBMFF > strings, > not an 8-bit length followed by string characters.

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: fix hls start and tail segment duration problem

2017-01-09 Thread Steven Liu
2017-01-10 1:06 GMT+08:00 Steven Liu : > fix ticket: #6067 > > Signed-off-by: Steven Liu > --- > libavformat/hlsenc.c | 16 > 1 file changed, 12 insertions(+), 4 deletions(-) > > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c > index eeb450a..0fcb699 100644 > --- a/lib

Re: [FFmpeg-devel] [PATCH 1/3] avformat/avienc: write chained master index only if std_compliance is normal or below

2017-01-09 Thread Michael Niedermayer
On Mon, Jan 09, 2017 at 09:56:50AM +0100, Tobias Rapp wrote: > OpenDML specification v1.02 states that entries of a master index chunk > shall point to standard or field index chunks. > > Changed incorrect duration of last master index entry to -1 in case it > points to another master index. > >

Re: [FFmpeg-devel] [PATCH 2/3] avformat/avienc: add reserve_index_space option

2017-01-09 Thread Michael Niedermayer
On Mon, Jan 09, 2017 at 09:56:51AM +0100, Tobias Rapp wrote: > Allows the user to reserve space for the ODML master index. A sufficient > sized master index in the AVI header avoids storing follow-up master > indexes within the 'movi' data later. > > Signed-off-by: Tobias Rapp > --- > libavforma

[FFmpeg-devel] [PATCH 08/13] arm: vp9itxfm: Only reload the idct coeffs for the iadst_idct combination

2017-01-09 Thread Martin Storsjö
This avoids reloading them if they haven't been clobbered, if the first pass also was idct. This is similar to what was done in the aarch64 version. This is cherrypicked from libav commit 3c87039a404c5659ae9bf7454a04e186532eb40b. --- libavcodec/arm/vp9itxfm_neon.S | 2 +- 1 file changed, 1 inser

[FFmpeg-devel] [PATCH 07/13] aarch64: vp9itxfm: Don't repeatedly set x9 when nothing overwrites it

2017-01-09 Thread Martin Storsjö
This is cherrypicked from libav commit 2f99117f6ff24ce5be2abb9e014cb8b86c2aa0e0. --- libavcodec/aarch64/vp9itxfm_neon.S | 26 +++--- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/libavcodec/aarch64/vp9itxfm_neon.S b/libavcodec/aarch64/vp9itxfm_neon.S index d5

[FFmpeg-devel] [PATCH 12/13] aarch64: vp9dsp: Fix vertical alignment in the init file

2017-01-09 Thread Martin Storsjö
This is cherrypicked from libav commit 65074791e8f8397600aacc9801efdd1eb6e3. --- libavcodec/aarch64/vp9dsp_init_aarch64.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/aarch64/vp9dsp_init_aarch64.c b/libavcodec/aarch64/vp9dsp_init_aarch64.c index 7e34375

[FFmpeg-devel] [PATCH 03/13] arm: vp9itxfm: Simplify the stack alignment code

2017-01-09 Thread Martin Storsjö
From: Janne Grunau This is one instruction less for thumb, and only have got 1/2 arm/thumb specific instructions. This is cherrypicked from libav commit e5b0fc170f85b00f7dd0ac514918fb5c95253d39. --- libavcodec/arm/vp9itxfm_neon.S | 28 1 file changed, 12 insertions(

[FFmpeg-devel] [PATCH 02/13] aarch64: vp9: loop filter: replace 'orr; cbn?z' with 'adds; b.{eq,ne};

2017-01-09 Thread Martin Storsjö
From: Janne Grunau The latter is 1 cycle faster on a cortex-53 and since the operands are bytewise (or larger) bitmask (impossible to overflow to zero) both are equivalent. This is cherrypicked from libav commit e7ae8f7a715843a5089d18e033afb3ee19ab3057. --- libavcodec/aarch64/vp9lpf_neon.S | 31

[FFmpeg-devel] [PATCH 04/13] aarch64: vp9itxfm: Use w3 instead of x3 for the int eob parameter

2017-01-09 Thread Martin Storsjö
The clobbering tests in checkasm are only invoked when testing correctness, so this bug didn't show up when benchmarking the dc-only version. This is cherrypicked from libav commit 4d960a11855f4212eb3a4e470ce890db7f01df29. --- libavcodec/aarch64/vp9itxfm_neon.S | 8 1 file changed, 4 ins

[FFmpeg-devel] [PATCH 11/13] arm: vp9mc: Fix vertical alignment of operands

2017-01-09 Thread Martin Storsjö
This is cherrypicked from libav commit c536e5e8698110c139b1c17938998a5547550aa3. --- libavcodec/arm/vp9mc_neon.S | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/arm/vp9mc_neon.S b/libavcodec/arm/vp9mc_neon.S index 5fe3024..83235ff 100644 --- a/libavcodec/arm/vp

[FFmpeg-devel] [PATCH 05/13] arm/aarch64: vp9itxfm: Fix indentation of macro arguments

2017-01-09 Thread Martin Storsjö
This is cherrypicked from libav commit 721bc37522c5c1d6a8c3cea5e9c3fcde8d256c05. --- libavcodec/aarch64/vp9itxfm_neon.S | 16 libavcodec/arm/vp9itxfm_neon.S | 8 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/libavcodec/aarch64/vp9itxfm_neon.S b/li

[FFmpeg-devel] [PATCH 13/13] aarch64: vp9mc: Fix a comment to refer to a register with the right name

2017-01-09 Thread Martin Storsjö
This is cherrypicked from libav commit 85ad5ea72ce3983947a3b07e4b35c66cb16dfaba. --- libavcodec/aarch64/vp9mc_neon.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/aarch64/vp9mc_neon.S b/libavcodec/aarch64/vp9mc_neon.S index 69dad6d..80d1d23 100644 --- a/libavcodec/

[FFmpeg-devel] [PATCH 06/13] arm: vp9itxfm: Rename a macro parameter to fit better

2017-01-09 Thread Martin Storsjö
Since the same parameter is used for both input and output, the name inout is more fitting. This matches the naming used below in the dmbutterfly macro. This is cherrypicked from libav commit 79566ec8c77969d5f9be533de04b1349834cca62. --- libavcodec/arm/vp9itxfm_neon.S | 14 +++--- 1 file

[FFmpeg-devel] [PATCH 09/13] arm: vp9itxfm: Skip empty slices in the first pass of idct_idct 16x16 and 32x32

2017-01-09 Thread Martin Storsjö
This work is sponsored by, and copyright, Google. Previously all subpartitions except the eob=1 (DC) case ran with the same runtime: Cortex A7 A8 A9 A53 vp9_inv_dct_dct_16x16_sub16_add_neon: 3188.1 2435.4 2499.0 1969.0 vp9_inv_dct_dct_

[FFmpeg-devel] [PATCH 10/13] aarch64: vp9itxfm: Skip empty slices in the first pass of idct_idct 16x16 and 32x32

2017-01-09 Thread Martin Storsjö
This work is sponsored by, and copyright, Google. Previously all subpartitions except the eob=1 (DC) case ran with the same runtime: vp9_inv_dct_dct_16x16_sub16_add_neon: 1373.2 vp9_inv_dct_dct_32x32_sub32_add_neon: 8089.0 By skipping individual 8x16 or 8x32 pixel slices in the first pass, w

[FFmpeg-devel] [PATCH 01/13] aarch64: vp9: use alternative returns in the core loop filter function

2017-01-09 Thread Martin Storsjö
From: Janne Grunau Since aarch64 has enough free general purpose registers use them to branch to the appropiate storage code. 1-2 cycles faster for the functions using loop_filter 8/16, ... on a cortex-a53. Mixed results (up to 2 cycles faster/slower) on a cortex-a57. This is cherrypicked from l

[FFmpeg-devel] fate/psd : add test for 256 color psd file

2017-01-09 Thread Martin Vignali
Hello, Sample can be found here : https://we.tl/CX9svvyEdI Patch in attach Martin From bcf22f647ad41f3cbf1a8411dec0faf1ee5f933d Mon Sep 17 00:00:00 2001 From: Martin Vignali Date: Mon, 9 Jan 2017 23:03:42 +0100 Subject: [PATCH] fate/psd : add test for 256 color --- tests/fate/image.mak

Re: [FFmpeg-devel] swresample/aarch64: add s16 and fltp neon resampling functions

2017-01-09 Thread Michael Niedermayer
On Mon, Jan 09, 2017 at 05:45:29PM +0100, Matthieu Bouron wrote: > Hello, > > The following patch is a port of 6d82b52712af87c7831872c27fd2abfd18b04f88 to > the aarch64 platform. > > Here are some results of an rpi3 running in aarch64 mode. > > ./ffmpeg -f lavfi -i > sine=440,aformat=sample_fmt

Re: [FFmpeg-devel] [PATCH v3] doc/scaler: mention default scaling algorithm

2017-01-09 Thread Lou Logan
On Mon, 9 Jan 2017 01:19:28 +0100, Michael Niedermayer wrote: > should be ok > > thx Should have been [PATCH v2], but anyway, pushed. Thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v3 2/2] Newtek SpeedHQ decoder.

2017-01-09 Thread Steinar H. Gunderson
On Mon, Jan 09, 2017 at 09:44:42PM +0100, Michael Niedermayer wrote: > anything that works and is thread safe is fine with me > pick what you prefer from what people like I did the ff_thread_once() variant. v4 coming up. Fuzzing has been running for 22 hours now with no results at all (2.5B execs

[FFmpeg-devel] [PATCH v4 2/2] Newtek SpeedHQ decoder.

2017-01-09 Thread Steinar H. Gunderson
This decoder can decode all existing SpeedHQ formats (SHQ0–5, 7, and 9), including correct decoding of the alpha channel. 1080p is decoded in 142 fps on one core of my i7-4600U (2.1 GHz Haswell), about evenly split between bitstream reader and IDCT. There is currently no attempt at slice or frame

[FFmpeg-devel] [PATCH v4 1/2] Move bitswap_32() into a header file.

2017-01-09 Thread Steinar H. Gunderson
Allows more codecs than mpeg12video to make use of it. --- libavcodec/bitstream.c | 8 libavcodec/mathops.h | 8 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/libavcodec/bitstream.c b/libavcodec/bitstream.c index 6c8dca1d85..c26650099f 100644 --- a/libavcodec/b

Re: [FFmpeg-devel] [PATCH] avcodec/qdm2: make use of bytestream2

2017-01-09 Thread Michael Niedermayer
On Sun, Jan 08, 2017 at 08:00:01PM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/qdm2.c | 66 > ++- > 1 file changed, 21 insertions(+), 45 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B1

[FFmpeg-devel] [PATCH] tcp: set socket buffer sizes before listen/connect/accept

2017-01-09 Thread Joel Cunningham
From e24d95c0e06a878d401ee34fd6742fcaddeeb95f Mon Sep 17 00:00:00 2001 From: Joel Cunningham Date: Mon, 9 Jan 2017 13:37:51 -0600 Subject: [PATCH] tcp: set socket buffer sizes before listen/connect/accept Attempting to set SO_RCVBUF and SO_SNDBUF on TCP sockets after connection establishment is i

Re: [FFmpeg-devel] [PATCH v3 2/2] Newtek SpeedHQ decoder.

2017-01-09 Thread Michael Niedermayer
On Mon, Jan 09, 2017 at 08:21:25PM +0100, Steinar H. Gunderson wrote: > On Mon, Jan 09, 2017 at 08:17:18PM +0100, Michael Niedermayer wrote: > >> +/* Reverse the AC VLC, because INIT_VLC_LE wants it in that > >> order. */ > >> +for (i = 0; i < FF_ARRAY_ELEMS(speedhq_vlc); ++i) { >

[FFmpeg-devel] [PATCH] avformat: Add rtp_mpegtsraw (MPEG-TS RAW Stream output)

2017-01-09 Thread Andreas Håkon
Hi, This patch adds a new MUXER named "rtp_mpegtsraw". This new muxer works with the "mpegtsraw" input demuxer. The objective is enable direct bitstream input-output of MPEG-TS for RTP+FEC. Using the recent support for Pro-MPEG CoP FEC protocol, you can now use FFmpeg for streaming any UDP/RTP MPE

Re: [FFmpeg-devel] [PATCH v3 2/2] Newtek SpeedHQ decoder.

2017-01-09 Thread Kieran Kunhya
On Mon, 9 Jan 2017 at 19:21 Steinar H. Gunderson < steinar+ffm...@gunderson.no> wrote: > On Mon, Jan 09, 2017 at 08:17:18PM +0100, Michael Niedermayer wrote: > >> +/* Reverse the AC VLC, because INIT_VLC_LE wants it in that > order. */ > >> +for (i = 0; i < FF_ARRAY_ELEMS(speedhq_v

Re: [FFmpeg-devel] [PATCH] libopenmpt: add missing avio_read return value check

2017-01-09 Thread Jörn Heusipp
On 01/01/2017 08:27 PM, Andreas Cadhalpun wrote: This fixes heap-buffer-overflows in libopenmpt caused by interpreting the negative size value as unsigned size_t. Signed-off-by: Andreas Cadhalpun --- libavformat/libopenmpt.c | 5 + 1 file changed, 5 insertions(+) Looks fine to me. Rega

Re: [FFmpeg-devel] [PATCH v3 2/2] Newtek SpeedHQ decoder.

2017-01-09 Thread Steinar H. Gunderson
On Mon, Jan 09, 2017 at 08:17:18PM +0100, Michael Niedermayer wrote: >> +/* Reverse the AC VLC, because INIT_VLC_LE wants it in that order. >> */ >> +for (i = 0; i < FF_ARRAY_ELEMS(speedhq_vlc); ++i) { >> +speedhq_vlc[i][0] = reverse(speedhq_vlc[i][0], >> speedhq_vlc[i

Re: [FFmpeg-devel] [PATCH v3 2/2] Newtek SpeedHQ decoder.

2017-01-09 Thread Michael Niedermayer
On Sun, Jan 08, 2017 at 12:07:09PM +0100, Steinar H. Gunderson wrote: > This decoder can decode all existing SpeedHQ formats (SHQ0–5, 7, and 9), > including correct decoding of the alpha channel. > > 1080p is decoded in 142 fps on one core of my i7-4600U (2.1 GHz Haswell), > about evenly split bet

Re: [FFmpeg-devel] [PATCH] lavc/cuvid: fail early if GPU can't handle given video parameters

2017-01-09 Thread Pavel Koshevoy
On Mon, Jan 9, 2017 at 3:12 AM, wm4 wrote: > On Mon, 2 Jan 2017 14:26:45 -0700 > pkoshe...@gmail.com wrote: > >> From: Pavel Koshevoy >> +ret = convert_to_cuda_video_chroma_format(avctx->pix_fmt, >> &probed_chroma_format); >> +if (ret < 0) { >> +// pixel format is not supported

[FFmpeg-devel] [PATCH] mov: Fix spherical metadata_source field parsing.

2017-01-09 Thread Aaron Colwell
The attached patch fixes MOV spherical metadata parsing when the metadata_source field is not an empty string. The metadata_source field is a null-terminated string, like other ISOBMFF strings, not an 8-bit length followed by string characters. This patch fixes the parsing code so it skips over th

Re: [FFmpeg-devel] [PATCH] lavc/cuvid: fail early if GPU can't handle given video parameters

2017-01-09 Thread Pavel Koshevoy
On Sun, Jan 8, 2017 at 2:53 PM, Hendrik Leppkes wrote: > On Tue, Jan 3, 2017 at 8:26 AM, wrote: >> +ret = convert_to_cuda_video_chroma_format(avctx->pix_fmt, >> &probed_chroma_format); >> +if (ret < 0) { >> +// pixel format is not supported: >> +return ret; >> +} >

[FFmpeg-devel] [PATCH] lavc/cuvid: fail early if GPU can't handle video parameters

2017-01-09 Thread pkoshevoy
From: Pavel Koshevoy Evidently CUVID doesn't support decoding 422 or 444 chroma formats, and only a limited set of resolutions per codec are supported. Given that stream resolution and pixel format are typically known as a result of probing it is better to use this info during avcodec_open2 call

Re: [FFmpeg-devel] [PATCH v3 2/2] Newtek SpeedHQ decoder.

2017-01-09 Thread Steinar H. Gunderson
On Mon, Jan 09, 2017 at 06:47:12PM +0100, Paul B Mahol wrote: > If get_vlc2 fails, and return negative number and this does not cause crash > than patch LGTM. There's no crash no matter what decode_dc_le() returns, just a bogus DC coefficient. /* Steinar */ -- Homepage: https://www.sesse.net/ __

Re: [FFmpeg-devel] [PATCH v3 2/2] Newtek SpeedHQ decoder.

2017-01-09 Thread Paul B Mahol
On 1/9/17, Steinar H. Gunderson wrote: > On Mon, Jan 09, 2017 at 06:30:52PM +0100, Paul B Mahol wrote: >>> +/* AC codes: Very similar but not identical to MPEG-2. */ >>> +static uint16_t speedhq_vlc[123][2] = { >> Can this be uint8_t too? > > No, it goes into an RLTable, which expects an uint16_t*

Re: [FFmpeg-devel] [PATCH] avutil/frame: Remove requirements to use accessors for AVFrame

2017-01-09 Thread Michael Niedermayer
On Mon, Jan 09, 2017 at 10:27:09AM -0500, Ronald S. Bultje wrote: > Hi, > > On Mon, Jan 9, 2017 at 8:48 AM, wm4 wrote: > > > On Mon, 9 Jan 2017 08:17:08 -0500 > > "Ronald S. Bultje" wrote: > > > > > Hi, > > > > > > On Mon, Jan 9, 2017 at 7:15 AM, wm4 wrote: > > > > > > > /** > > > > *

Re: [FFmpeg-devel] [PATCH v3 2/2] Newtek SpeedHQ decoder.

2017-01-09 Thread Steinar H. Gunderson
On Mon, Jan 09, 2017 at 06:30:52PM +0100, Paul B Mahol wrote: >> +/* AC codes: Very similar but not identical to MPEG-2. */ >> +static uint16_t speedhq_vlc[123][2] = { > Can this be uint8_t too? No, it goes into an RLTable, which expects an uint16_t*. Besides, after bit-reversing, several elements

[FFmpeg-devel] [PATCH] lavf/mov.c: Add -editlist_fix_index option for mov format.

2017-01-09 Thread Sasi Inguva
Adding an MOV format option to turn on/off the editlist supporting code, introduced in https://github.com/FFmpeg/FFmpeg/commit/ca6cae73db207f17a0d5507609de12842d8f0ca3 Signed-off-by: Sasi Inguva --- libavformat/isom.h | 1 + libavformat/mov.c | 28 2 files change

Re: [FFmpeg-devel] [PATCH v3 2/2] Newtek SpeedHQ decoder.

2017-01-09 Thread Paul B Mahol
On 1/8/17, Steinar H. Gunderson wrote: > + * for the longest (10-bit) codes. > + */ > +#define ALPHA_VLC_BITS 5 > + > +typedef struct SHQContext { > +AVCodecContext *avctx; > +BlockDSPContext bdsp; > +IDCTDSPContext idsp; > +ScanTable intra_scantable; > +int quant_matrix[64]; >

[FFmpeg-devel] [PATCH] avformat/hlsenc: fix hls start and tail segment duration problem

2017-01-09 Thread Steven Liu
fix ticket: #6067 Signed-off-by: Steven Liu --- libavformat/hlsenc.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index eeb450a..0fcb699 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -103,6 +10

[FFmpeg-devel] [PATCH 1/2] swresample/aarch64: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2017-01-09 Thread Matthieu Bouron
--- libswresample/aarch64/Makefile| 6 +- libswresample/aarch64/resample.S | 76 + libswresample/aarch64/resample_init.c | 120 ++ libswresample/resample.h | 1 + libswresample/resample_dsp.c | 1 + 5 fil

[FFmpeg-devel] [PATCH 2/2] swresample/arm: cosmetic fixes

2017-01-09 Thread Matthieu Bouron
--- libswresample/arm/resample.S | 32 libswresample/arm/resample_init.c | 10 +- 2 files changed, 21 insertions(+), 21 deletions(-) diff --git a/libswresample/arm/resample.S b/libswresample/arm/resample.S index c231301b2b..3ce7623246 100644 --- a/lib

[FFmpeg-devel] swresample/aarch64: add s16 and fltp neon resampling functions

2017-01-09 Thread Matthieu Bouron
Hello, The following patch is a port of 6d82b52712af87c7831872c27fd2abfd18b04f88 to the aarch64 platform. Here are some results of an rpi3 running in aarch64 mode. ./ffmpeg -f lavfi -i sine=440,aformat=sample_fmts=fltp,asetnsamples=4096,abench=start,aresample=48000,abench=stop -t 1000 -f null

Re: [FFmpeg-devel] [PATCH] avutil/frame: Remove requirements to use accessors for AVFrame

2017-01-09 Thread Ronald S. Bultje
Hi, On Mon, Jan 9, 2017 at 8:48 AM, wm4 wrote: > On Mon, 9 Jan 2017 08:17:08 -0500 > "Ronald S. Bultje" wrote: > > > Hi, > > > > On Mon, Jan 9, 2017 at 7:15 AM, wm4 wrote: > > > > > /** > > > * Not to be accessed directly from outside libavutil > > > */ > > > AVBufferRef *qp_

Re: [FFmpeg-devel] [PATCH] avutil/frame: Remove requirements to use accessors for AVFrame

2017-01-09 Thread wm4
On Mon, 9 Jan 2017 08:17:08 -0500 "Ronald S. Bultje" wrote: > Hi, > > On Mon, Jan 9, 2017 at 7:15 AM, wm4 wrote: > > > /** > > * Not to be accessed directly from outside libavutil > > */ > > AVBufferRef *qp_table_buf; > > > > The comment should be adjusted and allow direct ac

Re: [FFmpeg-devel] [PATCH] avutil/frame: Remove requirements to use accessors for AVFrame

2017-01-09 Thread Ronald S. Bultje
Hi, On Mon, Jan 9, 2017 at 7:15 AM, wm4 wrote: > /** > * Not to be accessed directly from outside libavutil > */ > AVBufferRef *qp_table_buf; > > The comment should be adjusted and allow direct access. Also, > it should be moved out of the FF_API_FRAME_QP ifdef (why was > it un

Re: [FFmpeg-devel] [PATCH] avutil/frame: Remove requirements to use accessors for AVFrame

2017-01-09 Thread wm4
On Mon, 9 Jan 2017 13:06:35 +0100 Michael Niedermayer wrote: > After this it is not allowed to remove, insert or change fields except to > add them at the end. > > The accessors were in the past required to achieve ABI compatibility > with the fork. The community decided long ago that this ABI

Re: [FFmpeg-devel] [PATCH] avutil/frame: Remove requirements to use accessors for AVFrame

2017-01-09 Thread Paul B Mahol
On 1/9/17, Michael Niedermayer wrote: > After this it is not allowed to remove, insert or change fields except to > add them at the end. > > The accessors were in the past required to achieve ABI compatibility > with the fork. The community decided long ago that this ABI compatibility > is not nee

[FFmpeg-devel] [PATCH] avutil/frame: Remove requirements to use accessors for AVFrame

2017-01-09 Thread Michael Niedermayer
After this it is not allowed to remove, insert or change fields except to add them at the end. The accessors were in the past required to achieve ABI compatibility with the fork. The community decided long ago that this ABI compatibility is not needed and should be dropped, so the accessors lost t

Re: [FFmpeg-devel] [PATCH] lavfi: remove the duplicate expression evaluation.

2017-01-09 Thread wm4
On Mon, 9 Jan 2017 11:46:01 +0100 Nicolas George wrote: > Le decadi 20 nivôse, an CCXXV, Jun Zhao a écrit : > > I didn't find any case about x depends on y or y depends y. > > Some users have occasionally asked for it, IIRC. > > > If have this dependency relationship , why the code don't hand

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-09 Thread Nicolas George
Le decadi 20 nivôse, an CCXXV, wm4 a écrit : > Can we have a document in doc/ or a doxygen section that gives an > overview how the "new" internal API works? I already stated several times this was planned. BTW, « API-vomit » is not acceptable IMHO. -- Nicolas George _

Re: [FFmpeg-devel] [PATCH] lavfi: remove the duplicate expression evaluation.

2017-01-09 Thread Nicolas George
Le decadi 20 nivôse, an CCXXV, Jun Zhao a écrit : > I didn't find any case about x depends on y or y depends y. Some users have occasionally asked for it, IIRC. > If have this dependency relationship , why the code don't handle out_w/out_h > in > crop? Ideally, all variables should be able to

Re: [FFmpeg-devel] [PATCH 00/16] vaapi: Improve infrastructure, add VP8 support

2017-01-09 Thread wm4
On Sun, 8 Jan 2017 18:08:38 + Mark Thompson wrote: > Hi, > > This merges the improved VAAPI decode infrastructure from libav, with various > related parts updated to work together. > > The aims of this series are: > * Remove struct vaapi_context and move everything it used to do into lavc.

Re: [FFmpeg-devel] [PATCH] lavc/cuvid: fail early if GPU can't handle given video parameters

2017-01-09 Thread wm4
On Mon, 2 Jan 2017 14:26:45 -0700 pkoshe...@gmail.com wrote: > From: Pavel Koshevoy > > Evidently CUVID doesn't support decoding 422 or 444 chroma formats, > and only a limited set of resolutions per codec are supported. > > Given that stream resolution and pixel format are typically known as

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Add -editlist_fix_index option for mov format.

2017-01-09 Thread wm4
On Thu, 29 Dec 2016 15:04:07 -0800 Sasi Inguva wrote: > Signed-off-by: Sasi Inguva > --- > libavformat/isom.h | 1 + > libavformat/mov.c | 23 --- > 2 files changed, 21 insertions(+), 3 deletions(-) > > diff --git a/libavformat/isom.h b/libavformat/isom.h > index 12cefc9.

Re: [FFmpeg-devel] [PATCH 01/17] lavfi: add ff_inlink_acknowledge_status().

2017-01-09 Thread wm4
On Thu, 29 Dec 2016 15:33:47 +0100 Nicolas George wrote: > Also introduce libavfilter/filters.h for all functions needed > to implement filters. > > Signed-off-by: Nicolas George Can we have a document in doc/ or a doxygen section that gives an overview how the "new" internal API works? __

Re: [FFmpeg-devel] [PATCH 6/9] lavfi: move AVFilterLink declaration to its own header.

2017-01-09 Thread wm4
On Tue, 27 Dec 2016 21:09:26 +0100 Nicolas George wrote: > Le septidi 7 nivôse, an CCXXV, James Almer a écrit : > > It may not be something added with this patchset, but It's pretty much new > > and > > recent code, questioned by more people than just me prior to it being > > pushed. > > The

Re: [FFmpeg-devel] [PATCH 1/6] lavfi/buffersink: add accessors for the stream properties.

2017-01-09 Thread wm4
On Fri, 23 Dec 2016 21:02:09 +0100 Michael Niedermayer wrote: > On Fri, Dec 23, 2016 at 06:51:38PM +0100, Nicolas George wrote: > > Le tridi 3 nivôse, an CCXXV, Michael Niedermayer a écrit : > > > libavfilter had a public API that allowed filters to be implemented > > > these changes move it fa

Re: [FFmpeg-devel] [PATCH 2/5] libautil: add support for AV_HWDEVICE_TYPE_D3D11VA

2017-01-09 Thread wm4
On Wed, 4 Jan 2017 08:52:26 +0100 Steve Lhomme wrote: > On Fri, Dec 16, 2016 at 5:32 PM, wm4 wrote: > > On Fri, 16 Dec 2016 11:12:02 +0100 > > Steve Lhomme wrote: > > > >> From: Steve Lhomme > >> > >> --- > >> libavutil/Makefile | 3 + > >> libavutil/hwcontext.c | 3

[FFmpeg-devel] [PATCH 3/3] doc/muxers: add AVI muxer documentation

2017-01-09 Thread Tobias Rapp
Signed-off-by: Tobias Rapp --- doc/muxers.texi | 33 + 1 file changed, 33 insertions(+) diff --git a/doc/muxers.texi b/doc/muxers.texi index 351cd8c..d8353c7 100644 --- a/doc/muxers.texi +++ b/doc/muxers.texi @@ -57,6 +57,39 @@ fragmentation or muxer overhead depe

[FFmpeg-devel] [PATCH 1/3] avformat/avienc: write chained master index only if std_compliance is normal or below

2017-01-09 Thread Tobias Rapp
OpenDML specification v1.02 states that entries of a master index chunk shall point to standard or field index chunks. Changed incorrect duration of last master index entry to -1 in case it points to another master index. Propagate error when index write fails. Signed-off-by: Tobias Rapp --- l

[FFmpeg-devel] [PATCH 2/3] avformat/avienc: add reserve_index_space option

2017-01-09 Thread Tobias Rapp
Allows the user to reserve space for the ODML master index. A sufficient sized master index in the AVI header avoids storing follow-up master indexes within the 'movi' data later. Signed-off-by: Tobias Rapp --- libavformat/avi.h | 1 - libavformat/avienc.c | 36