Re: [FFmpeg-devel] [PATCH v3 1/5] lavf/avienc: Add support for palette side data packets

2016-03-01 Thread Mats Peterson
On 03/01/2016 08:14 AM, Mats Peterson wrote: This patch set adds support for non-raw codecs that use a palette. Try to do stream copy to and from avi with the files below: QuickTime Animation (RLE): https://drive.google.com/open?id=0B3_pEBoLs0faREo1SlRydmV1LU0 QuickTime Graphics (SMC): https:/

[FFmpeg-devel] [PATCH v3 5/5 v2] lavf/internal.h: Add declaration for ff_get_raw_palette()

2016-03-01 Thread Mats Peterson
Slightly more detailed doxy documentation. -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From ffdd2d19cfe2f87e9a12601a3b809521af5fa89b Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Tue, 1 Mar 2016 09:38:37 +0100 Subject: [PATCH v3 5/5 v2] lavf/internal.h: Add declaration for ff_get_raw

Re: [FFmpeg-devel] [PATCH]concatdec: measure duration when it is not available

2016-03-01 Thread Bodecs Bela
ping 2016.02.23. 9:21 keltezéssel, Bodecs Bela írta: ping 2016.02.17. 14:13 keltezéssel, Bodecs Bela írta: 2016.02.16. 10:42 keltezéssel, Nicolas George írta: L'octidi 28 pluviôse, an CCXXIV, Bodecs Bela a écrit : Do you have any suggestion for me what to do now? Please give me a hint whe

[FFmpeg-devel] [PATCH]configure: Check for msghdr struct

2016-03-01 Thread Carl Eugen Hoyos
Hi! A user claims that not all (Solaris) systems have a sufficiently new msghdr struct. Attached patch adds an additional check. Please comment, Carl Eugen diff --git a/configure b/configure index 8491fa1..2831ce4 100755 --- a/configure +++ b/configure @@ -1929,6 +1929,7 @@ TYPES_LIST=" str

Re: [FFmpeg-devel] [PATCH] lavfi: add bench and abench filters

2016-03-01 Thread Clément Bœsch
On Mon, Feb 29, 2016 at 07:37:35PM +0100, Paul B Mahol wrote: > On 2/29/16, Clement Boesch wrote: > > From: Clement Boesch > > > > --- > > TODO: doc, bump, Changelog > > > > example: -vf bench=start,gradfun,format=rgba,hqx,bench=stop > > Nice, LGTM with documentation changes. Added average/min/

Re: [FFmpeg-devel] [PATCH]lavf/mov: Set display aspect ratio for avid dv

2016-03-01 Thread Carl Eugen Hoyos
On Monday 29 February 2016 01:44:13 pm Michael Niedermayer wrote: > On Mon, Feb 29, 2016 at 11:52:24AM +0100, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch fixes ticket #5271 for me. > > > > Please comment, Carl Eugen > > > > mov.c |5 + > > 1 file changed, 5 insertions(+) > > ef0

[FFmpeg-devel] [PATCH 2/2] lavc: add AudioToolbox encoders

2016-03-01 Thread Rodger Combs
--- Changelog| 1 + configure| 10 + libavcodec/Makefile | 5 + libavcodec/allcodecs.c | 10 +- libavcodec/audiotoolboxenc.c | 471 +++ libavcodec/version.h | 2 +- 6 files changed, 493 i

[FFmpeg-devel] OSX AudioToolbox codec support

2016-03-01 Thread Rodger Combs
This series adds decoders and encoders based on OSX's AudioToolbox framework. They may be faster than the native ones (particularly in cases where they can be hardware-accelerated), the encoders are competitive on quality, and some users may find them convenient from a licensing perspective. ADPCM

[FFmpeg-devel] [PATCH 1/2] lavc: add AudioToolbox decoders

2016-03-01 Thread Rodger Combs
--- Changelog| 1 + configure| 24 libavcodec/Makefile | 14 ++ libavcodec/allcodecs.c | 14 ++ libavcodec/audiotoolboxdec.c | 334 +++ libavcodec/version.h | 2 +- 6 files changed, 3

[FFmpeg-devel] Accepted for GSoC 2016

2016-03-01 Thread Carl Eugen Hoyos
Hi! https://summerofcode.withgoogle.com/organizations/6504812191416320/ FFmpeg was accepted for GSoC 2016, thank you to everybody who helped with the wiki page and / or added himself as a mentor! If you have an idea for a doable, strictly defined project that a student has a chance to finish,

Re: [FFmpeg-devel] [PATCH]concatdec: measure duration when it is not available

2016-03-01 Thread Nicolas George
Le nonidi 29 pluviôse, an CCXXIV, Bodecs Bela a écrit : > I have checked the code where and when duration filled and made some > reasoning about it. > Duration value for input files has been populated some time after opening. > (estimate_timings function in utils.c) And never again corrected or cal

Re: [FFmpeg-devel] OSX AudioToolbox codec support

2016-03-01 Thread Carl Eugen Hoyos
Rodger Combs gmail.com> writes: > This series adds decoders and encoders based on OSX's AudioToolbox framework. They may be faster than the native ones (particularly in > cases where they can be hardware-accelerated), the encoders are > competitive on quality, and some I think this is a great

Re: [FFmpeg-devel] [PATCH v2] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

