Re: [libav-devel] [PATCH 060/115] tta: Convert to the new bitstream reader

2016-10-05 Thread Diego Biurrun
On Thu, Aug 04, 2016 at 11:43:38AM +0200, Diego Biurrun wrote: > On Sat, Jul 30, 2016 at 02:17:54PM +0200, Anton Khirnov wrote: > > Quoting Diego Biurrun (2016-06-09 17:12:42) > > > --- a/libavcodec/tta.c > > > +++ b/libavcodec/tta.c > > > @@ -363,17 +363,17 @@ static int

[libav-devel] [PATCH] d3d11va: use the proper slice index

2016-10-05 Thread Steve Lhomme
The slice index expected by D3D11VA is the one from the texture not from the array or texture/slices. In VLC the slices we provide the decoder don't start from 0 and thus pictures appear in bogus order. With possible crashes and corruptions when using an invalid index. -- * forgot to bump the

Re: [libav-devel] [PATCH] d3d11va: use the proper slice index

2016-10-05 Thread Diego Biurrun
On Wed, Oct 05, 2016 at 09:55:08AM +0200, Steve Lhomme wrote: > The slice index expected by D3D11VA is the one from the texture not from the > array or texture/slices. > > In VLC the slices we provide the decoder don't start from 0 and thus pictures > appear in bogus order. With possible crashes

Re: [libav-devel] [PATCH 1/2] hevc: x86: Refactor IDCT macro declarations

2016-10-05 Thread Luca Barbato
On 05/10/16 11:49, Diego Biurrun wrote: > --- > libavcodec/x86/hevc_idct.asm | 34 +++--- > 1 file changed, 11 insertions(+), 23 deletions(-) > OK. ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 2/2] hevc: Add SSE2 and AVX IDCT

2016-10-05 Thread Luca Barbato
On 05/10/16 11:49, Diego Biurrun wrote: > From: Alexandra Hájková > > Signed-off-by: Diego Biurrun > --- > libavcodec/x86/hevc_idct.asm | 768 > ++ > libavcodec/x86/hevcdsp_init.c | 58 > 2 files

[libav-devel] [PATCH 1/2] hevc: Add SSE2 and AVX IDCT

2016-10-05 Thread Alexandra Hájková
--- Move the transform sizes into the macro when initialize. libavcodec/x86/hevc_idct.asm | 798 +++-- libavcodec/x86/hevcdsp_init.c | 65 +++- 2 files changed, 834 insertions(+), 29 deletions(-) diff --git a/libavcodec/x86/hevc_idct.asm

[libav-devel] [PATCH 1/2] hevc: x86: Refactor IDCT macro declarations

2016-10-05 Thread Diego Biurrun
--- libavcodec/x86/hevc_idct.asm | 34 +++--- 1 file changed, 11 insertions(+), 23 deletions(-) diff --git a/libavcodec/x86/hevc_idct.asm b/libavcodec/x86/hevc_idct.asm index d662aa9..f95a2cb 100644 --- a/libavcodec/x86/hevc_idct.asm +++ b/libavcodec/x86/hevc_idct.asm

[libav-devel] [PATCH 2/2] hevc: Add SSE2 and AVX IDCT

2016-10-05 Thread Diego Biurrun
From: Alexandra Hájková Signed-off-by: Diego Biurrun --- libavcodec/x86/hevc_idct.asm | 768 ++ libavcodec/x86/hevcdsp_init.c | 58 2 files changed, 826 insertions(+) diff --git

Re: [libav-devel] [PATCH] d3d11va: use the proper slice index

2016-10-05 Thread Steve Lhomme
On Wed, Oct 5, 2016 at 10:20 AM, Diego Biurrun wrote: > On Wed, Oct 05, 2016 at 09:55:08AM +0200, Steve Lhomme wrote: >> The slice index expected by D3D11VA is the one from the texture not from the >> array or texture/slices. >> >> In VLC the slices we provide the decoder don't

[libav-devel] [PATCH] configure: #include stdint.h as part of libxavs test

2016-10-05 Thread Diego Biurrun
Unfortunately the xavs.h API header is not self-sufficient and relies on manual stdint.h inclusion by its users. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index fa8c958..3b8d539 100755 --- a/configure +++ b/configure @@ -4652,7 +4652,7

Re: [libav-devel] [PATCH 1/2] hevc: Add SSE2 and AVX IDCT

2016-10-05 Thread Diego Biurrun
On Tue, Oct 04, 2016 at 09:01:21PM +0200, Alexandra Hájková wrote: > --- a/libavcodec/x86/hevc_idct.asm > +++ b/libavcodec/x86/hevc_idct.asm > @@ -74,34 +286,579 @@ cglobal hevc_idct_%1x%1_dc_%2, 1, 2, 1, coeff, tmp > > -; 8-bit > -INIT_MMX mmxext > -IDCT_DC_NL 4, 8 > -IDCT_DC 8, 2,

Re: [libav-devel] [PATCH] configure: #include stdint.h as part of libxavs test

