Hi,
2015-10-14 0:04 GMT+02:00 Michael Niedermayer :
> the ome and syserr values worsen by this
I have mixed feelings about this, too.
On the one hand, omse in any case says anyway none of those idcts are
accurate enough to some sense (inter error propagation? / per the mpeg
specs), as there will
2015-10-14 12:48 GMT+08:00 Timothy Gu :
> All other classes use lowercase names.
> ---
> libavutil/opencl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavutil/opencl.c b/libavutil/opencl.c
> index f720ce0..62706f3 100644
> --- a/libavutil/opencl.c
> +++ b/libavutil
Hi
A simple idea is that
1. Check whether the device support cl_khr_fp64
2. Set a macro in build option. you can reference the unsharp filter,
the code is as follow:
snprintf(build_opts, 96, "-D LU_RADIUS_X=%d -D LU_RADIUS_Y=%d -D
CH_RADIUS_X=%d -D CH_RADIUS_Y=%d",
2*unsharp->luma.
2015-10-14 14:31 GMT+08:00 Timo Rothenpieler :
> Hi
>>
>> I have check the error, for some card, yes, just like what I said above,
>> some cards does not support double type, so we must check the opencl
>> extension: cl_khr_fp64 before the kernel compile. If the device support,
>> we use GPU, else
Ganesh Ajjanagadde mit.edu> writes:
> >> I think the general case, it'd be nice to figure out
> >> why Carl's results are slightly different from yours
> >
> > Why do you think they are different at all?
> > Did you look at the tables?
>
> They are different, and our conclusions are different
>
Hi
I have check the error, for some card, yes, just like what I said above,
some cards does not support double type, so we must check the opencl
extension: cl_khr_fp64 before the kernel compile. If the device support,
we use GPU, else, we use CPU or use float type, Is it OK?
Thanks
Best regards
---
libavutil/fifo.c| 55 +
libavutil/fifo.h| 11 +++
tests/ref/fate/fifo | 27 ++
3 files changed, 93 insertions(+)
diff --git a/libavutil/fifo.c b/libavutil/fifo.c
index 07fb4ec..a7da591 100644
--- a/lib
On Tue, Oct 13, 2015 at 6:24 PM Ganesh Ajjanagadde wrote:
> On Tue, Oct 13, 2015 at 9:20 PM, Timothy Gu wrote:
> > Patch LGTM.
>
> Go ahead and push
Pushed, thanks.
Timothy
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mail
Silences warnings regarding `clCreateCommandQueue` being deprecated.
Only a very limited number of products support 2.0. Since the
replacement API (`clCreateCommandQueueWithProperties`) is only available
in 2.0, we should not update it just yet.
---
libavutil/opencl.h | 1 +
1 file changed, 1 ins
All other classes use lowercase names.
---
libavutil/opencl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavutil/opencl.c b/libavutil/opencl.c
index f720ce0..62706f3 100644
--- a/libavutil/opencl.c
+++ b/libavutil/opencl.c
@@ -84,7 +84,7 @@ static const AVOption opencl_o
Useful when debugging.
---
libavutil/opencl.c | 33 ++---
1 file changed, 30 insertions(+), 3 deletions(-)
diff --git a/libavutil/opencl.c b/libavutil/opencl.c
index c2956fd..f720ce0 100644
--- a/libavutil/opencl.c
+++ b/libavutil/opencl.c
@@ -449,12 +449,14 @@ end:
c
---
libavutil/opencl.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavutil/opencl.c b/libavutil/opencl.c
index 8686493..c2956fd 100644
--- a/libavutil/opencl.c
+++ b/libavutil/opencl.c
@@ -484,7 +484,8 @@ cl_program av_opencl_compile(const char *program_name,
const cha
---
libavfilter/vf_chromakey.c | 25 +
1 file changed, 17 insertions(+), 8 deletions(-)
diff --git a/libavfilter/vf_chromakey.c b/libavfilter/vf_chromakey.c
index 47fdea631..3309748 100644
--- a/libavfilter/vf_chromakey.c
+++ b/libavfilter/vf_chromakey.c
@@ -35,6 +35,9 @@
On Tue, Oct 13, 2015, at 03:55 PM, Ganesh Ajjanagadde wrote:
> ping
LGTM & pushed. Thanks.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
On Tue, Oct 13, 2015 at 11:15 PM, Carl Eugen Hoyos wrote:
> Ronald S. Bultje gmail.com> writes:
>
>> I think the general case, it'd be nice to figure out
>> why Carl's results are slightly different from yours
>
> Why do you think they are different at all?
> Did you look at the tables?
They are
Ronald S. Bultje gmail.com> writes:
> I think the general case, it'd be nice to figure out
> why Carl's results are slightly different from yours
Why do you think they are different at all?
Did you look at the tables?
Carl Eugen
___
ffmpeg-devel mai
Another attempt at trac #4924: use cat->avf->start_time to setup opened
file structure in concatdec.c. This field always contains the correct
timestamp.
---
libavformat/concatdec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/concatdec.c b/libavformat/concatd
On Tue, Oct 13, 2015 at 3:28 AM Timo Rothenpieler
wrote:
> > Hi
> >
> > I use your filter, but the kernel can't pass the compile, you should
> consider the "double" type in the kernel, some GPU card does not support
> double type
> > I add "#pragma OPENCL_EXTENSION cl_khr_fp64: enable " to the ke
2015-10-13 18:27 GMT+08:00 Timo Rothenpieler :
> Hi
>>
>> I use your filter, but the kernel can't pass the compile, you should
>> consider the "double" type in the kernel, some GPU card does not support
>> double type
>> I add "#pragma OPENCL_EXTENSION cl_khr_fp64: enable " to the kernel, but
>> i
On Tue, Oct 13, 2015 at 9:20 PM, Timothy Gu wrote:
> El martes, 13 de octubre de 2015, Michael Niedermayer
> escribió:
>
>> On Sun, Oct 11, 2015 at 04:27:49PM -0400, Ganesh Ajjanagadde wrote:
>> > ---
>> > .gitignore | 1 +
>> > 1 file changed, 1 insertion(+)
>> >
>> > diff --git a/.gitignore b/
El martes, 13 de octubre de 2015, Michael Niedermayer
escribió:
> On Sun, Oct 11, 2015 at 04:27:49PM -0400, Ganesh Ajjanagadde wrote:
> > ---
> > .gitignore | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/.gitignore b/.gitignore
> > index 0b26f68..b215828 100644
> > --- a/.gitigno
On Tue, Oct 13, 2015 at 8:58 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Tue, Oct 13, 2015 at 8:09 PM, Ganesh Ajjanagadde
> wrote:
>
>> On Tue, Oct 13, 2015 at 2:45 AM, Clément Bœsch wrote:
>> > On Tue, Oct 13, 2015 at 12:31:10AM -0400, Ganesh Ajjanagadde wrote:
>> >> On Tue, Oct 13, 2015 at 12:26
Hi,
On Tue, Oct 13, 2015 at 8:09 PM, Ganesh Ajjanagadde
wrote:
> On Tue, Oct 13, 2015 at 2:45 AM, Clément Bœsch wrote:
> > On Tue, Oct 13, 2015 at 12:31:10AM -0400, Ganesh Ajjanagadde wrote:
> >> On Tue, Oct 13, 2015 at 12:26 AM, Ganesh Ajjanagadde
> wrote:
> >> > On Tue, Oct 13, 2015 at 12:16
On Tue, Oct 13, 2015 at 01:19:59PM +0200, Carl Eugen Hoyos wrote:
> On Monday 05 October 2015 03:53:32 am Michael Niedermayer wrote:
> > On Sun, Oct 04, 2015 at 11:47:59PM +0200, Carl Eugen Hoyos wrote:
> > > Hi!
> > >
> > > Attached is a patch implementing vc-1 autodetection.
> > >
> > > Please re
On Tue, Oct 13, 2015 at 8:31 PM, Ganesh Ajjanagadde wrote:
> On Tue, Oct 13, 2015 at 8:13 PM, Michael Niedermayer
> wrote:
>> On Sun, Oct 11, 2015 at 04:27:49PM -0400, Ganesh Ajjanagadde wrote:
>>> ---
>>> .gitignore | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/.gitignore b/.gi
Hi all,
Attached is a new version reflecting all freedback received so far.
Thanks,
Ganesh
From a218d8274498f67cfbaadc1240dbf2cfe7b20dee Mon Sep 17 00:00:00 2001
From: Ganesh Ajjanagadde
Date: Sun, 4 Oct 2015 23:39:25 -0400
Subject: [PATCHv3 1/2] avfilter/all: propagate errors of functions from
On Tue, Oct 13, 2015 at 8:13 PM, Michael Niedermayer
wrote:
> On Sun, Oct 11, 2015 at 04:27:49PM -0400, Ganesh Ajjanagadde wrote:
>> ---
>> .gitignore | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/.gitignore b/.gitignore
>> index 0b26f68..b215828 100644
>> --- a/.gitignore
>> +++ b/
On Sun, Oct 11, 2015 at 04:27:49PM -0400, Ganesh Ajjanagadde wrote:
> ---
> .gitignore | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/.gitignore b/.gitignore
> index 0b26f68..b215828 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -3,3 +3,4 @@
> /htdocs/main.rss
> /htdocs/components
On Tue, Oct 13, 2015 at 2:45 AM, Clément Bœsch wrote:
> On Tue, Oct 13, 2015 at 12:31:10AM -0400, Ganesh Ajjanagadde wrote:
>> On Tue, Oct 13, 2015 at 12:26 AM, Ganesh Ajjanagadde
>> wrote:
>> > On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos
>> > wrote:
>> >> Ganesh Ajjanagadde mit.edu> w
On Wed, Oct 14, 2015 at 01:50:13AM +0200, Andreas Cadhalpun wrote:
> It was replaced by avpriv_ac3_parse_header2.
>
> Signed-off-by: Andreas Cadhalpun
> ---
> libavcodec/ac3_parser.c | 9 -
> libavcodec/ac3_parser.h | 2 --
> 2 files changed, 11 deletions(-)
should be ok
[...]
--
Mich
On 10/13/2015 8:48 PM, Andreas Cadhalpun wrote:
> Also replace the last two usages of avpriv_float_dsp_init with
> avpriv_float_dsp_alloc.
>
> Signed-off-by: Andreas Cadhalpun
> ---
> libavutil/float_dsp.c | 47 ++-
> libavutil/float_dsp.h | 9 ---
On Tue, Oct 13, 2015 at 12:10 PM, Ganesh Ajjanagadde
wrote:
> ---
> src/about | 6 +-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/src/about b/src/about
> index 8d8f606..67342a1 100644
> --- a/src/about
> +++ b/src/about
> @@ -4,7 +4,11 @@
> transcode, mux, demux,
>
On Sun, Oct 11, 2015 at 4:27 PM, Ganesh Ajjanagadde
wrote:
> ---
> .gitignore | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/.gitignore b/.gitignore
> index 0b26f68..b215828 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -3,3 +3,4 @@
> /htdocs/main.rss
> /htdocs/components
> /htdo
On 10/13/2015 8:50 PM, Andreas Cadhalpun wrote:
> It was replaced by avpriv_ac3_parse_header2.
You could rename it to avpriv_ac3_parse_header() while at it.
The 2 suffix becomes silly with this.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http:
On Tue, Oct 13, 2015 at 12:54 PM, Moritz Barsnick wrote:
> On Wed, Oct 07, 2015 at 23:35:08 +0200, Moritz Barsnick wrote:
>> On Wed, Oct 07, 2015 at 16:07:53 -0400, Ronald S. Bultje wrote:
>> > > BTW, are the "remaining" (meaning: more than from gcc) warnings emitted
>> > > by the Intel C(++) Comp
It is only used inside libavcodec.
Signed-off-by: Andreas Cadhalpun
---
libavcodec/h264_slice.c | 2 +-
libavcodec/internal.h | 2 +-
libavcodec/utils.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c
index a346ccb..
It was replaced by avpriv_ac3_parse_header2.
Signed-off-by: Andreas Cadhalpun
---
libavcodec/ac3_parser.c | 9 -
libavcodec/ac3_parser.h | 2 --
2 files changed, 11 deletions(-)
diff --git a/libavcodec/ac3_parser.c b/libavcodec/ac3_parser.c
index 5ab5627..0dd98b9 100644
--- a/libavcodec
On Tue, Oct 13, 2015 at 1:49 PM, Michael Niedermayer
wrote:
> On Tue, Oct 13, 2015 at 12:26:24PM -0400, Ganesh Ajjanagadde wrote:
>> Partially fixes Ticket 4727.
>>
>> -duration is not a safe expression, since duration can be INT_MIN.
>> One might ask how it can become INT_MIN.
>> Although it is t
Also replace the last two usages of avpriv_float_dsp_init with
avpriv_float_dsp_alloc.
Signed-off-by: Andreas Cadhalpun
---
libavutil/float_dsp.c | 47 ++-
libavutil/float_dsp.h | 9 -
2 files changed, 26 insertions(+), 30 deletions(-)
diff -
On Tue, Oct 13, 2015 at 3:45 PM, Michael Niedermayer
wrote:
> On Fri, Oct 09, 2015 at 11:31:22AM -0400, Ganesh Ajjanagadde wrote:
>> FFABS(INT_MIN) is not safe. Alternative of using FFNABS is not as
>> readable.
>>
>> Signed-off-by: Ganesh Ajjanagadde
>> ---
>> ffserver_config.c | 2 +-
>> 1 fil
On Tue, Oct 13, 2015 at 9:37 AM, compn wrote:
> On Tue, 13 Oct 2015 09:18:35 -0400
> Ganesh Ajjanagadde wrote:
>
>> On Tue, Oct 13, 2015 at 9:13 AM, compn wrote:
>> > On Mon, 12 Oct 2015 15:10:22 -0700
>> > "Reynaldo H. Verdejo Pinochet" wrote:
>> >
>> >> Signed-off-by: Reynaldo H. Verdejo Pino
On 10/13/2015 7:37 PM, Andreas Cadhalpun wrote:
> These headers contain functions supposed to be public.
>
> libavutil/des.h:
> av_des_alloc
> av_des_crypt
> av_des_init
> av_des_mac
> libavutil/rc4.h:
> av_rc4_alloc
> av_rc4_crypt
> av_rc4_init
Since these two were never installed, we can
On 10/13/2015 8:10 PM, Henrik Gramner wrote:
> On Wed, Oct 14, 2015 at 1:04 AM, James Almer wrote:
>> Signed-off-by: James Almer
>> ---
>> libavcodec/x86/vp9itxfm_16bpp.asm | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavcodec/x86/vp9itxfm_16bpp.asm
>> b/libavco
On Tue, 13 Oct 2015, Bodecs Bela wrote:
Dear Marton Balint,
2015.10.12. 23:07 keltezéssel, Marton Balint írta:
On Mon, 12 Oct 2015, Bodecs Bela wrote:
Dear All,
currently stream specifiers may contain stream index, stream type,
stream id, program id, metadata key/value es usable config se
On Wed, Oct 14, 2015 at 1:04 AM, James Almer wrote:
> Signed-off-by: James Almer
> ---
> libavcodec/x86/vp9itxfm_16bpp.asm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/x86/vp9itxfm_16bpp.asm
> b/libavcodec/x86/vp9itxfm_16bpp.asm
> index 06b39ff..902685e 10
Signed-off-by: James Almer
---
libavcodec/x86/vp9itxfm_16bpp.asm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/x86/vp9itxfm_16bpp.asm
b/libavcodec/x86/vp9itxfm_16bpp.asm
index 06b39ff..902685e 100644
--- a/libavcodec/x86/vp9itxfm_16bpp.asm
+++ b/libavcodec/x86/vp
On Tue, Oct 13, 2015 at 04:41:50PM -0300, James Almer wrote:
> On 10/12/2015 11:04 PM, Michael Niedermayer wrote:
> > On Tue, Oct 13, 2015 at 12:00:07AM +0200, Michael Niedermayer wrote:
> >> On Mon, Oct 12, 2015 at 10:39:23PM +0200, Christophe Gisquet wrote:
> >>> Le 12 oct. 2015 21:37, "Michael N
On 13.10.2015 04:21, Michael Niedermayer wrote:
> On Mon, Oct 12, 2015 at 08:15:10PM +0200, Andreas Cadhalpun wrote:
>> It seems several headers are intended to be installed, but actually aren't:
>
>> libavutil/des.h:
>> av_des_alloc
>> av_des_crypt
>> av_des_init
>> av_des_mac
>> libavutil/rc
On Tue, Oct 13, 2015 at 9:12 AM, Ganesh Ajjanagadde wrote:
> On Tue, Oct 13, 2015 at 4:02 AM, Clément Bœsch wrote:
>> On Tue, Oct 13, 2015 at 09:25:03AM +0200, Paul B Mahol wrote:
>> [...]
>>> What about fmax/FFMAX?
>>
>> Feel free to try that out (it looks OT regarding the patch), but fmax()
>>
On Tue, Oct 13, 2015 at 09:21:41PM +0200, Christophe Gisquet wrote:
> Allows testing simple_idct12 correctness/bitexactness, as the sample
> was generated using faani as idct.
> ---
> tests/fate/image.mak | 6 ++
> tests/ref/fate/jpg-12bpp | 2 ++
> 2 files changed, 8 insertions(+)
> crea
These headers contain functions supposed to be public.
libavutil/des.h:
av_des_alloc
av_des_crypt
av_des_init
av_des_mac
libavutil/rc4.h:
av_rc4_alloc
av_rc4_crypt
av_rc4_init
libavutil/tree.h
av_tree_destroy
av_tree_enumerate
av_tree_find
av_tree_insert
av_tree_node_alloc
av_tree_nod
On Tue, Oct 13, 2015 at 6:16 PM, Henrik Gramner wrote:
> On Tue, Oct 13, 2015 at 11:58 PM, Ganesh Ajjanagadde wrote:
>> On Tue, Oct 13, 2015 at 5:55 PM, Henrik Gramner wrote:
>>> On Tue, Oct 13, 2015 at 11:28 PM, Ganesh Ajjanagadde
>>> wrote:
Not really that important, but unless this inc
On Tue, Oct 13, 2015 at 11:58 PM, Ganesh Ajjanagadde wrote:
> On Tue, Oct 13, 2015 at 5:55 PM, Henrik Gramner wrote:
>> On Tue, Oct 13, 2015 at 11:28 PM, Ganesh Ajjanagadde
>> wrote:
>>> Not really that important, but unless this increases FATE time
>>> significantly, I would recommend a much l
On Tue, Oct 13, 2015 at 6:04 PM, Michael Niedermayer
wrote:
> On Tue, Oct 13, 2015 at 09:21:40PM +0200, Christophe Gisquet wrote:
>> Results for omse on the 3 idct dct-test.
>>
>> C: 0.16915859 0.11848359 0.12913125
>> x86: 0.16883281 0.11849063 0.19041875
>>
>> Using 14 and 17 as shifts
On Tue, Oct 13, 2015 at 09:21:40PM +0200, Christophe Gisquet wrote:
> Results for omse on the 3 idct dct-test.
>
> C: 0.16915859 0.11848359 0.12913125
> x86: 0.16883281 0.11849063 0.19041875
>
> Using 14 and 17 as shifts subtantially improve those, but actually
> cause overflows and inc
On Tue, Oct 13, 2015 at 5:55 PM, Henrik Gramner wrote:
> On Tue, Oct 13, 2015 at 11:28 PM, Ganesh Ajjanagadde wrote:
>> Not really that important, but unless this increases FATE time
>> significantly, I would recommend a much larger MAX_COUNT, and multiple
>> iterations (e.g #define an ITER_COUNT
On Tue, Oct 13, 2015 at 11:28 PM, Ganesh Ajjanagadde wrote:
> Not really that important, but unless this increases FATE time
> significantly, I would recommend a much larger MAX_COUNT, and multiple
> iterations (e.g #define an ITER_COUNT), so that the testing is more
> extensive over random inputs
On 10/13/2015 5:06 PM, Christophe Gisquet wrote:
> Hi,
>
> 2015-10-13 21:41 GMT+02:00 James Almer :
>> > This test is failing on pretty much every fate client. Valgrind seems to
>> > complain about uninitialized values.
>> > http://fate.ffmpeg.org/report.cgi?time=20151013040721&slot=x86_64-archlin
Fixes ticket #4924.
Found-by: Jaroslav Šnajdr
Signed-off-by: Marton Balint
---
libavformat/concatdec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c
index 832b7f4..7686f28 100644
--- a/libavformat/concatdec.c
+++ b/libavfor
On Tue, Oct 13, 2015 at 5:06 PM, Henrik Gramner wrote:
> Added some randomization to count and made report() less verbose:
>
> diff --git a/tests/checkasm/aes.c b/tests/checkasm/aes.c
> index 67b..4549a3d 100644
> --- a/tests/checkasm/aes.c
> +++ b/tests/checkasm/aes.c
> @@ -23,12 +23,14 @@
>
Added some randomization to count and made report() less verbose:
diff --git a/tests/checkasm/aes.c b/tests/checkasm/aes.c
index 67b..4549a3d 100644
--- a/tests/checkasm/aes.c
+++ b/tests/checkasm/aes.c
@@ -23,12 +23,14 @@
#include "libavutil/aes_internal.h"
#include "libavutil/internal.h"
On Tue, Oct 13, 2015 at 04:10:15PM -0400, Ronald S. Bultje wrote:
> Hi,
>
> On Tue, Oct 13, 2015 at 3:21 PM, Christophe Gisquet <
> christophe.gisq...@gmail.com> wrote:
>
> > This avoid going through constants.c while still sharing them
> > with proresdsp.asm
> > ---
> > libavcodec/x86/constants
I changed the asm a bit and made it about 1 cycle faster on Haswell
and slightly smaller (-48 bytes overall incl. alignment on 64-bit
Linux).
%macro AES_CRYPT 1
cglobal aes_%1rypt, 6,6,2
shl r3d, 4
add r5d, r5d
add r0, 0x60
add r2, r3
add r1, r3
Hi,
On Tue, Oct 13, 2015 at 3:21 PM, Christophe Gisquet <
christophe.gisq...@gmail.com> wrote:
> This avoid going through constants.c while still sharing them
> with proresdsp.asm
> ---
> libavcodec/x86/constants.c| 28
>
> libavcodec/x86/constants.h
Hi,
2015-10-13 21:41 GMT+02:00 James Almer :
> This test is failing on pretty much every fate client. Valgrind seems to
> complain about uninitialized values.
> http://fate.ffmpeg.org/report.cgi?time=20151013040721&slot=x86_64-archlinux-gcc-valgrindundef
Can someone test the following patch? I do
On Fri, Oct 09, 2015 at 11:31:22AM -0400, Ganesh Ajjanagadde wrote:
> FFABS(INT_MIN) is not safe. Alternative of using FFNABS is not as
> readable.
>
> Signed-off-by: Ganesh Ajjanagadde
> ---
> ffserver_config.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
LGTM
thanks
[...]
--
Mic
On 10/12/2015 11:04 PM, Michael Niedermayer wrote:
> On Tue, Oct 13, 2015 at 12:00:07AM +0200, Michael Niedermayer wrote:
>> On Mon, Oct 12, 2015 at 10:39:23PM +0200, Christophe Gisquet wrote:
>>> Le 12 oct. 2015 21:37, "Michael Niedermayer" a
>>> écrit :
On Mon, Oct 12, 2015 at 07:37:43
While I'm at it, I've tested the 8-bits version, with the following
omse results:
SIMPLE: 0.00851563 0.00675937 0.00273906
xvid: 0.00917266 0.01386797 0.00784922
SIMPLE8: 0.00756641 0.00654844 0.
I don't think having yet another idct8x8 is so nice, so I'm not going
to submit a
First patch addresses an issue raised by Donald.
Second is one forgotten during the patchset iterations
Third requires sample from ticket #4683, whose license isn't clear,
all the more since it has a watermark indicating an unlicensed s/w.
Having a 12bpp sample of that nature has been however very
Results for omse on the 3 idct dct-test.
C: 0.16915859 0.11848359 0.12913125
x86: 0.16883281 0.11849063 0.19041875
Using 14 and 17 as shifts subtantially improve those, but actually
cause overflows and incorrect decoding of 12bpp content.
---
libavcodec/simple_idct_template.c | 17
Allows testing simple_idct12 correctness/bitexactness, as the sample
was generated using faani as idct.
---
tests/fate/image.mak | 6 ++
tests/ref/fate/jpg-12bpp | 2 ++
2 files changed, 8 insertions(+)
create mode 100644 tests/ref/fate/jpg-12bpp
diff --git a/tests/fate/image.mak b/tests
This avoid going through constants.c while still sharing them
with proresdsp.asm
---
libavcodec/x86/constants.c| 28
libavcodec/x86/constants.h| 16
libavcodec/x86/proresdsp.asm | 13 +
libavcode
On Tue, Oct 13, 2015 at 03:07:36PM +0200, Paul B Mahol wrote:
> Signed-off-by: Paul B Mahol
> ---
> libavformat/Makefile | 1 +
> libavformat/allformats.c | 1 +
> libavformat/vag.c| 78
>
> 3 files changed, 80 insertions(+)
> creat
On Tue, Oct 13, 2015 at 12:26:24PM -0400, Ganesh Ajjanagadde wrote:
> Partially fixes Ticket 4727.
>
> -duration is not a safe expression, since duration can be INT_MIN.
> One might ask how it can become INT_MIN.
> Although it is true that line 2574 is no longer reached with INT_MIN due
> to commi
On Wed, Oct 07, 2015 at 23:35:08 +0200, Moritz Barsnick wrote:
> On Wed, Oct 07, 2015 at 16:07:53 -0400, Ronald S. Bultje wrote:
> > > BTW, are the "remaining" (meaning: more than from gcc) warnings emitted
> > > by the Intel C(++) Compiler redundant or false? The icc configuration
> > > should the
On Tue, Oct 13, 2015 at 06:30:46PM +0800, Zhang Rui wrote:
> ---
> libavutil/fifo.c| 37 +
> libavutil/fifo.h| 11 +++
> tests/ref/fate/fifo | 14 ++
> 3 files changed, 62 insertions(+)
>
> diff --git a/libavutil/fifo.c b/libavutil/f
2015-10-13 17:44 GMT+02:00 Christophe Gisquet :
> But I'll check.
Indeed not bit-exact to faani and C simple idct:
stddev:0.00 PSNR:163.48 MAXDIFF:1
This would at least result in fate no longer passing as this stands. I
don't think it's worth the speed difference.
No change on dct-test r
Partially fixes Ticket 4727.
-duration is not a safe expression, since duration can be INT_MIN.
One might ask how it can become INT_MIN.
Although it is true that line 2574 is no longer reached with INT_MIN due
to commit 053e80f6eaf8d87521fe58ea96886b6ee0bbe59d (which fixed another
integer overflow
On Tue, Oct 13, 2015 at 04:14:51PM +0200, Nedeljko Babic wrote:
> This patch fixes build of AAC encoder optimized for mips that was broken due
> to some changes in generic code that were not propagated to the optimized
> code.
>
> Also, some functions in the optimized code are basically duplicat
---
src/about | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/about b/src/about
index 8d8f606..67342a1 100644
--- a/src/about
+++ b/src/about
@@ -4,7 +4,11 @@
transcode, mux, demux,
stream, filter and play
pretty much anything
that humans and machines have cr
2015-10-13 15:43 GMT+02:00 Michael Niedermayer :
> On Tue, Oct 13, 2015 at 01:33:07PM +0200, Christophe Gisquet wrote:
>> Hi,
>>
>> 2015-10-13 13:10 GMT+02:00 Michael Niedermayer :
>> > hmm, iam a bit concerned that adding the rounder (which effectively is
>> > 0.5) causes a overflow, that would if
On Tue, Oct 13, 2015 at 2:33 AM, Rodger Combs wrote:
> +%macro AES_CRYPT 1
> +%if %1 == 1
> +%define CRYPT aesdec
> +%define LAST aesdeclast
> +cglobal aes_decrypt, 6,6,2
> +%else
> +%define CRYPT aesenc
> +%define LAST aesenclast
> +cglobal aes_encrypt, 6,6,2
> +%endif
> +pxor xm1, xm1
> +
> + environments, machine architecture, and configurations.
^^
plural -> architectures
Moritz
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
On Tue, Oct 13, 2015 at 02:46:55PM +0200, Moritz Barsnick wrote:
> On Tue, Oct 13, 2015 at 04:27:07 +0200, Michael Niedermayer wrote:
> > > @@ -117,6 +117,7 @@ static const struct {
> > > { "SSSE3","ssse3",AV_CPU_FLAG_SSSE3|AV_CPU_FLAG_ATOM },
> > > { "SSE4.1", "sse4", AV_CP
On Mon, Oct 12, 2015 at 07:37:50PM +0200, Christophe Gisquet wrote:
> In particular for 10 and 12 bits.
> ---
> libavcodec/dct-test.c | 2 ++
> libavcodec/x86/dct-test.c | 10 ++
> 2 files changed, 12 insertions(+)
>
> diff --git a/libavcodec/dct-test.c b/libavcodec/dct-test.c
> inde
This patch fixes build of AAC encoder optimized for mips that was broken due
to some changes in generic code that were not propagated to the optimized code.
Also, some functions in the optimized code are basically duplicate of functions
from generic code. Since they do not bring enough improveme
On Mon, Oct 12, 2015 at 07:37:49PM +0200, Christophe Gisquet wrote:
> On 12 frames of a 444p 12 bits DNxHR sequence, _put function:
> C: 78902 decicycles in idct, 262071 runs, 73 skips
> avx: 32478 decicycles in idct, 262045 runs, 99 skips
>
> Difference between the 2:
> st
On 13.10.2015 08:42, Tomas Härdin wrote:
On Mon, 2015-10-05 at 14:25 +0200, Tobias Rapp wrote:
On 05.10.2015 09:10, tim nicholson wrote:
On 04/10/15 13:07, Tomas Härdin wrote:
On Mon, 2015-09-28 at 15:11 +0200, Tobias Rapp wrote:
[...]
For me the most important thing is that anything dealin
On Tue, Oct 13, 2015 at 01:33:07PM +0200, Christophe Gisquet wrote:
> Hi,
>
> 2015-10-13 13:10 GMT+02:00 Michael Niedermayer :
> > hmm, iam a bit concerned that adding the rounder (which effectively is
> > 0.5) causes a overflow, that would if iam not mistaken imlpy that
> > things are very close
On Mon, Oct 12, 2015 at 07:37:47PM +0200, Christophe Gisquet wrote:
> Modeled from the prores version. Clips to [0;1023] and is bitexact.
> Bitexactness requires to add offsets in different places compared to
> prores or C, and makes the function approximately 2% slower.
>
> For 16 frames of a DNx
On Tue, 13 Oct 2015 09:18:35 -0400
Ganesh Ajjanagadde wrote:
> On Tue, Oct 13, 2015 at 9:13 AM, compn wrote:
> > On Mon, 12 Oct 2015 15:10:22 -0700
> > "Reynaldo H. Verdejo Pinochet" wrote:
> >
> >> Signed-off-by: Reynaldo H. Verdejo Pinochet
> >> ---
> >> src/index | 33 +
On Mon, Oct 12, 2015 at 07:37:48PM +0200, Christophe Gisquet wrote:
> This is done by not passing clamping regs to the macro, and there,
> deducing that this is a pure idct.
>
> There are not many users (vf_spp is one), and the _add version doesn't
> look like it will ever be used.
> ---
> libavc
On Tue, Oct 13, 2015 at 9:13 AM, compn wrote:
> On Mon, 12 Oct 2015 15:10:22 -0700
> "Reynaldo H. Verdejo Pinochet" wrote:
>
>> Signed-off-by: Reynaldo H. Verdejo Pinochet
>> ---
>> src/index | 33 +
>> 1 file changed, 33 insertions(+)
>
> thanks
>
> applied.
Wa
On Mon, Oct 12, 2015 at 3:44 PM, Ganesh Ajjanagadde wrote:
> On Sun, Oct 11, 2015 at 8:27 PM, Ganesh Ajjanagadde wrote:
>> On Sun, Oct 11, 2015 at 6:00 PM, Moritz Barsnick wrote:
>>> On Sun, Oct 11, 2015 at 16:29:16 -0400, Ganesh Ajjanagadde wrote:
+ Microsoft Windows, the BSD's, Solaris,
---
src/about | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/about b/src/about
index 8d8f606..cd0da12 100644
--- a/src/about
+++ b/src/about
@@ -4,7 +4,11 @@
transcode, mux, demux,
stream, filter and play
pretty much anything
that humans and machines have cr
On Mon, 12 Oct 2015 15:10:22 -0700
"Reynaldo H. Verdejo Pinochet" wrote:
> Signed-off-by: Reynaldo H. Verdejo Pinochet
> ---
> src/index | 33 +
> 1 file changed, 33 insertions(+)
thanks
applied.
-compn
___
ffmpeg-d
On Tue, Oct 13, 2015 at 4:02 AM, Clément Bœsch wrote:
> On Tue, Oct 13, 2015 at 09:25:03AM +0200, Paul B Mahol wrote:
> [...]
>> What about fmax/FFMAX?
>
> Feel free to try that out (it looks OT regarding the patch), but fmax()
> looks glibc specific
Maybe (long term) we can use an av_fabs, av_fa
On Tue, Oct 13, 2015 at 2:45 AM, Clément Bœsch wrote:
> On Tue, Oct 13, 2015 at 12:31:10AM -0400, Ganesh Ajjanagadde wrote:
>> On Tue, Oct 13, 2015 at 12:26 AM, Ganesh Ajjanagadde
>> wrote:
>> > On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos
>> > wrote:
>> >> Ganesh Ajjanagadde mit.edu> w
Signed-off-by: Paul B Mahol
---
libavcodec/Makefile| 1 +
libavcodec/adpcm.c | 43 +++
libavcodec/allcodecs.c | 1 +
libavcodec/avcodec.h | 1 +
4 files changed, 46 insertions(+)
diff --git a/libavcodec/Makefile b/libavcodec/Makefile
index b8b
Signed-off-by: Paul B Mahol
---
libavformat/Makefile | 1 +
libavformat/allformats.c | 1 +
libavformat/vag.c| 78
3 files changed, 80 insertions(+)
create mode 100644 libavformat/vag.c
diff --git a/libavformat/Makefile b/libavforma
1 - 100 of 126 matches
Mail list logo