2016-03-01 Thread Clément Bœsch
On Mon, Feb 29, 2016 at 10:55:49AM +0100, Clément Bœsch wrote: > From: Clément Bœsch > > --- > Changes since latest version: > - remove unused 32-bit path > - make 16-bit path more accurate by mirroring the MMX code (still not > bitexact) > - the code as originally trying to process 2 lines at a

Re: [FFmpeg-devel] [PATCH]concatdec: measure duration when it is not available

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 10:57:57AM +0100, Nicolas George wrote: > Le nonidi 29 pluviôse, an CCXXIV, Bodecs Bela a écrit : > > I have checked the code where and when duration filled and made some > > reasoning about it. > > Duration value for input files has been populated some time after opening. >

Re: [FFmpeg-devel] [PATCH]configure: Check for msghdr struct

2016-03-01 Thread Moritz Barsnick
On Tue, Mar 01, 2016 at 10:32:55 +0100, Carl Eugen Hoyos wrote: > A user claims that not all (Solaris) systems have a sufficiently new msghdr > struct. Attached patch adds an additional check. This disables the sctp protocol on those systems then? Probably fine, but *searchengine* tells me that s

Re: [FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 01:40:53PM +0800, Rick Kern wrote: > Autodetected by default. Encode using -codec:v vtenc. [...] > +static void set_async_error(VTEncContext *vtctx, int err) > +{ > +BufNode *info; > + > +pthread_mutex_lock(&vtctx->lock); > + > +vtctx->async_error = err; > + > +

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-03-01 Thread Carl Eugen Hoyos
wm4 googlemail.com> writes: > Adding dozens of small very specialized features leads to > unmaintainable and unusable software, even if the change > itself is inoffensive. I don't think this is a "small" feature, I consider it a very useful request. I also wonder why it is "very specialized":

Re: [FFmpeg-devel] [PATCH]configure: Check for msghdr struct

2016-03-01 Thread Carl Eugen Hoyos
Moritz Barsnick gmx.net> writes: > On Tue, Mar 01, 2016 at 10:32:55 +0100, Carl Eugen Hoyos wrote: > > A user claims that not all (Solaris) systems have a sufficiently > > new msghdr struct. Attached patch adds an additional check. > > This disables the sctp protocol on those systems then? Prob

Re: [FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 11:31:18 +0100 Michael Niedermayer wrote: > On Tue, Mar 01, 2016 at 01:40:53PM +0800, Rick Kern wrote: > > Autodetected by default. Encode using -codec:v vtenc. > [...] > > +static void set_async_error(VTEncContext *vtctx, int err) > > +{ > > +BufNode *info; > > + > > +

Re: [FFmpeg-devel] [PATCH]configure: Check for msghdr struct

2016-03-01 Thread Moritz Barsnick
On Tue, Mar 01, 2016 at 12:01:10 +, Carl Eugen Hoyos wrote: > Which of the links you provided is related to msghdr? Possibly only the first one. I assumed that CMSG_SPACE was related, but that was my mind running amok. It would be easier to see if we had the Solaris headers or a Solaris insta

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace sqrtf(Q*sqrtf(Q)) by precomputed value

2016-03-01 Thread Derek Buitenhuis
On 3/1/2016 3:21 AM, Ganesh Ajjanagadde wrote: [...] > --- > libavcodec/aacenc_utils.h | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) Cool. Looks like an obvious/easy win, assuming it's identical. - Derek ___ ffmpeg-devel mailing list ffmp

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-03-01 Thread Ronald S. Bultje
Hi, On Tue, Mar 1, 2016 at 7:00 AM, Carl Eugen Hoyos wrote: > wm4 googlemail.com> writes: > > > Adding dozens of small very specialized features leads to > > unmaintainable and unusable software, even if the change > > itself is inoffensive. > > I don't think this is a "small" feature, I consid

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-03-01 Thread Carl Eugen Hoyos
Ronald S. Bultje gmail.com> writes: > > So how does this mechanism look like for the requested > > use case? > > man ln. As said, this is a completely ridiculous argument: FFmpeg has always tried to provide new features, no matter if other solutions existed or not. Apart from the fact that ln

[FFmpeg-devel] [PATCH 1/2] disabled loongson2, loongson3 and mmi for non-loongson cpu

2016-03-01 Thread shivraj.patil
From: Shivraj Patil For mips P5600/I6400 configure, assembler throws errors at check_inline_asm for loongson2, loongson3 and mmi as the instructions opcode not supported on this processor. Hence disabled loongson2, loongson3 and mmi in non loogson mips cpus. Signed-off-by: Shivraj Patil ---

[FFmpeg-devel] [PATCH 2/2] configure: build fix for mips cpu p5600

2016-03-01 Thread shivraj.patil
From: Shivraj Patil For P5600 mips cpu, cpuflags="-march=p5600" sets mips32r5 by default. Current configuration sets mips32r2 for p5600 cpu, hence ldflag check results in, error: '-mips32r2' conflicts with the other architecture options, which specify a mips32r5 processor Due to above error, m

Re: [FFmpeg-devel] [PATCH]configure: Check for msghdr struct

2016-03-01 Thread Moritz Barsnick
On Tue, Mar 01, 2016 at 14:00:14 +0100, Moritz Barsnick wrote: > Possibly only the first one. I assumed that CMSG_SPACE was related, but > that was my mind running amok. Actually, other sources hint that they are all related, defined as X/Open Unix Extensions. And these need to be enabled under So

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-03-01 Thread Ronald S. Bultje
Hi, On Tue, Mar 1, 2016 at 8:04 AM, Carl Eugen Hoyos wrote: > Ronald S. Bultje gmail.com> writes: > > > > So how does this mechanism look like for the requested > > > use case? > > > > man ln. > > As said, this is a completely ridiculous argument: > FFmpeg has always tried to provide new featur

Re: [FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 11:31:18 +0100 Michael Niedermayer wrote: > On Tue, Mar 01, 2016 at 01:40:53PM +0800, Rick Kern wrote: > > Autodetected by default. Encode using -codec:v vtenc. > [...] > > +static void set_async_error(VTEncContext *vtctx, int err) > > +{ > > +BufNode *info; > > + > > +

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace sqrtf(Q*sqrtf(Q)) by precomputed value

2016-03-01 Thread Rostislav Pehlivanov
On 1 March 2016 at 03:21, Ganesh Ajjanagadde wrote: > It makes no sense whatsoever to do this at each function call; we > already have a table for this. > > Yields a 2x improvement in find_min_book (x86-64, Haswell+GCC): > ffmpeg -i sin.flac -acodec aac -y sin.aac > find_min_book > old > 605

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-03-01 Thread Paul B Mahol
On 3/1/16, Ronald S. Bultje wrote: > Hi, > > On Tue, Mar 1, 2016 at 8:04 AM, Carl Eugen Hoyos wrote: > >> Ronald S. Bultje gmail.com> writes: >> >> > > So how does this mechanism look like for the requested >> > > use case? >> > >> > man ln. >> >> As said, this is a completely ridiculous argumen

Re: [FFmpeg-devel] [PATCH] Fix a bug in ff_thread_report_progress in updating progress[field].

2016-03-01 Thread Wan-Teh Chang
On Mon, Feb 29, 2016 at 7:00 PM, Ronald S. Bultje wrote: > > Do you have a sample+commandline to reproduce? The thing is, in all cases > where we use this, only one thread writes to a specific progress[n]. Two > threads may write to progress[], one per field, but one will write to > progress[0] an

Re: [FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-03-01 Thread Timothy Gu
Hi, On Mon, Feb 29, 2016 at 9:42 PM Rick Kern wrote: > Autodetected by default. Encode using -codec:v vtenc. > > Signed-off-by: Rick Kern > --- > MAINTAINERS|1 + > configure | 19 + > libavcodec/Makefile|1 + > libavcodec/allcodecs.c |1 + > libavcod

Re: [FFmpeg-devel] [PATCH] Add the relaxed and acquire/releae flavors of avpriv_atomic_int_get and avpriv_atomic_int_set.

2016-03-01 Thread Wan-Teh Chang
Hi, I have some remarks on this patch. 1. This patch is an alternative fix for the data races in the access to frame progress values in ff_thread_report_progress and ff_thread_await_progress. Between the two patches I submitted, I prefer the first patch (http://ffmpeg.org/pipermail/ffmpeg-devel/2

Re: [FFmpeg-devel] [PATCH] Fix a bug in ff_thread_report_progress in updating progress[field].

2016-03-01 Thread Ronald S. Bultje
Hi, On Tue, Mar 1, 2016 at 9:34 AM, Wan-Teh Chang wrote: > By the way, I'm also wondering why ff_thread_report_progress is > sometimes called with progress[field] >= n? I saw it happen when I ran > make fate-h264, but did not investigate it. I don't know, which sample (test name) specifically?

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 08:01:26AM -0500, Ronald S. Bultje wrote: > Hi, > > On Tue, Mar 1, 2016 at 7:00 AM, Carl Eugen Hoyos wrote: > > > wm4 googlemail.com> writes: > > > > > Adding dozens of small very specialized features leads to > > > unmaintainable and unusable software, even if the chang

Re: [FFmpeg-devel] [PATCH v9] VideoToolbox H.264 Encoder

2016-03-01 Thread wm4
On Tue, 01 Mar 2016 14:57:29 + Timothy Gu wrote: > Hi, > > On Mon, Feb 29, 2016 at 9:42 PM Rick Kern wrote: > > > Autodetected by default. Encode using -codec:v vtenc. > > > > Signed-off-by: Rick Kern > > --- > > MAINTAINERS|1 + > > configure | 19 + > > l

[FFmpeg-devel] [WIP] vc2 encoder simd assembly

2016-03-01 Thread James Darnley
Hello I've been working on assembly for the vc2 encoder and have reached an impasse. My code results in very visible errors, very obvious vertical streaks in the bottom-right half of the image and some low-frequency effect (I think). I cannot see the problem in my code so I need some fresh eyes

Re: [FFmpeg-devel] [PATCH v2] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 11:11:36AM +0100, Clément Bœsch wrote: > On Mon, Feb 29, 2016 at 10:55:49AM +0100, Clément Bœsch wrote: > > From: Clément Bœsch > > > > --- > > Changes since latest version: > > - remove unused 32-bit path > > - make 16-bit path more accurate by mirroring the MMX code (sti

Re: [FFmpeg-devel] [PATCH v2] sws/aarch64: add {nv12, nv21, yuv420p, yuv422p}_to_{argb, rgba, abgr, rgba}_neon

2016-03-01 Thread Clément Bœsch
On Tue, Mar 01, 2016 at 05:18:36PM +0100, Michael Niedermayer wrote: > On Tue, Mar 01, 2016 at 11:11:36AM +0100, Clément Bœsch wrote: > > On Mon, Feb 29, 2016 at 10:55:49AM +0100, Clément Bœsch wrote: > > > From: Clément Bœsch > > > > > > --- > > > Changes since latest version: > > > - remove unu

[FFmpeg-devel] [PATCH] lavc/mjpegdec: avoid unneeded allocation if the frame is to be skipped

2016-03-01 Thread Matthieu Bouron
From: Matthieu Bouron --- libavcodec/mjpegdec.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c index f41f3a5..1e83e88 100644 --- a/libavcodec/mjpegdec.c +++ b/libavcodec/mjpegdec.c @@ -614,6 +614,13 @@ unk_pixfmt: return AVERROR_

[FFmpeg-devel] [PATCH] Add the relaxed and acquire/releae flavors of avpriv_atomic_int_get and avpriv_atomic_int_set.

2016-03-01 Thread Wan-Teh Chang
Correct the order of the load/store operation and the memory barrier in avpriv_atomic_int_get and avpriv_atomic_int_set. Signed-off-by: Wan-Teh Chang --- libavutil/atomic.c | 48 libavutil/atomic.h | 45

Re: [FFmpeg-devel] [PATCH] avformat/udp: Add a delay between packets for streaming to clients with short buffer

2016-03-01 Thread Michael Niedermayer
On Mon, Feb 29, 2016 at 10:40:02PM +0300, Pavel Nikiforov wrote: > Hello ! > > This patch enables background sending of UDP packets with specified delay. > When sending packets without a delay some devices with small RX buffer > ( MAG200 STB, for example) will drop tail packets in burst causing >

[FFmpeg-devel] Patch: Add test for avpriv_get_trc_function_from_trc function

2016-03-01 Thread NagaChaitanya Vellanki
Hi all, This is my first patch. git send-mail did not work for me. Sending the patch via gmail gui. Please review my patch. Thank you, NagaChaitanya Vellanki 0001-Add-test-for-avpriv_get_trc_function_from_trc-functi.patch Description: Binary data ___ f

Re: [FFmpeg-devel] [PATCH] Add the relaxed and acquire/releae flavors of avpriv_atomic_int_get and avpriv_atomic_int_set.

2016-03-01 Thread Wan-Teh Chang
Hi, I removed the ff_thread_report_progress and ff_thread_await_progress changes from this patch. This patch now only changes libavutil/atomic*. I also added some comments to libavutil/atomic.h to describe how the acquire and release operations are intended to be used. While working on the commen

[FFmpeg-devel] [PATCH] Move the |die| member of FrameThreadContext to PerThreadContext.

2016-03-01 Thread Wan-Teh Chang
This fixes a data race warning by ThreadSanitizer. FrameThreadContext.die is read by all the worker threads but is not protected by any mutex. Move it to PerThreadContext so that each worker thread reads its own copy of |die|, which can then be protected with PerThreadContext.mutex. Signed-off-by:

Re: [FFmpeg-devel] [PATCH] Move the |die| member of FrameThreadContext to PerThreadContext.

2016-03-01 Thread Wan-Teh Chang
On Fri, Feb 26, 2016 at 12:14 PM, Ronald S. Bultje wrote: > > Is probably OK since it doesn't introduce new lock/unlock pairs, yes. We > typically don't care about a few bytes of memory. Hi Ronald, I just signed off this patch and submitted it in http://ffmpeg.org/pipermail/ffmpeg-devel/2016-Mar

[FFmpeg-devel] [PATCH 0/6] Add decoding/encoding API with decoupled input/output

2016-03-01 Thread wm4
This has been often discussed, and everyone agreed it was a good idea. Here are patches which add such an API, but without making proper use of it yet. wm4 (6): lavu: improve documentation of some AVFrame functions lavc: factor apply_param_change() AV_EF_EXPLODE handling lavc: introduce a ne

[FFmpeg-devel] [PATCH 3/6] lavc: introduce a new decoding/encoding API with decoupled input/output

2016-03-01 Thread wm4
Until now, the decoding API was restricted to outputting 0 or 1 frames per input packet. It also enforces a somewhat rigid dataflow in general. This new API seeks to relax these restrictions by decoupling input and output. Instead of doing a single call on each decode step, which may consume the p

[FFmpeg-devel] [PATCH 1/6] lavu: improve documentation of some AVFrame functions

2016-03-01 Thread wm4
--- libavutil/frame.h | 12 1 file changed, 12 insertions(+) diff --git a/libavutil/frame.h b/libavutil/frame.h index 76a8123..2d6299b 100644 --- a/libavutil/frame.h +++ b/libavutil/frame.h @@ -591,6 +591,10 @@ void av_frame_free(AVFrame **frame); * If src is not reference counted,

[FFmpeg-devel] [PATCH 2/6] lavc: factor apply_param_change() AV_EF_EXPLODE handling

2016-03-01 Thread wm4
Remove the duplicated code for handling failure of apply_param_change(). --- libavcodec/utils.c | 34 +++--- 1 file changed, 19 insertions(+), 15 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index f435588..86236f8 100644 --- a/libavcodec/utils.c ++

[FFmpeg-devel] [PATCH 5/6] ffmpeg: use new decode API

2016-03-01 Thread wm4
--- Unfortunately requires a hack to make FATE pass: ffmpeg.c will pass a flush AVPacket (data/size are null) with the dts field set, and will expect to get the dts back with the flushed frame. Should probably be fixed somehow. I'm not even sure if this is valid API usage (with the old API). --- f

[FFmpeg-devel] [PATCH 4/6] ffmpeg: remove sub-frame warning

2016-03-01 Thread wm4
It's not practical to keep this with the new decode API. --- ffmpeg.c | 7 --- ffmpeg.h | 1 - 2 files changed, 8 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index d148588..f24ee41 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -2313,13 +2313,6 @@ static int process_input_packet(InputStream *ist,

[FFmpeg-devel] [PATCH 6/6] ffmpeg: use new encode API

2016-03-01 Thread wm4
--- I'm wondering if the ost->sync_opts++; and ost->frame_number++; lines in do_video_out() should be moved into the loop. --- ffmpeg.c | 71 +--- 1 file changed, 46 insertions(+), 25 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index 5

[FFmpeg-devel] [PATCH] Add test for avpriv_get_trc_function_from_trc function

2016-03-01 Thread NagaChaitanya Vellanki
Improve coverage by adding test. NagaChaitanya Vellanki (1): Add test for avpriv_get_trc_function_from_trc function libavutil/Makefile | 1 + libavutil/color_utils.c | 52 + 2 files changed, 53 insertions(+) -- 2.7.2

[FFmpeg-devel] [PATCH] Add test for avpriv_get_trc_function_from_trc function

2016-03-01 Thread NagaChaitanya Vellanki
--- libavutil/Makefile | 1 + libavutil/color_utils.c | 52 + 2 files changed, 53 insertions(+) diff --git a/libavutil/Makefile b/libavutil/Makefile index a4d79cd..934564f 100644 --- a/libavutil/Makefile +++ b/libavutil/Makefile @@ -176,6 +176

Re: [FFmpeg-devel] [PATCH] Read |state| under the protection of |mutex| or |progress_mutex|.

2016-03-01 Thread Wan-Teh Chang
Hi Ronald, On Fri, Feb 26, 2016 at 12:16 PM, Ronald S. Bultje wrote: > > This will be slower, right? Is this an actual issue? Like, can you point to > cases like [1] (i.e. a real-world race condition with real-world > consequences) that were fixed with your patch? > > Ronald > > https://blogs.gno

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread Matthieu Bouron
On Fri, Feb 26, 2016 at 05:36:40PM +0100, Matthieu Bouron wrote: > On Fri, Feb 26, 2016 at 4:41 PM, Matthieu Bouron > wrote: > > > On Mon, Feb 22, 2016 at 12:20:35PM +0100, Matthieu Bouron wrote: > > > From: Matthieu Bouron > > > > > > > [...] > > > > > > Patch updated with the following diff

Re: [FFmpeg-devel] [PATCH] Move the |die| member of FrameThreadContext to PerThreadContext.

2016-03-01 Thread Ronald S. Bultje
Hi, On Tue, Mar 1, 2016 at 1:15 PM, Wan-Teh Chang wrote: > On Fri, Feb 26, 2016 at 12:14 PM, Ronald S. Bultje > wrote: > > > > Is probably OK since it doesn't introduce new lock/unlock pairs, yes. We > > typically don't care about a few bytes of memory. > > Hi Ronald, > > I just signed off this

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 19:52:08 +0100 Matthieu Bouron wrote: > On Fri, Feb 26, 2016 at 05:36:40PM +0100, Matthieu Bouron wrote: > > On Fri, Feb 26, 2016 at 4:41 PM, Matthieu Bouron > > wrote: > > > > > On Mon, Feb 22, 2016 at 12:20:35PM +0100, Matthieu Bouron wrote: > > > > From: Matthieu Bouro

Re: [FFmpeg-devel] [PATCH 2/2] lavc: add h264 mediacodec decoder

2016-03-01 Thread Matthieu Bouron
On Sat, Feb 27, 2016 at 04:28:43PM +0100, Michael Niedermayer wrote: > On Fri, Feb 26, 2016 at 04:54:47PM +0100, Matthieu Bouron wrote: > > On Tue, Feb 23, 2016 at 11:28:01AM +0100, wm4 wrote: > > > On Tue, 23 Feb 2016 09:53:43 +0100 > > > Matthieu Bouron wrote: > > > > > > > On Mon, Feb 22, 2016

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread Matthieu Bouron
On Tue, Mar 01, 2016 at 07:56:30PM +0100, wm4 wrote: > On Tue, 1 Mar 2016 19:52:08 +0100 > Matthieu Bouron wrote: > > > On Fri, Feb 26, 2016 at 05:36:40PM +0100, Matthieu Bouron wrote: > > > On Fri, Feb 26, 2016 at 4:41 PM, Matthieu Bouron > > > > > > wrote: > > > > > > > On Mon, Feb 22, 201

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 20:10:50 +0100 Matthieu Bouron wrote: > On Tue, Mar 01, 2016 at 07:56:30PM +0100, wm4 wrote: > > On Tue, 1 Mar 2016 19:52:08 +0100 > > Matthieu Bouron wrote: > > > > > On Fri, Feb 26, 2016 at 05:36:40PM +0100, Matthieu Bouron wrote: > > > > On Fri, Feb 26, 2016 at 4:41 PM

Re: [FFmpeg-devel] [PATCH] sdp: fix opus sprop-stereo fmtp syntax

2016-03-01 Thread Michael Niedermayer
On Mon, Feb 29, 2016 at 08:08:14PM -0800, Mark Harris wrote: > --- > libavformat/sdp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) https://tools.ietf.org/html/draft-ietf-payload-rtp-opus-11 uses "=" in the example so ok and applied thanks [...] -- Michael GnuPG fingerprint: 9FF

Re: [FFmpeg-devel] [PATCH] Read |state| under the protection of |mutex| or |progress_mutex|.

2016-03-01 Thread Ronald S. Bultje
Hi, On Tue, Mar 1, 2016 at 1:47 PM, Wan-Teh Chang wrote: > Hi Ronald, > > On Fri, Feb 26, 2016 at 12:16 PM, Ronald S. Bultje > wrote: > > > > This will be slower, right? Is this an actual issue? Like, can you point > to > > cases like [1] (i.e. a real-world race condition with real-world > > co

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread Matthieu Bouron
On Tue, Mar 01, 2016 at 08:13:48PM +0100, wm4 wrote: > On Tue, 1 Mar 2016 20:10:50 +0100 > Matthieu Bouron wrote: > > > On Tue, Mar 01, 2016 at 07:56:30PM +0100, wm4 wrote: > > > On Tue, 1 Mar 2016 19:52:08 +0100 > > > Matthieu Bouron wrote: > > > > > > > On Fri, Feb 26, 2016 at 05:36:40PM +0

Re: [FFmpeg-devel] [PATCH] Add test for avpriv_get_trc_function_from_trc function

2016-03-01 Thread Clément Bœsch
On Tue, Mar 01, 2016 at 10:41:32AM -0800, NagaChaitanya Vellanki wrote: > --- > libavutil/Makefile | 1 + > libavutil/color_utils.c | 52 > + > 2 files changed, 53 insertions(+) > I'm afraid testing the result of a switch with an array has a

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread Matthieu Bouron
On Tue, Mar 01, 2016 at 08:24:27PM +0100, Matthieu Bouron wrote: > On Tue, Mar 01, 2016 at 08:13:48PM +0100, wm4 wrote: > > On Tue, 1 Mar 2016 20:10:50 +0100 > > Matthieu Bouron wrote: > > > > > On Tue, Mar 01, 2016 at 07:56:30PM +0100, wm4 wrote: > > > > On Tue, 1 Mar 2016 19:52:08 +0100 > > > >

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 20:33:14 +0100 Matthieu Bouron wrote: > On Tue, Mar 01, 2016 at 08:24:27PM +0100, Matthieu Bouron wrote: > > On Tue, Mar 01, 2016 at 08:13:48PM +0100, wm4 wrote: > > > On Tue, 1 Mar 2016 20:10:50 +0100 > > > Matthieu Bouron wrote: > > > > > > > On Tue, Mar 01, 2016 at 07:

[FFmpeg-devel] [PATCH] Add test for avpriv_get_trc_function_from_trc function

2016-03-01 Thread NagaChaitanya Vellanki
--- libavutil/Makefile | 1 + libavutil/color_utils.c | 53 + 2 files changed, 54 insertions(+) diff --git a/libavutil/Makefile b/libavutil/Makefile index a4d79cd..934564f 100644 --- a/libavutil/Makefile +++ b/libavutil/Makefile @@ -176,6 +176

Re: [FFmpeg-devel] [PATCH] lavc/mjpegdec: avoid unneeded allocation if the frame is to be skipped

2016-03-01 Thread Paul B Mahol
On 3/1/16, Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > libavcodec/mjpegdec.c | 7 +++ > 1 file changed, 7 insertions(+) > probbably ok ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-dev

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread Matthieu Bouron
On Tue, Mar 01, 2016 at 08:38:04PM +0100, wm4 wrote: > On Tue, 1 Mar 2016 20:33:14 +0100 > Matthieu Bouron wrote: > > > On Tue, Mar 01, 2016 at 08:24:27PM +0100, Matthieu Bouron wrote: > > > On Tue, Mar 01, 2016 at 08:13:48PM +0100, wm4 wrote: > > > > On Tue, 1 Mar 2016 20:10:50 +0100 > > > > M

Re: [FFmpeg-devel] [PATCH]lavf/mov: Set display aspect ratio for avid dv

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 10:46:17AM +0100, Carl Eugen Hoyos wrote: > On Monday 29 February 2016 01:44:13 pm Michael Niedermayer wrote: > > On Mon, Feb 29, 2016 at 11:52:24AM +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes ticket #5271 for me. > > > > > > Please comment, Car

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread Matthieu Bouron
On Tue, Mar 01, 2016 at 08:55:23PM +0100, Matthieu Bouron wrote: > On Tue, Mar 01, 2016 at 08:38:04PM +0100, wm4 wrote: > > On Tue, 1 Mar 2016 20:33:14 +0100 > > Matthieu Bouron wrote: > > > > > On Tue, Mar 01, 2016 at 08:24:27PM +0100, Matthieu Bouron wrote: > > > > On Tue, Mar 01, 2016 at 08:13

Re: [FFmpeg-devel] [PATCH] Add test for avpriv_get_trc_function_from_trc function

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 11:37:37AM -0800, NagaChaitanya Vellanki wrote: > --- > libavutil/Makefile | 1 + > libavutil/color_utils.c | 53 > + > 2 files changed, 54 insertions(+) to test avpriv_get_trc_function_from_trc(), one could for each i

Re: [FFmpeg-devel] [PATCH] Add test for avpriv_get_trc_function_from_trc function

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 09:11:21PM +0100, Michael Niedermayer wrote: > On Tue, Mar 01, 2016 at 11:37:37AM -0800, NagaChaitanya Vellanki wrote: > > --- > > libavutil/Makefile | 1 + > > libavutil/color_utils.c | 53 > > + > > 2 files changed, 5

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

2016-03-01 Thread wm4
On Tue, 1 Mar 2016 20:57:12 +0100 Matthieu Bouron wrote: > On Tue, Mar 01, 2016 at 08:55:23PM +0100, Matthieu Bouron wrote: > > On Tue, Mar 01, 2016 at 08:38:04PM +0100, wm4 wrote: > > > On Tue, 1 Mar 2016 20:33:14 +0100 > > > Matthieu Bouron wrote: > > > > > > > On Tue, Mar 01, 2016 at 08:

Re: [FFmpeg-devel] [PATCH] Add the relaxed and acquire/releae flavors of avpriv_atomic_int_get and avpriv_atomic_int_set.

2016-03-01 Thread Ronald S. Bultje
Hi, On Tue, Mar 1, 2016 at 10:26 AM, Wan-Teh Chang wrote: > I'd like to reiterate that this is difficult for many good > programmers. I think you just hit the nail right on the head :). I think my personal problem is that I don't quite understand what the actual problem is (your wiki page on m

Re: [FFmpeg-devel] [PATCH 2/6] lavc: factor apply_param_change() AV_EF_EXPLODE handling

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 07:21:37PM +0100, wm4 wrote: > Remove the duplicated code for handling failure of apply_param_change(). > --- > libavcodec/utils.c | 34 +++--- > 1 file changed, 19 insertions(+), 15 deletions(-) LGTM thanks [...] -- Michael GnuPG fingerp

[FFmpeg-devel] [PATCH] aacenc: avoid double in quantize_bands.

2016-03-01 Thread Reimar Döffinger
I cannot see any point whatsoever to use double here instead of float. Using float allows for use of SIMD. --- libavcodec/aacenc_utils.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavcodec/aacenc_utils.h b/libavcodec/aacenc_utils.h index cb5bc8d..571b1e6 100644 ---

Re: [FFmpeg-devel] [PATCH 2/2] lavc: add h264 mediacodec decoder

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 08:01:45PM +0100, Matthieu Bouron wrote: > On Sat, Feb 27, 2016 at 04:28:43PM +0100, Michael Niedermayer wrote: > > On Fri, Feb 26, 2016 at 04:54:47PM +0100, Matthieu Bouron wrote: > > > On Tue, Feb 23, 2016 at 11:28:01AM +0100, wm4 wrote: > > > > On Tue, 23 Feb 2016 09:53:4

Re: [FFmpeg-devel] [PATCH 1/2] disabled loongson2, loongson3 and mmi for non-loongson cpu

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 06:41:17PM +0530, shivraj.pa...@imgtec.com wrote: > From: Shivraj Patil > > For mips P5600/I6400 configure, assembler throws errors at check_inline_asm > for loongson2, loongson3 and mmi as the instructions opcode not supported on > this processor. Hence disabled loongso

Re: [FFmpeg-devel] [PATCH 2/2] configure: build fix for mips cpu p5600

2016-03-01 Thread Michael Niedermayer
On Tue, Mar 01, 2016 at 06:41:18PM +0530, shivraj.pa...@imgtec.com wrote: > From: Shivraj Patil > > For P5600 mips cpu, cpuflags="-march=p5600" sets mips32r5 by default. > Current configuration sets mips32r2 for p5600 cpu, hence ldflag check results > in, > error: '-mips32r2' conflicts with the

[FFmpeg-devel] [PATCH] aacenc: optimize cost cache.

2016-03-01 Thread Reimar Döffinger
Avoids trashing the CPU cache each time the cost cache is cleared. --- libavcodec/aaccoder_twoloop.h | 20 libavcodec/aacenc.c | 7 +-- libavcodec/aacenc.h | 4 +--- libavcodec/aacenc_quantization_misc.h | 17 +++--

Re: [FFmpeg-devel] [PATCH] Fix a bug in ff_thread_report_progress in updating progress[field].

2016-03-01 Thread Wan-Teh Chang
On Tue, Mar 1, 2016 at 7:44 AM, Ronald S. Bultje wrote: > Hi, > > On Tue, Mar 1, 2016 at 9:34 AM, Wan-Teh Chang > wrote: > >> By the way, I'm also wondering why ff_thread_report_progress is >> sometimes called with progress[field] >= n? I saw it happen when I ran >> make fate-h264, but did not in

Re: [FFmpeg-devel] [PATCH]lavf/mov: Set display aspect ratio for avid dv

2016-03-01 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > > New patch attached. > > > > Please comment, Carl Eugen > > probably ok Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffm

Re: [FFmpeg-devel] [PATCH] avformat/seek-test: Support passing options to demuxers and protocols

2016-03-01 Thread Michael Niedermayer
On Mon, Feb 29, 2016 at 09:37:51PM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/seek-test.c |2 ++ > 1 file changed, 2 insertions(+) applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Its not that you shou

Re: [FFmpeg-devel] [PATCH] fate: add pipe and cache test

2016-03-01 Thread Michael Niedermayer
On Mon, Feb 29, 2016 at 09:36:55PM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > tests/fate/seek.mak |2 ++ > tests/ref/seek/cache-pipe | 49 > + > 2 files changed, 51 insertions(+) > create mode 100644 tes

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-03-01 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > and ffmpeg supports alot more than files, one could want to do > > -reverse 1 -i http://foobar.com/image%d.jpeg This patch is unrelated to demuxing, it is meant to implement: $ ffmpeg -i input -reverse 1 -start_number xyz out%3d.jpg Carl Eugen __

[FFmpeg-devel] [PATCH v4 1/5] lavf/avienc: Add support for palette side data packets

2016-03-01 Thread Mats Peterson
Hopefully the last version of the patch set. Try to do stream copy to and from avi/mov with the files below: QuickTime Animation (RLE): https://drive.google.com/open?id=0B3_pEBoLs0faREo1SlRydmV1LU0 QuickTime Graphics (SMC): https://drive.google.com/open?id=0B3_pEBoLs0faODd5RVBldkdvVGc Microsof

[FFmpeg-devel] [PATCH v4 2/5] lavf/movenc: Add support for palette side data packets

2016-03-01 Thread Mats Peterson
-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From e4246cb978aa165cf2c1b51e62a97733398f5149 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Wed, 2 Mar 2016 03:12:17 +0100 Subject: [PATCH v4 2/5] lavf/movenc: Add support for palette side data packets --- libavformat/movenc.c | 45

[FFmpeg-devel] [PATCH v4 3/5] lavf/riffenc: Handle palette for non-raw codecs

2016-03-01 Thread Mats Peterson
-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From bb8b4c33f782d8c121a0575c5175739036906d27 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Wed, 2 Mar 2016 03:13:14 +0100 Subject: [PATCH v4 3/5] lavf/riffenc: Handle palette for non-raw codecs --- libavformat/riffenc.c | 20 ++

[FFmpeg-devel] [PATCH v4 4/5] lavf/utils: New function ff_get_packet_palette()

2016-03-01 Thread Mats Peterson
-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 403d4db37d2690a8263db53b28225409eeb9bb8c Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Wed, 2 Mar 2016 03:14:05 +0100 Subject: [PATCH v4 4/5] lavf/utils: New function ff_get_packet_palette() --- libavformat/utils.c | 16 +++

[FFmpeg-devel] [PATCH v4 5/5] lavf/internals.h: Add declaration for ff_get_packet_palette()

2016-03-01 Thread Mats Peterson
-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 0fab64e2c08519302c3b44ae53ab01cfb3322812 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Wed, 2 Mar 2016 03:14:34 +0100 Subject: [PATCH v4 5/5] lavf/internals.h: Add declaration for ff_get_packet_palette() --- libavformat/internal.h |

[FFmpeg-devel] Patch: Add test for avpriv_get_trc_function_from_trc function

2016-03-01 Thread NagaChaitanya Vellanki
Hi all, This is my first patch. git send-mail did not work for me. Sending the patch via gmail gui. Please review my patch. Thank you, NagaChaitanya Vellanki 0001-Add-test-for-avpriv_get_trc_function_from_trc-functi.patch Description: Binary data ___ f

Re: [FFmpeg-devel] [PATCH] aacenc: optimize cost cache.

2016-03-01 Thread Ganesh Ajjanagadde
On Tue, Mar 1, 2016 at 6:34 PM, Reimar Döffinger wrote: > Avoids trashing the CPU cache each time the > cost cache is cleared. Just curious as to roughly the magnitude of perf improvement? No comments on the patch itself. [...] ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH]lavf/img2enc: Allow to reverse frame order

2016-03-01 Thread Ganesh Ajjanagadde
On Tue, Mar 1, 2016 at 9:17 AM, Paul B Mahol wrote: > On 3/1/16, Ronald S. Bultje wrote: >> Hi, >> >> On Tue, Mar 1, 2016 at 8:04 AM, Carl Eugen Hoyos wrote: >> >>> Ronald S. Bultje gmail.com> writes: >>> >>> > > So how does this mechanism look like for the requested >>> > > use case? >>> > >>>

Re: [FFmpeg-devel] [PATCH] aacenc: avoid double in quantize_bands.

2016-03-01 Thread Ganesh Ajjanagadde
On Tue, Mar 1, 2016 at 4:55 PM, Reimar Döffinger wrote: > I cannot see any point whatsoever to use > double here instead of float. There can be some negligible accuracy differences, but I do not see any harm myself; aac anyway sticks to floats in most places. > Using float allows for use of SIMD

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace sqrtf(Q*sqrtf(Q)) by precomputed value

2016-03-01 Thread Ganesh Ajjanagadde
On Tue, Mar 1, 2016 at 7:52 AM, Derek Buitenhuis wrote: > On 3/1/2016 3:21 AM, Ganesh Ajjanagadde wrote: > > [...] > >> --- >> libavcodec/aacenc_utils.h | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) > > Cool. Looks like an obvious/easy win, assuming it's identical. They are not prec

Re: [FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace sqrtf(Q*sqrtf(Q)) by precomputed value

2016-03-01 Thread Ganesh Ajjanagadde
On Tue, Mar 1, 2016 at 9:14 AM, Rostislav Pehlivanov wrote: > On 1 March 2016 at 03:21, Ganesh Ajjanagadde wrote: >> >> It makes no sense whatsoever to do this at each function call; we >> already have a table for this. >> >> Yields a 2x improvement in find_min_book (x86-64, Haswell+GCC): >> ffmp

[FFmpeg-devel] FLIC in AVI

2016-03-01 Thread Mats Peterson
I noticed that you, Paul, added a patch somewhere in 2012 with the purpose of making it possible to use FLI(C) in AVI. It doesn't work very well in my book. Try the following command with the file below: ffmpeg -i 3DMORPH.FLI -vcodec copy 3dmorph.avi File: https://drive.google.com/open?id=0B3_

  1   2   >