2016-10-05 Thread Janne Grunau
On 2016-10-05 09:06:47 +0200, Diego Biurrun wrote: > Unfortunately the xavs.h API header is not self-sufficient and relies > on manual stdint.h inclusion by its users. have you tried fixing it upstream? Not that I have high hopes of getting this fixed considering the last commit was over 5 years

[libav-devel] [PATCH] d3d11va: use the proper slice index

2016-10-05 Thread Steve Lhomme
The slice index expected by D3D11VA is the one from the texture not from the array or texture/slices. In VLC the slices we provide the decoder don't start from 0 and thus pictures appear in bogus order. With possible crashes and corruptions when using an invalid index. -- * forgot to bump the

Re: [libav-devel] [PATCH] configure: #include stdint.h as part of libxavs test

2016-10-05 Thread Diego Biurrun
On Wed, Oct 05, 2016 at 10:33:12AM +0200, Janne Grunau wrote: > On 2016-10-05 09:06:47 +0200, Diego Biurrun wrote: > > Unfortunately the xavs.h API header is not self-sufficient and relies > > on manual stdint.h inclusion by its users. > > have you tried fixing it upstream? Not that I have high

[libav-devel] [0.8] [PATCH 1/2] mov: Check the entries value when parsing dref boxes

2016-10-05 Thread Diego Biurrun
From: Luca Barbato And properly reset the entries count when resetting the entries. CC: libav-sta...@libav.org Bug-Id: 929 Bug-Id: CVE-2016-3062 (cherry picked from commit 7e01d48cfd168c3dfc663f03a3b6a98e0ecba328) Signed-off-by: Diego Biurrun ---

[libav-devel] [0.8] [PATCH 2/2] asfenc: Check pts

2016-10-05 Thread Diego Biurrun
From: Michael Niedermayer Fixes integer overflow Fixes: 0063df8be3aaa30dd6d76f59c8f818c8/signal_sigsegv_7b7b59_3634_bf418b6822bbfa68734411d96b667be3.mov Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer

[libav-devel] [PATCH 2/2] avformat/output-example: Declare link dependency on libswscale in the Makefile

2016-10-05 Thread Diego Biurrun
(cherry picked from commit 47b6cfc2a0333fb24f074d27830bf35ae5007050) Signed-off-by: Diego Biurrun --- common.mak | 3 ++- libavformat/Makefile | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/common.mak b/common.mak index 5d80986..a63eaf8 100644

[libav-devel] [9] small build fixes

2016-10-05 Thread Diego Biurrun
These two patches fix linking output-example on x86_32. It's annoying to have "make check" break when testing patches. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 1/2] build: output-example: Add avutil to ELIBS in link command

2016-10-05 Thread Diego Biurrun
output-example links against swscale, which depends on avutil. In standard configurations, by pure good luck, the libs before swscale pull in all of swscale's dependencies and linking succeeds. However, in some configurations like --disable-asm this is not the case and linking fails. Hardcode

[libav-devel] [PATCH 1/4] hevc: x86: Refactor IDCT macro declarations

2016-10-05 Thread Diego Biurrun
--- libavcodec/x86/hevc_idct.asm | 34 +++--- 1 file changed, 11 insertions(+), 23 deletions(-) diff --git a/libavcodec/x86/hevc_idct.asm b/libavcodec/x86/hevc_idct.asm index d662aa9..71e4607 100644 --- a/libavcodec/x86/hevc_idct.asm +++ b/libavcodec/x86/hevc_idct.asm

[libav-devel] [PATCH 2/4] hevc: x86: Rename hevc_idct.asm to hevc_idct_dc.asm

2016-10-05 Thread Diego Biurrun
Also rename related init macros to simplify addition of new IDCT code. --- The new file takes a lot of time to assemble, so this helps keep total compilation time bearable... libavcodec/x86/Makefile| 2 +- libavcodec/x86/{hevc_idct.asm => hevc_idct_dc.asm} | 2 +-

[libav-devel] [PATCH 3/4] hevc: x86: Add SSE2 and AVX IDCT

2016-10-05 Thread Diego Biurrun
From: Alexandra Hájková Signed-off-by: Diego Biurrun --- libavcodec/x86/Makefile | 1 + libavcodec/x86/hevc_idct.asm | 798 ++ libavcodec/x86/hevcdsp_init.c | 49 +++ 3 files changed, 848

[libav-devel] [PATCH 4/4] hevc: x86: Refactor IDCT initialization macros

2016-10-05 Thread Diego Biurrun
--- libavcodec/x86/hevc_idct.asm | 35 ++- 1 file changed, 10 insertions(+), 25 deletions(-) diff --git a/libavcodec/x86/hevc_idct.asm b/libavcodec/x86/hevc_idct.asm index 7ad09fb..71e2a88 100644 --- a/libavcodec/x86/hevc_idct.asm +++

[libav-devel] [0.8] small build fixes

2016-10-05 Thread Diego Biurrun
These two patches fix linking output-example on x86_32. It's annoying to have "make check" break when testing patches. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 1/2] build: output-example: Add avutil to ELIBS in link command

2016-10-05 Thread Diego Biurrun
output-example links against swscale, which depends on avutil. In standard configurations, by pure good luck, the libs before swscale pull in all of swscale's dependencies and linking succeeds. However, in some configurations like --disable-asm this is not the case and linking fails. Hardcode

