Re: [FFmpeg-devel] [PATCH 2/9] arm: Clear the gp register alias at the end of functions

2016-11-14 Thread Ronald S. Bultje
Hi, assume this is my reply on all patches in the range 2-9: On Mon, Nov 14, 2016 at 5:32 AM, Martin Storsjö wrote: > We reset .Lpic_gp to zero at the start of each function, which means > that the logic within movrelx for clearing gp when necessary will > be missed. > > This

Re: [FFmpeg-devel] [PATCH] vp9_mc_template: disable assert for SCALED == 0

2016-11-12 Thread Ronald S. Bultje
Hi Andreas, On Nov 12, 2016 5:52 PM, "Andreas Cadhalpun" < andreas.cadhal...@googlemail.com> wrote: > > The handling of the other block sizes was disabled for 'SCALED == 0' in > commit dc96c0f9fc96bf4167633befc074394062793322, so this assert should > be disabled, too, as it can now be triggered.

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Hi, On Tue, Nov 8, 2016 at 6:33 PM, compn <te...@mi.rr.com> wrote: > On Tue, 8 Nov 2016 14:44:32 -0500 > "Ronald S. Bultje" <rsbul...@gmail.com> wrote: > > > >> x86-32 linux gets stomach ache from this: > > >> > > >> YASMli

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Hi, On Tue, Nov 8, 2016 at 2:43 PM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > Hi, > > On Tue, Nov 8, 2016 at 2:39 PM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > >> On Tue, Nov 08, 2016 at 11:22:31AM -0500, Ronald S. Bultje wrote: >> >

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Hi, On Tue, Nov 8, 2016 at 2:39 PM, Michael Niedermayer <mich...@niedermayer.cc> wrote: > On Tue, Nov 08, 2016 at 11:22:31AM -0500, Ronald S. Bultje wrote: > > Also a small cosmetic change to the avx2 idct16 version to make it > > explicit that one of the arguments to

[FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Also a small cosmetic change to the avx2 idct16 version to make it explicit that one of the arguments to the write-out macros is unused for >=avx2 (it uses pmovzxbw instead of punpcklbw). --- libavcodec/x86/vp9dsp_init.c | 8 +- libavcodec/x86/vp9itxfm.asm | 188

Re: [FFmpeg-devel] [PATCH 2/3] vf_colorspace: Add support for ycgco color space

2016-11-01 Thread Ronald S. Bultje
Hi, On Sun, Oct 30, 2016 at 9:14 AM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > Hi, > > On Sun, Oct 30, 2016 at 3:07 AM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > >> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> >>

Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add support for film primaries

2016-11-01 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 2:49 PM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > Hi, > > On Mon, Oct 31, 2016 at 2:43 PM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > >> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com>

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-11-01 Thread Ronald S. Bultje
Hi, On Tue, Nov 1, 2016 at 8:09 AM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > Hi, > > On Mon, Oct 31, 2016 at 5:51 PM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > >> On Mon, Oct 31, 2016 at 4:08 PM, Ronald S. Bultje <rsbul...@gmail.com> &

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-11-01 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 5:51 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Mon, Oct 31, 2016 at 4:08 PM, Ronald S. Bultje <rsbul...@gmail.com> > wrote: > > Hi, > > > > On Mon, Oct 31, 2016 at 3:31 PM, Vittorio Giovara > > <vittor

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 3:31 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Mon, Oct 31, 2016 at 2:53 PM, Ronald S. Bultje <rsbul...@gmail.com> > wrote: > > Hi, > > > > On Mon, Oct 31, 2016 at 11:13 AM, Vittorio Giovara > > <vittor

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 11:13 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Mon, Oct 31, 2016 at 7:43 AM, Ronald S. Bultje <rsbul...@gmail.com> > wrote: > > Hi, > > > > On Mon, Oct 31, 2016 at 5:50 AM, Kevin Wheatley <

Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add support for film primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 2:43 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara > --- > This is the last easy picking, the remaining trc/prm require more > convoluted > changes to the filter that will need more discussion.

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-31 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 5:50 AM, Kevin Wheatley <kevin.j.wheat...@gmail.com> wrote: > On Sun, Oct 30, 2016 at 1:18 PM, Ronald S. Bultje <rsbul...@gmail.com> > wrote: > > Hmm... So, the wikipedia page https://en.wikipedia.org/wiki/DCI-P3 > refers > > to the t

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-30 Thread Ronald S. Bultje
Hi Kieran, On Sun, Oct 30, 2016 at 10:06 AM, Kieran O Leary wrote: > Hi, > > On Sun, Oct 30, 2016 at 7:07 AM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > > > Signed-off-by: Vittorio Giovara > > --- > > I couldn't find any

Re: [FFmpeg-devel] [PATCH 2/3] vf_colorspace: Add support for ycgco color space

2016-10-30 Thread Ronald S. Bultje
Hi, On Sun, Oct 30, 2016 at 3:07 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara > --- > This is a little hackish, not sure how to represent the matrix otherwise. Haha :) I have to admit I hadn't put much thought into

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-10-30 Thread Ronald S. Bultje
Hi, On Sun, Oct 30, 2016 at 3:07 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara > --- > I couldn't find any reference to the name of the whitepoint used for 431, > so I came up with DCI, since it looks like it is only used