[libav-devel] [PATCH 2/2] avformat/output-example: Declare link dependency on libswscale in the Makefile

2016-10-05 Thread Diego Biurrun
(cherry picked from commit 47b6cfc2a0333fb24f074d27830bf35ae5007050) Signed-off-by: Diego Biurrun (cherry picked from commit b5cacdb3e12927c63084d26d4c7d9d43ed00dd66) Signed-off-by: Diego Biurrun --- common.mak | 3 ++- libavformat/Makefile | 1 + 2

Re: [libav-devel] [PATCH] d3d11va: use the proper slice index

2016-10-05 Thread Diego Biurrun
On Wed, Oct 05, 2016 at 12:52:00PM +0200, Steve Lhomme wrote: > The slice index expected by D3D11VA is the one from the texture not from the > array or texture/slices. Array of? This sentence is confusing.. Patch probably OK otherwise, I'm going to queue it once I know in which direction to

Re: [libav-devel] [PATCH 2/4] aarch64: Add a missing sign extension in ff_h264_idct8_add_neon

2016-10-05 Thread Diego Biurrun
On Fri, Sep 30, 2016 at 12:18:10PM +0300, Martin Storsjö wrote: > --- > libavcodec/aarch64/h264idct_neon.S | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/aarch64/h264idct_neon.S > b/libavcodec/aarch64/h264idct_neon.S > index 5395e14..6354c03 100644 > ---

Re: [libav-devel] [PATCH 2/4] aarch64: Add a missing sign extension in ff_h264_idct8_add_neon

2016-10-05 Thread Martin Storsjö
On Wed, 5 Oct 2016, Diego Biurrun wrote: On Fri, Sep 30, 2016 at 12:18:10PM +0300, Martin Storsjö wrote: --- libavcodec/aarch64/h264idct_neon.S | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/aarch64/h264idct_neon.S b/libavcodec/aarch64/h264idct_neon.S index 5395e14..6354c03

Re: [libav-devel] [PATCH 2/4] aarch64: Add a missing sign extension in ff_h264_idct8_add_neon

2016-10-05 Thread Diego Biurrun
On Wed, Oct 05, 2016 at 10:09:58PM +0300, Martin Storsjö wrote: > On Wed, 5 Oct 2016, Diego Biurrun wrote: > > On Fri, Sep 30, 2016 at 12:18:10PM +0300, Martin Storsjö wrote: > >> --- a/libavcodec/aarch64/h264idct_neon.S > >> +++ b/libavcodec/aarch64/h264idct_neon.S > >> @@ -264,6 +264,7 @@

Re: [libav-devel] [PATCH] h264_intrapred: x86: Update comments left behind in 95c89da36ebeeb96b7146c0d70f46c582397da7f

2016-10-05 Thread Martin Storsjö
On Wed, 5 Oct 2016, Diego Biurrun wrote: --- libavcodec/x86/h264_intrapred.asm | 65 +++- libavcodec/x86/h264_intrapred_10bit.asm | 77 ++--- 2 files changed, 76 insertions(+), 66 deletions(-) Probably ok // Martin

[libav-devel] [PATCH] audiodsp: x86: Remove pointless header file

2016-10-05 Thread Diego Biurrun
Its single forward declaration can be moved to the only place it is used, like is done for all other dsp init files. --- libavcodec/x86/audiodsp.h | 25 - libavcodec/x86/audiodsp_init.c | 4 +++- 2 files changed, 3 insertions(+), 26 deletions(-) delete mode 100644

[libav-devel] [PATCH] h264_intrapred: x86: Update comments left behind in 95c89da36ebeeb96b7146c0d70f46c582397da7f

2016-10-05 Thread Diego Biurrun
--- libavcodec/x86/h264_intrapred.asm | 65 +++- libavcodec/x86/h264_intrapred_10bit.asm | 77 ++--- 2 files changed, 76 insertions(+), 66 deletions(-) diff --git a/libavcodec/x86/h264_intrapred.asm b/libavcodec/x86/h264_intrapred.asm

[libav-devel] [PATCH 2/2] mov: Evaluate the movie display matrix

2016-10-05 Thread Vittorio Giovara
This matrix needs to be applied after all others have (currently only display matrix from trak), but cannot be handled in movie box, since streams are not allocated yet. So store it in main context and if not identity, apply it when appropriate, handling the case when trak display matrix is

[libav-devel] [PATCH 1/2] mov: Rewrite aspect ratio handling from display matrix

2016-10-05 Thread Vittorio Giovara
From: Michael Niedermayer This should make this code more robust and simpler. The use of hypot is by Ganesh Ajjanagadde. Also use the context display matrix instead of the local one. Signed-off-by: Vittorio Giovara --- libavformat/mov.c | 22

[libav-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-05 Thread Vittorio Giovara
This matrix needs to be applied after all others have (currently only display matrix from trak), but cannot be handled in movie box, since streams are not allocated yet. So store it in main context and if not identity, apply it when appropriate, handling the case when trak display matrix is