Re: [FFmpeg-devel] [PATCH 1/3] vf_colorspace: Add support for iec61966-2.4 (xvYCC) transfer

2016-10-30 Thread Ronald S. Bultje
Hi, On Sun, Oct 30, 2016 at 3:07 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara > --- > As described in http://www.color.org/chardata/rgb/xvycc.xalter > Please CC. > Vittorio > > libavfilter/vf_colorspace.c | 3 +++ > 1

Re: [FFmpeg-devel] [PATCH] Remove the ffserver program and the ffm muxer/demuxer

2016-10-27 Thread Ronald S. Bultje
Hi, On Thu, Oct 27, 2016 at 2:36 PM, Reynaldo H. Verdejo Pinochet < reyna...@osg.samsung.com> wrote: > Hello > > On 10/27/2016 11:25 AM, James Almer wrote: > >> [..] >> >> This is not how things were agreed. >> >> > I haven't agreed to this. Agreement that does not have to be unanimous.

Re: [FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-26 Thread Ronald S. Bultje
Hi, On Wed, Oct 26, 2016 at 12:04 PM, wrote: > On Wed, Oct 26, 2016 at 05:08:56PM +0200, Hendrik Leppkes wrote: > > you just have to replace this stuff. > > For $100 you can buy a system at least 10 times as fast then those. > > Heck even a RPi might rival those for $20 or so.

Re: [FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-25 Thread Ronald S. Bultje
Hi, On Tue, Oct 25, 2016 at 5:12 PM, Moritz Barsnick wrote: > On Tue, Oct 25, 2016 at 14:28:38 +0200, u-9...@aetey.se wrote: > > In a perfect world the user might be offered a build time choice: > > either lose N% of the performance or take the consequences of not > >

Re: [FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-24 Thread Ronald S. Bultje
Hi, On Mon, Oct 24, 2016 at 4:26 PM, Henrik Gramner <hen...@gramner.com> wrote: > On Mon, Oct 24, 2016 at 9:59 PM, Ronald S. Bultje <rsbul...@gmail.com> > wrote: > > Good idea to reference Hendrik Gramner here, who keeps insisting we get > rid > > of all MMX code

Re: [FFmpeg-devel] [PATCH] vf_colorspace: don't spam console with warnings if range is unspecified.

2016-10-24 Thread Ronald S. Bultje
Hi, On Sat, Oct 22, 2016 at 10:02 PM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > Hi, > > On Sat, Oct 22, 2016 at 12:19 PM, Moritz Barsnick <barsn...@gmx.net> > wrote: > >> On Sat, Oct 22, 2016 at 08:43:26 -0400, Ronald S. Bultje wrote: >> > Is th

Re: [FFmpeg-devel] [PATCH] vp9: change order of operations in adapt_prob().

2016-10-24 Thread Ronald S. Bultje
Hi, On Fri, Oct 14, 2016 at 2:09 PM, James Zern <jz...@google.com> wrote: > Ronald, > > On Fri, Oct 14, 2016 at 10:01 AM, Ronald S. Bultje <rsbul...@gmail.com> > wrote: > > This is intended to workaround bug "665 Integer Divide Instruction May > > Cause

Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add support for iec61966-2.1 (sRGB) transfer

2016-10-24 Thread Ronald S. Bultje
Hi, On Thu, Oct 20, 2016 at 12:37 PM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Tue, Oct 18, 2016 at 03:17:26PM -0400, Vittorio Giovara wrote: > > Signed-off-by: Vittorio Giovara > > --- > > Special thanks to Andrew Roland for helping me figure out

Re: [FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-24 Thread Ronald S. Bultje
Hi, On Mon, Oct 24, 2016 at 3:34 PM, wm4 <nfx...@googlemail.com> wrote: > On Mon, 24 Oct 2016 21:19:46 +0200 > Andreas Cadhalpun <andreas.cadhal...@googlemail.com> wrote: > > > On 24.10.2016 16:14, Ronald S. Bultje wrote: > > > On Mon, Oct 24, 2016 at 8:47 A

Re: [FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-24 Thread Ronald S. Bultje
Hi, On Mon, Oct 24, 2016 at 8:47 AM, wm4 <nfx...@googlemail.com> wrote: > On Mon, 24 Oct 2016 07:54:47 -0400 > "Ronald S. Bultje" <rsbul...@gmail.com> wrote: > > > Hi, > > > > On Mon, Oct 24, 2016 at 3:36 AM, wm4 <nfx...@googlemail.com&

Re: [FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-23 Thread Ronald S. Bultje
Hi, On Sat, Oct 22, 2016 at 11:36 PM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Sat, Oct 22, 2016 at 10:10:01PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > general comment about all other dec patches. > > > > On Sat, Oct 22, 2016

Re: [FFmpeg-devel] [PATCH 09/13] avcodec/svq1dec: clear MMX state after MB decode loop

2016-10-22 Thread Ronald S. Bultje
Hi, general comment about all other dec patches. On Sat, Oct 22, 2016 at 3:02 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/svq1dec.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git

Re: [FFmpeg-devel] [PATCH 07/13] avcodec/vp56: Clear mmx state after mb decode loop

2016-10-22 Thread Ronald S. Bultje
Hi, On Sat, Oct 22, 2016 at 3:02 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/vp56.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/vp56.c b/libavcodec/vp56.c > index

Re: [FFmpeg-devel] [PATCH 02/13] avcodec: Clear MMX state in ff_thread_report_progress(INT_MAX)

2016-10-22 Thread Ronald S. Bultje
Hi, On Sat, Oct 22, 2016 at 3:02 PM, Michael Niedermayer wrote: > This decreases the number of FPU state violations in fate on x86-64 from > 309 to 79 > > Signed-off-by: Michael Niedermayer > --- > libavcodec/pthread_frame.c | 3 +++ >

Re: [FFmpeg-devel] [PATCH] vf_colorspace: don't spam console with warnings if range is unspecified.

2016-10-22 Thread Ronald S. Bultje
Hi, On Sat, Oct 22, 2016 at 12:19 PM, Moritz Barsnick <barsn...@gmx.net> wrote: > On Sat, Oct 22, 2016 at 08:43:26 -0400, Ronald S. Bultje wrote: > > Is there documentation on what coverity expects? > > I found this: > https://lost-contact.mit.edu/afs/cs.stanford.

Re: [FFmpeg-devel] [PATCH] vf_colorspace: don't spam console with warnings if range is unspecified.

2016-10-22 Thread Ronald S. Bultje
Hi, On Fri, Oct 21, 2016 at 4:36 PM, Moritz Barsnick <barsn...@gmx.net> wrote: > On Thu, Oct 20, 2016 at 12:06:14 -0400, Ronald S. Bultje wrote: > > +s->did_range_warn = 1; > > +} > > // fall-through > > +} > > c

Re: [FFmpeg-devel] [PATCH] lavc/libvpxenc: fix -auto-alt-ref option type

2016-10-21 Thread Ronald S. Bultje
Hi, On Thu, Oct 20, 2016 at 11:14 PM, James Zern wrote: > On Thu, Oct 20, 2016 at 8:31 AM, Kagami Hiiragi > wrote: > > vp9_cx_iface actually allows values in range [0..2]. > > This fixes ticket #5894. > > > > Signed-off-by: Kagami Hiiragi

Re: [FFmpeg-devel] [PATCH] vf_colorspace: don't spam console with warnings if range is unspecified.

2016-10-20 Thread Ronald S. Bultje
Hi, On Thu, Oct 20, 2016 at 1:08 PM, Josh de Kock <j...@itanimul.li> wrote: > On 20/10/2016 17:06, Ronald S. Bultje wrote: > >> --- >> libavfilter/vf_colorspace.c | 12 ++-- >> 1 file changed, 10 insertions(+), 2 deletions(-) >> >> diff --git a

[FFmpeg-devel] [PATCH] vf_colorspace: don't spam console with warnings if range is unspecified.

2016-10-20 Thread Ronald S. Bultje
--- libavfilter/vf_colorspace.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c index c74fe00..f64163f 100644 --- a/libavfilter/vf_colorspace.c +++ b/libavfilter/vf_colorspace.c @@ -163,6 +163,8 @@ typedef

Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add support for iec61966-2.1 (sRGB) transfer

2016-10-20 Thread Ronald S. Bultje
Hi, On Tue, Oct 18, 2016 at 3:17 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara > --- > Special thanks to Andrew Roland for helping me figure out these numbers. > Please keep me in CC. > Vittorio > >

Re: [FFmpeg-devel] Memcpy Operation Duration

2016-10-18 Thread Ronald S. Bultje
Hi Ali, On Tue, Oct 18, 2016 at 3:57 PM, Ali KIZIL wrote: > 2016-10-18 22:44 GMT+03:00 Sven C. Dack : > > > On 18/10/16 20:26, Ali KIZIL wrote: > > > >> Hi Everyone, > >> > >> Today, I was analyzing memcpy duration in FFmpeg. I noticed that it is > >>

Re: [FFmpeg-devel] [PATCHv2] vf_colorspace: Interpret unspecified color range as limited range

2016-10-17 Thread Ronald S. Bultje
Hi Vittorio, On Mon, Oct 17, 2016 at 5:28 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Mon, Sep 19, 2016 at 8:36 AM, Ronald S. Bultje <rsbul...@gmail.com> > wrote: > > Hi, > > > > On Fri, Sep 16, 2016 at 9:27 AM, Clément Bœsch <u...@pkh.me

Re: [FFmpeg-devel] comma at the end of enumerator lists

2016-10-17 Thread Ronald S. Bultje
Hi Michael, On Mon, Oct 17, 2016 at 5:23 PM, Michael Behrisch <o...@behrisch.de> wrote: > Hi Ronald, > > Am 17.10.2016 um 21:37 schrieb Ronald S. Bultje: > > Hi Michael, > > > > On Mon, Oct 17, 2016 at 3:16 PM, Michael Behrisch <o...@behrisch.de> >

Re: [FFmpeg-devel] comma at the end of enumerator lists

2016-10-17 Thread Ronald S. Bultje
Hi Michael, On Mon, Oct 17, 2016 at 3:16 PM, Michael Behrisch wrote: > Am 17.10.2016 um 15:29 schrieb Michael Niedermayer: > > On Mon, Oct 17, 2016 at 01:34:55PM +0200, wm4 wrote: > >> On Mon, 17 Oct 2016 13:09:36 +0200 > >> Michael Niedermayer wrote:

Re: [FFmpeg-devel] comma at the end of enumerator lists

2016-10-16 Thread Ronald S. Bultje
Hi, On Sun, Oct 16, 2016 at 6:11 PM, Michael Behrisch wrote: > Am 16.10.2016 um 23:24 schrieb Clément Bœsch: > > On Sun, Oct 16, 2016 at 09:46:20PM +0200, Michael Behrisch wrote: > >> Hi all, > >> first of all thanks for providing this great library. > >> > >> Today my pull

Re: [FFmpeg-devel] [PATCH] vp9: change order of operations in adapt_prob().

2016-10-14 Thread Ronald S. Bultje
> > On Fri, Oct 14, 2016 at 10:01 AM, Ronald S. Bultje <rsbul...@gmail.com> > wrote: > > > > This is intended to workaround bug "665 Integer Divide Instruction > May > > > > Cause Unpredictable Behavior" on some early AMD CPUs, which c

[FFmpeg-devel] [PATCH] vp9: change order of operations in adapt_prob().

2016-10-14 Thread Ronald S. Bultje
This is intended to workaround bug "665 Integer Divide Instruction May Cause Unpredictable Behavior" on some early AMD CPUs, which causes a div-by-zero in this codepath, such as reported in Mozilla bug #1293996. Note that this isn't guaranteed to fix the bug, since a compiler is free to reorder

Re: [FFmpeg-devel] [PATCH] Apple Quicktime fix

2016-10-12 Thread Ronald S. Bultje
Hi, On Oct 11, 2016 2:40 PM, "Alexey Eromenko" wrote: > > On Tue, Oct 11, 2016 at 8:29 PM, Josh de Kock wrote: > > On 11/10/2016 18:24, Alexey Eromenko wrote: > >> > >> --- > >> libavformat/movenc.c | 14 +- > >> 1 file changed, 9 insertions(+),

Re: [FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

2016-10-10 Thread Ronald S. Bultje
Hi, On Sun, Oct 9, 2016 at 7:32 AM, Steven Liu wrote: > 2016-09-16 15:07 GMT+08:00 Michael Niedermayer : > > > On Thu, Sep 15, 2016 at 05:53:58PM -0800, Lou Logan wrote: > > > On Wed, 7 Sep 2016 14:27:07 +0200, Michael Niedermayer wrote: > > > >

Re: [FFmpeg-devel] lurking bugs in the mmx-related assembler code (?)

2016-10-03 Thread Ronald S. Bultje
Hi, On Mon, Oct 3, 2016 at 4:11 PM, Clément Bœsch wrote: > On Mon, Oct 03, 2016 at 09:48:47PM +0200, Nicolas George wrote: > > Le duodi 12 vendémiaire, an CCXXV, u-9...@aetey.se a écrit : > > > The author of the referred code acts in his actual area of competence > > > (C libraries

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Ronald S. Bultje
Hi Rune, On Mon, Oct 3, 2016 at 10:26 AM, <u-9...@aetey.se> wrote: > On Mon, Oct 03, 2016 at 08:01:05AM -0400, Ronald S. Bultje wrote: > > > Ping on the patch: > > > The patch is unlikely to assist in fixing the issue and is likely to > cause > > further in

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Ronald S. Bultje
Hi, On Mon, Oct 3, 2016 at 8:46 AM, Carl Eugen Hoyos wrote: > 2016-10-03 13:57 GMT+02:00 Hendrik Leppkes : > > > The underlying problem is that mmx code is mixed with allocations, > > Definitely. > > > which seems like an unusual case to begin with > > I

Re: [FFmpeg-devel] lurking bugs in the mmx-related assembler code (?)

2016-10-03 Thread Ronald S. Bultje
Hi, On Sun, Oct 2, 2016 at 11:04 PM, compn <te...@mi.rr.com> wrote: > On Sun, 2 Oct 2016 08:29:22 -0400 > "Ronald S. Bultje" <rsbul...@gmail.com> wrote: > > > I also think we could contact musl developers and see what's going on > > there. We certain

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-03 Thread Ronald S. Bultje
Hi, On Mon, Oct 3, 2016 at 7:37 AM, Carl Eugen Hoyos wrote: > Hi! > > 2016-10-03 13:05 GMT+02:00 Guillaume POIRIER : > > Hello, > > > > On Mon, Oct 3, 2016 at 11:38 AM, Carl Eugen Hoyos > wrote: > >> 2016-10-03 3:52 GMT+02:00 Carl

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-02 Thread Ronald S. Bultje
Hi, On Sun, Oct 2, 2016 at 8:42 PM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > 2016-10-03 2:36 GMT+02:00 Ronald S. Bultje <rsbul...@gmail.com>: > > Hi, > > > > On Sun, Oct 2, 2016 at 7:51 PM, Carl Eugen Hoyos <ceho...@ag.or.at> > wrote: > >

Re: [FFmpeg-devel] [PATCH]doc/platform: Mention musl where x86_32 is not supported

2016-10-02 Thread Ronald S. Bultje
Hi, On Sun, Oct 2, 2016 at 7:51 PM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch adds a musl section to doc/platform. This is counter-productive... Let's work with the musl-devs and fix their libc-alternative? Ronald ___

Re: [FFmpeg-devel] lurking bugs in the mmx-related assembler code (?)

2016-10-02 Thread Ronald S. Bultje
Hi, On Sun, Oct 2, 2016 at 6:41 PM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > 2016-10-02 14:29 GMT+02:00 Ronald S. Bultje <rsbul...@gmail.com>: > > I also think we could contact musl developers and see what's > > going on there. > > I suspect I found the re

Re: [FFmpeg-devel] lurking bugs in the mmx-related assembler code (?)

2016-10-02 Thread Ronald S. Bultje
Hi, On Sun, Oct 2, 2016 at 4:24 PM, Carl Eugen Hoyos wrote: > 2016-10-02 16:01 GMT+02:00 Carl Eugen Hoyos : > > > PS: There are several unrelated build issues. > > The build issues were just seen here because I > had to add "-melf_i386" to the musl spec

Re: [FFmpeg-devel] lurking bugs in the mmx-related assembler code (?)

2016-10-02 Thread Ronald S. Bultje
Hi, On Sun, Oct 2, 2016 at 2:17 AM, wrote: > Hello Henrik, > > On Sun, Oct 02, 2016 at 01:18:29AM +0200, Henrik Gramner wrote: > > Ensuring that emms is issued before every single libc function call is > > likely problematic. > > > > What if we simply document the requirement

Re: [FFmpeg-devel] lurking bugs in the mmx-related assembler code (?)

2016-10-01 Thread Ronald S. Bultje
Hi, On Sat, Oct 1, 2016 at 4:12 PM, <u-h...@aetey.se> wrote: > Hello Ronald, > > On Sat, Oct 01, 2016 at 03:11:38PM -0400, Ronald S. Bultje wrote: > > I'm not sure we want to go down the rabbit hole of allowing any code to > be > > executed inside frame en/decodin

Re: [FFmpeg-devel] lurking bugs in the mmx-related assembler code (?)

2016-10-01 Thread Ronald S. Bultje
ibc.org/ > > On Sat, Oct 01, 2016 at 12:31:05PM -0400, Ronald S. Bultje wrote: > > > This means when malloc()/free()/... happens to be called internally, > > > it corrupts the malloc structures, in a non-straightforward ways. > > > > That's correct. > > > &

Re: [FFmpeg-devel] [PATCH] avcodec: add missing xmm/neon clobber test wrappers for the new encode API

2016-10-01 Thread Ronald S. Bultje
Hi, On Sat, Oct 1, 2016 at 12:25 PM, James Almer wrote: > Signed-off-by: James Almer > --- > configure | 4 > libavcodec/aarch64/neontest.c | 10 ++ > libavcodec/arm/neontest.c | 10 ++ >

Re: [FFmpeg-devel] lurking bugs in the mmx-related assembler code (?)

2016-10-01 Thread Ronald S. Bultje
Hi, On Sat, Oct 1, 2016 at 11:56 AM, wrote: > On Sat, Oct 01, 2016 at 05:44:13PM +0200, wm4 wrote: > > AFAIK most MMX code in FFmpeg does not run emms (i.e. keeps the FPU > > state trashed) until returning to the API user. > > This means when malloc()/free()/... happens to be

Re: [FFmpeg-devel] [PATCH/RFC]lavf/aacdec: Do not auto-detect a single frame

2016-09-24 Thread Ronald S. Bultje
Hi, On Sat, Sep 24, 2016 at 9:11 AM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > Hi! > > 2016-09-24 14:58 GMT+02:00 Ronald S. Bultje <rsbul...@gmail.com>: > > > > On Fri, Sep 23, 2016 at 6:00 PM, Carl Eugen Hoyos <ceho...@ag.or.at> > wrote: > &

Re: [FFmpeg-devel] [PATCH/RFC]lavf/aacdec: Do not auto-detect a single frame

2016-09-24 Thread Ronald S. Bultje
Hi, On Fri, Sep 23, 2016 at 6:00 PM, Carl Eugen Hoyos wrote: > Hi! > > I use this patch locally for some time. It fixes > many misdetections, a similar patch was applied to > loas detection three years ago. > An alternative would be to return "1" for two frames. Is there any

Re: [FFmpeg-devel] [PATCH] x86/h264_weight: use appropriate register size for weight parameters

2016-09-23 Thread Ronald S. Bultje
Hi, On Fri, Sep 23, 2016 at 4:21 AM, Hendrik Leppkes wrote: > Fixes trac 5579 > --- > libavcodec/x86/h264_weight.asm | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/libavcodec/x86/h264_weight.asm b/libavcodec/x86/h264_weight. > asm >

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-20 Thread Ronald S. Bultje
Hi, On Mon, Sep 19, 2016 at 9:47 PM, Lukasz Marek <lukasz.m.lu...@gmail.com> wrote: > On Sep 18, 2016 04:57, "Ronald S. Bultje" <rsbul...@gmail.com> wrote: > > > > Hi, > > > > On Sat, Sep 17, 2016 at 4:26 PM, Marton Balint <c...@passwd.hu&g

Re: [FFmpeg-devel] [PATCHv2] vf_colorspace: Interpret unspecified color range as limited range

2016-09-19 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 9:27 AM, Clément Bœsch wrote: > On Fri, Sep 16, 2016 at 03:20:49PM +0200, Vittorio Giovara wrote: > > This is the assumption that is made in pixel format conversion do > > throughout the code (in particular swscale), and BT-specifications > > mandate. >

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-17 Thread Ronald S. Bultje
Hi, On Sat, Sep 17, 2016 at 4:26 PM, Marton Balint wrote: > I think we should wait until Lukasz does the port of the opengl device to > SDL2, and do the switch after that. When do we think that'll be finished? Ronald ___

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 12:22 PM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > 2016-09-16 18:19 GMT+02:00 Ronald S. Bultje <rsbul...@gmail.com>: > > Hi, > > > > On Fri, Sep 16, 2016 at 12:16 PM, Carl Eugen Hoyos <ceffm...@gmail.com> > > wrote

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 12:16 PM, Carl Eugen Hoyos wrote: > 2016-09-16 16:07 GMT+02:00 Hendrik Leppkes : > > > You want other developers to do more work > > Of which work are you talking about? Installing libsdl1. Ronald

Re: [FFmpeg-devel] [PATCHv2] vf_colorspace: Interpret unspecified color range as limited range

2016-09-16 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 9:20 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > This is the assumption that is made in pixel format conversion do > throughout the code (in particular swscale), and BT-specifications > mandate. > > Add a warning to inform the user that an automatic

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 8:48 AM, Carl Eugen Hoyos wrote: > Do we allow developers to continue to use sdl1 to test on their > old (but supported) Linux distributions or do we forcibly stop > them from contributing here? All developers are welcome. You'll just have to

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Ronald S. Bultje
Hi folks, On Thu, Sep 15, 2016 at 1:36 PM, James Almer wrote: > The vote will end 1 week from now, simple majority wins, and > it's open only to those in the voting committee[1]. > Marton Balint, while not in the committee, should IMO also > have the chance to vote or at

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 8:01 AM, Carl Eugen Hoyos wrote: > 2016-09-16 13:32 GMT+02:00 Paul B Mahol : > > >>> That's why i started this vote, to see how many devs are > >>> effectively in favor or against and make the final decision > >>> official. > >>

Re: [FFmpeg-devel] [PATCH 1/2] vf_colorspace: Interpret unspecified color range as limited range

2016-09-15 Thread Ronald S. Bultje
Hi, On Thu, Sep 15, 2016 at 9:32 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > > On Sep 15, 2016, at 2:45 PM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > > Hi Vittorio, > > On Thu, Sep 15, 2016 at 8:33 AM, Vittorio Giovara < > vittorio.giov.

Re: [FFmpeg-devel] [PATCH 1/2] vf_colorspace: Interpret unspecified color range as limited range

2016-09-15 Thread Ronald S. Bultje
Hi Vittorio, On Thu, Sep 15, 2016 at 8:33 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > I agree we should always be very very careful about automatic assignments, > although this one seems pretty safe. > > I don't think it's necessary, but if you prefer I can add a warning >

Re: [FFmpeg-devel] [PATCH 1/2] vf_colorspace: Interpret unspecified color range as limited range

2016-09-15 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 5:09 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > This is the assumption that is made in pixel format conversion do > throughout the code (in particular swscale and vf_colormatrix). > > Signed-off-by: Vittorio Giovara > ---

Re: [FFmpeg-devel] [PATCH 2/2] vf_colorspace: Add modern names for color range option

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 5:09 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Allows to use values returned from API and from ffprobe directly. > > Signed-off-by: Vittorio Giovara > --- > libavfilter/vf_colorspace.c | 2 ++ > 1 file changed, 2

Re: [FFmpeg-devel] [PATCH]lavf/riff: Do not use a rogue twocc for adpcm_swf

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 1:46 PM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > 2016-09-14 19:21 GMT+02:00 Ronald S. Bultje <rsbul...@gmail.com>: > > [...] > > >> I wondered if somebody can easily fix muxing / demuxing which > >> could be con

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 12:18 PM, Carl Eugen Hoyos wrote: > 2016-09-14 18:15 GMT+02:00 James Almer : > > > Configure failed to detect it > > configure requires 2.0.1 > > Carl Eugen (who wonders how Ronald tests...) ffmpeg (the tool), of course! :)

Re: [FFmpeg-devel] [PATCH]lavf/riff: Do not use a rogue twocc for adpcm_swf

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 12:29 PM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > Hi! > > 2016-09-14 18:05 GMT+02:00 Ronald S. Bultje <rsbul...@gmail.com>: > > > > On Wed, Sep 14, 2016 at 12:01 PM, Carl Eugen Hoyos wrote: > > > >> 2016-09-12 12

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 12:04 PM, Carl Eugen Hoyos wrote: > 2016-09-14 17:59 GMT+02:00 Michael Niedermayer : > > On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: > >> 2016-09-12 20:58 GMT+02:00 Josh de Kock : >

Re: [FFmpeg-devel] [PATCH]lavf/riff: Do not use a rogue twocc for adpcm_swf

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 12:01 PM, Carl Eugen Hoyos wrote: > 2016-09-12 12:52 GMT+02:00 Carl Eugen Hoyos : > > > Attached patch "fixes" ticket #5829, I am happy if a better > > solution can be found. > > Ping. What makes you believe this is not the

Re: [FFmpeg-devel] [VDD2016] Technical Discussion Results

2016-09-13 Thread Ronald S. Bultje
Hi, On Mon, Sep 12, 2016 at 6:01 AM, Paul B Mahol wrote: > On 9/12/16, Thilo Borgmann wrote: > > Hi, > > > > these are the resulting notes we've taken during the FFmpeg technical > > discussion > > during VDD 2016. We've ended up with two slides on the

Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add BT-names for gamma22/28 transfer option

2016-09-13 Thread Ronald S. Bultje
Hi, On Mon, Sep 12, 2016 at 1:46 PM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > Hi, > > On Mon, Sep 12, 2016 at 1:44 PM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > >> Allows to use values returned from API and from ffprobe directly. >

Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add BT-names for gamma22/28 transfer option

2016-09-12 Thread Ronald S. Bultje
Hi, On Mon, Sep 12, 2016 at 1:44 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > Allows to use values returned from API and from ffprobe directly. > > Signed-off-by: Vittorio Giovara > --- > libavfilter/vf_colorspace.c | 2 ++ > 1 file changed, 2

Re: [FFmpeg-devel] questions about multi-thread issues for ffmpeg

2016-09-12 Thread Ronald S. Bultje
Hi, On Mon, Sep 12, 2016 at 4:30 AM, qw wrote: > hi Ronald, > > > >does ffmpeg support multi-thread in this case? If not, which ffmpeg > >> functions should be synchronized by application developer? > > > > > >Yes, ffmpeg supports multi-threading (internally), it is tested >

Re: [FFmpeg-devel] [PATCH] avformat/mov: Enable stream parsing for VP9.

2016-09-08 Thread Ronald S. Bultje
Hi, On Wed, Sep 7, 2016 at 7:37 AM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > Hi, > > On Tue, Sep 6, 2016 at 10:39 PM, Matthew Gregan <kine...@flim.org> wrote: > >> At 2016-09-06T22:18:18-0400, Ronald S. Bultje wrote: >> > I think the patch is fine, bu

Re: [FFmpeg-devel] [PATCH 1/2] Parse VP9 when coming out of MP4.

2016-09-08 Thread Ronald S. Bultje
Hi, On Thu, Sep 8, 2016 at 9:50 PM, James Almer wrote: > On 9/8/2016 8:41 PM, Alex Converse wrote: > > Fixes Netflix VP9 DASH samples. > > --- > > libavformat/mov.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/libavformat/mov.c b/libavformat/mov.c > > index

Re: [FFmpeg-devel] questions about multi-thread issues for ffmpeg

2016-09-08 Thread Ronald S. Bultje
Hi Andrew, On Thu, Sep 8, 2016 at 3:59 AM, qw wrote: > I have made one simple transcoding program by using ffmpeg lib, where > several av transcoding tasks are done in one process and each task > corresponds to one thread. > > But some ffmpeg function will report error, and

Re: [FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

2016-09-07 Thread Ronald S. Bultje
Hi, On Wed, Sep 7, 2016 at 7:51 AM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > doc/developer.texi | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/doc/developer.texi b/doc/developer.texi > index

Re: [FFmpeg-devel] [PATCH] avformat/mov: Enable stream parsing for VP9.

2016-09-07 Thread Ronald S. Bultje
Hi, On Tue, Sep 6, 2016 at 10:39 PM, Matthew Gregan <kine...@flim.org> wrote: > At 2016-09-06T22:18:18-0400, Ronald S. Bultje wrote: > > I think the patch is fine, but I wonder if it should set it to _FULL (the > > parser ignores the option, but it is semantically more cor

Re: [FFmpeg-devel] [PATCH] avformat/mov: Enable header parsing for VP9.

2016-09-06 Thread Ronald S. Bultje
Hi Matthew, On Tue, Sep 6, 2016 at 10:10 PM, Matthew Gregan wrote: > Hi, > > The attached patch fixes playback of MP4/VP9 media containing superframes > when using the built-in VP9 decoder. > > As an example, >

Re: [FFmpeg-devel] [PATCH] avformat: Allow forcing use of AVParsers

2016-09-05 Thread Ronald S. Bultje
Hi, On Mon, Sep 5, 2016 at 7:08 AM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: > On Mon, Sep 5, 2016 at 12:17 PM, Ronald S. Bultje <rsbul...@gmail.com> > wrote: > > It was proposed that we could go one step further and do parsing > > not in the demuxer step (av

Re: [FFmpeg-devel] [PATCH] avformat: Allow forcing use of AVParsers

2016-09-05 Thread Ronald S. Bultje
Hi, On Mon, Sep 5, 2016 at 6:52 AM, Paul B Mahol <one...@gmail.com> wrote: > On 9/5/16, Ronald S. Bultje <rsbul...@gmail.com> wrote: > > Hi, > > > > On Sep 5, 2016 12:05 PM, "Michael Niedermayer" <mich...@niedermayer.cc> > > wrote: >

Re: [FFmpeg-devel] [PATCH] avformat: Allow forcing use of AVParsers

2016-09-05 Thread Ronald S. Bultje
Hi, On Mon, Sep 5, 2016 at 6:49 AM, Michael Niedermayer <mich...@niedermayer.cc> wrote: > On Mon, Sep 05, 2016 at 06:17:02AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Sep 5, 2016 12:05 PM, "Michael Niedermayer" <mich...@niedermayer.cc> > >

Re: [FFmpeg-devel] [PATCH] avformat: Allow forcing use of AVParsers

2016-09-05 Thread Ronald S. Bultje
Hi, On Sep 5, 2016 12:05 PM, "Michael Niedermayer" <mich...@niedermayer.cc> wrote: > > On Mon, Sep 05, 2016 at 05:25:14AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Sep 5, 2016 11:18 AM, "Michael Niedermayer" <mich...@niedermaye

Re: [FFmpeg-devel] [PATCH] avformat: Allow forcing use of AVParsers

2016-09-05 Thread Ronald S. Bultje
Hi, On Sep 5, 2016 11:18 AM, "Michael Niedermayer" wrote: > > TODO: version bump, docs > > Signed-off-by: Michael Niedermayer > --- > libavformat/avformat.h | 8 > libavformat/options_table.h | 1 + > libavformat/utils.c | 3

Re: [FFmpeg-devel] order T-shirts

2016-09-04 Thread Ronald S. Bultje
Hi, On Sun, Sep 4, 2016 at 5:34 PM, Thomas Volkert wrote: > Hi, > > Some guys at the VDD asked for FFmpeg T-shirts. > I'd like to do a new T-shirt order. The shirts could be given to > multimedia devs who stop at one of our next booths. > > My idea is to order 25 shirts: > > 1*S

Re: [FFmpeg-devel] [PATCH v2] swscale: add unscaled conversion from yuv420p to p010

2016-09-04 Thread Ronald S. Bultje
Hi, On Sep 4, 2016 5:42 PM, "Timo Rothenpieler" wrote: > > On 9/4/2016 4:06 PM, Carl Eugen Hoyos wrote: > > 2016-09-04 16:02 GMT+02:00 Timo Rothenpieler : > >> The purpose of this patch is to make conversion from > >> yuv420p (8 bit) to p010 (10 bit)

Re: [FFmpeg-devel] [PATCH v2] swscale: add unscaled conversion from yuv420p to p010

2016-09-04 Thread Ronald S. Bultje
Hi, On Sun, Sep 4, 2016 at 10:06 AM, Carl Eugen Hoyos wrote: > 2016-09-04 16:02 GMT+02:00 Timo Rothenpieler : > > The purpose of this patch is to make conversion from > > yuv420p (8 bit) to p010 (10 bit) fast. > > Do I understand you correctly that

Re: [FFmpeg-devel] Performance of P010LE/BE pixel convertion

2016-09-01 Thread Ronald S. Bultje
Hi Timo, On Thu, Sep 1, 2016 at 7:59 AM, Timo Rothenpieler <t...@rothenpieler.org> wrote: > Am 01.09.2016 um 13:44 schrieb Ronald S. Bultje: > > Hi Timo, > > > > On Thu, Sep 1, 2016 at 7:34 AM, Timo Rothenpieler <t...@rothenpieler.org > > > > wrote: >

<    4   5   6   7   8   9   10   11   12   13   >