Re: [libav-devel] [PATCH 1/3] hevc: decouple calling get_format() from exporting the SPS parameters

2016-12-03 Thread Luca Barbato
On 03/12/2016 17:34, Anton Khirnov wrote: > This makes sure ff_get_format() does not get called unnecessarily from > update_thread_context(). > --- > libavcodec/hevcdec.c | 49 ++--- > 1 file changed, 30 insertions(+), 19 deletions(-) > Seems ok.

Re: [libav-devel] [PATCH 1/3] build: Create a component for MPEG audio header decoding

2016-12-03 Thread Vittorio Giovara
On Sat, Dec 3, 2016 at 9:16 AM, Diego Biurrun wrote: > Fixes standalone compilation of the libmp3lame encoder. > --- > configure | 5 - > libavcodec/Makefile | 10 -- > 2 files changed, 8 insertions(+), 7 deletions(-) I wonder if it should be split in two, but ok anyway. -

Re: [libav-devel] [PATCH 2/3] build: Add missing audio_frame_queue dependency for libwavpack encoder

2016-12-03 Thread Vittorio Giovara
On Sat, Dec 3, 2016 at 9:16 AM, Diego Biurrun wrote: > --- > configure | 1 + > 1 file changed, 1 insertion(+) should be ok -- Vittorio ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 3/5] configure: Simplify OMX check

2016-12-03 Thread Martin Storsjö
On Sat, 3 Dec 2016, Diego Biurrun wrote: On Sat, Dec 03, 2016 at 02:50:31PM +0100, Luca Barbato wrote: On 03/12/2016 11:07, Diego Biurrun wrote: On Sat, Dec 03, 2016 at 11:00:01AM +0100, Luca Barbato wrote: On 03/12/2016 07:49, Diego Biurrun wrote: +enabled omx && { check_heade

Re: [libav-devel] [PATCH 4/5] configure: Simplify MMAL check

2016-12-03 Thread Martin Storsjö
On Sat, 3 Dec 2016, Diego Biurrun wrote: On Sat, Dec 03, 2016 at 12:33:04PM +0200, Martin Storsjö wrote: On Sat, 3 Dec 2016, Diego Biurrun wrote: >--- a/configure >+++ b/configure >@@ -4677,14 +4677,12 @@ enabled libx265 && require_pkg_config x265 x265.h x265_api_get && >enabled libx

Re: [libav-devel] [PATCH 3/8] decode: restructure the core decoding code

2016-12-03 Thread Martin Storsjö
On Wed, 30 Nov 2016, Anton Khirnov wrote: Currently, the new decoding API is pretty much just a wrapper around the old deprecated one. This is problematic, since it interferes with making full use of the flexibility added by the new API. The old API should also be removed at some future point.

[libav-devel] [PATCH 1/3] hevc: decouple calling get_format() from exporting the SPS parameters

2016-12-03 Thread Anton Khirnov
This makes sure ff_get_format() does not get called unnecessarily from update_thread_context(). --- libavcodec/hevcdec.c | 49 ++--- 1 file changed, 30 insertions(+), 19 deletions(-) diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c index 9dd86c2

[libav-devel] [PATCH 3/3] pthread_frame: do not run hwaccel decoding asynchronously unless it's safe

2016-12-03 Thread Anton Khirnov
Certain hardware decoding APIs are often not thread-safe, so having the user access decoded hardware surfaces while the decoder is running in another thread can cause failures (this is mainly known to happen with DXVA2). For such hwaccels, only allow the decoding thread to run while the user is in

[libav-devel] [PATCH 2/3] pthread_frame: ensure the threads don't run simultaneously with hwaccel

2016-12-03 Thread Anton Khirnov
--- libavcodec/h263dec.c | 2 +- libavcodec/h264dec.c | 2 +- libavcodec/pthread_frame.c | 27 +++ 3 files changed, 29 insertions(+), 2 deletions(-) diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c index e4a7227..921ff5f 100644 --- a/libavcodec/h263de

[libav-devel] [PATCH] build: Add missing Makefile entries and ifdefs for QSV hwaccels

2016-12-03 Thread Diego Biurrun
--- Now the actually working version ... libavcodec/Makefile | 5 + libavcodec/qsvdec_h2645.c | 8 ++-- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 9cfc607..1e3971e 100644 --- a/libavcodec/Makefile +++ b/libavcodec

Re: [libav-devel] [PATCH 3/3] build: Have QSV hwaccels depend on the respective QSV decoder

2016-12-03 Thread Diego Biurrun
On Sat, Dec 03, 2016 at 03:16:32PM +0100, Diego Biurrun wrote: > The hwaccels do not make sense and do not build w/o the respective decoders. > --- > configure | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) That was stupid, ignore. Diego __

[libav-devel] [PATCH] build: Have old H.264/HEVC nvenc encoders select their new counterparts

2016-12-03 Thread Diego Biurrun
This makes sense and takes care of missing build dependencies. --- configure | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 61f39ff..c8cd19a 100755 --- a/configure +++ b/configure @@ -2228,8 +2228,8 @@ vc1_qsv_decoder_select="qsvdec vc1_qsv_hwac

[libav-devel] [PATCH] build: Add missing golomb dependency for HEVC QSV encoder

2016-12-03 Thread Diego Biurrun
--- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 0ae5f38..a4a55db 100755 --- a/configure +++ b/configure @@ -2290,7 +2290,7 @@ hevc_qsv_encoder_deps="libmfx" hevc_qsv_encoder_extralibs="libmfx_extralibs" hevc_qsv_decoder_select="hevc_m

[libav-devel] [PATCH 1/3] build: Create a component for MPEG audio header decoding

2016-12-03 Thread Diego Biurrun
Fixes standalone compilation of the libmp3lame encoder. --- configure | 5 - libavcodec/Makefile | 10 -- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/configure b/configure index 857a65e..4b89cbc 100755 --- a/configure +++ b/configure @@ -1728,6 +1728,7 @@

[libav-devel] [PATCH 3/3] build: Have QSV hwaccels depend on the respective QSV decoder

2016-12-03 Thread Diego Biurrun
The hwaccels do not make sense and do not build w/o the respective decoders. --- configure | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/configure b/configure index 4451369..630cc9a 100755 --- a/configure +++ b/configure @@ -2136,7 +2136,7 @@ h264_d3d11va_hwaccel_s

[libav-devel] [PATCH 2/3] build: Add missing audio_frame_queue dependency for libwavpack encoder

2016-12-03 Thread Diego Biurrun
--- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 4b89cbc..4451369 100755 --- a/configure +++ b/configure @@ -2291,6 +2291,7 @@ libvpx_vp8_encoder_deps="libvpx" libvpx_vp9_decoder_deps="libvpx" libvpx_vp9_encoder_deps="libvpx" libwavpack_encoder_deps=

Re: [libav-devel] [PATCH 3/5] configure: Simplify OMX check

2016-12-03 Thread Diego Biurrun
On Sat, Dec 03, 2016 at 02:50:31PM +0100, Luca Barbato wrote: > On 03/12/2016 11:07, Diego Biurrun wrote: > > On Sat, Dec 03, 2016 at 11:00:01AM +0100, Luca Barbato wrote: > >> On 03/12/2016 07:49, Diego Biurrun wrote: > >>> +enabled omx && { check_header OMX_Core.h || die > >>> "ERRO

Re: [libav-devel] [PATCH 1/4] dxva2: make ff_dxva2_get_surface() static

2016-12-03 Thread Steve Lhomme
Oops, it's based on a patch I never submitted. I'll redo them in the proper order. On Sat, Dec 3, 2016 at 1:26 PM, Steve Lhomme wrote: > From: Steve Lhomme > > --- > libavcodec/dxva2.c | 8 > libavcodec/dxva2_internal.h | 2 -- > 2 files changed, 4 insertions(+), 6 deletions(-

Re: [libav-devel] [PATCH 3/5] configure: Simplify OMX check

2016-12-03 Thread Luca Barbato
On 03/12/2016 11:07, Diego Biurrun wrote: > On Sat, Dec 03, 2016 at 11:00:01AM +0100, Luca Barbato wrote: >> On 03/12/2016 07:49, Diego Biurrun wrote: >>> +enabled omx && { check_header OMX_Core.h || die >>> "ERROR: OpenMAX IL headers not found"; } +enabled omx_rpi >>> && { ! enabled

Re: [libav-devel] [PATCH 4/5] configure: Simplify MMAL check

2016-12-03 Thread Diego Biurrun
On Sat, Dec 03, 2016 at 12:33:04PM +0200, Martin Storsjö wrote: > On Sat, 3 Dec 2016, Diego Biurrun wrote: > >--- a/configure > >+++ b/configure > >@@ -4677,14 +4677,12 @@ enabled libx265 && require_pkg_config x265 > >x265.h x265_api_get && > >enabled libxavs && require libxavs

Re: [libav-devel] [PATCH 06/12] mimic: Convert to the new bitstream reader

2016-12-03 Thread Anton Khirnov
Quoting Diego Biurrun (2016-11-27 17:32:37) > From: Alexandra Hájková > > --- > libavcodec/mimic.c | 24 > 1 file changed, 12 insertions(+), 12 deletions(-) > Ok -- Anton Khirnov ___ libav-devel mailing list libav-devel@lib

Re: [libav-devel] [PATCH 05/12] metasound: Convert to the new bitstream reader

2016-12-03 Thread Anton Khirnov
Quoting Diego Biurrun (2016-11-27 17:32:36) > From: Alexandra Hájková > > --- > libavcodec/metasound.c | 44 ++-- > 1 file changed, 22 insertions(+), 22 deletions(-) > Ok -- Anton Khirnov ___ libav-devel mail

[libav-devel] [PATCH 4/4] dxva2: allow an empty array of ID3D11VideoDecoderOutputView

2016-12-03 Thread Steve Lhomme
From: Steve Lhomme We can pick the correct slice index directly from the ID3D11VideoDecoderOutputView casted from data[3]. --- libavcodec/dxva2_internal.h | 4 ++-- libavcodec/version.h| 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/dxva2_internal.h b/l

[libav-devel] [PATCH 0/4] Updated dxva patches

2016-12-03 Thread Steve Lhomme
This patches take in account the remarks from Diego Biurrun. The main goal of those patches is to remove the requirement on d3d11 to preallocate decoding buffers. In VLC we can save a copy by not using a preallocated buffer. Steve Lhomme (4): dxva2: make ff_dxva2_get_surface() static dxva2: us

[libav-devel] [PATCH 3/4] dxva2: get the slice number directly from the surface in D3D11VA

2016-12-03 Thread Steve Lhomme
From: Steve Lhomme No need to loop through the known surfaces, we'll use it anyway. The loop is only done for DXVA2 --- libavcodec/dxva2.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/libavcodec/dxva2.c b/libavcodec/dxva2.c index 960a957..b74faac 100644 ---

[libav-devel] [PATCH 2/4] dxva2: use a single macro to test if the DXVA context is valid

2016-12-03 Thread Steve Lhomme
From: Steve Lhomme --- libavcodec/dxva2_h264.c | 4 +--- libavcodec/dxva2_hevc.c | 4 +--- libavcodec/dxva2_internal.h | 5 + libavcodec/dxva2_mpeg2.c| 4 +--- libavcodec/dxva2_vc1.c | 4 +--- 5 files changed, 9 insertions(+), 12 deletions(-) diff --git a/libavcodec/dxva2_h

[libav-devel] [PATCH 1/4] dxva2: make ff_dxva2_get_surface() static

2016-12-03 Thread Steve Lhomme
From: Steve Lhomme --- libavcodec/dxva2.c | 8 libavcodec/dxva2_internal.h | 2 -- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/libavcodec/dxva2.c b/libavcodec/dxva2.c index 12acfd2..960a957 100644 --- a/libavcodec/dxva2.c +++ b/libavcodec/dxva2.c @@ -31,7 +31

Re: [libav-devel] [PATCH 04/12] lagarith: Convert to the new bitstream reader

2016-12-03 Thread Anton Khirnov
Quoting Diego Biurrun (2016-11-27 17:32:35) > From: Alexandra Hájková > > --- > libavcodec/lagarith.c| 22 +++--- > libavcodec/lagarithrac.c | 12 ++-- > libavcodec/lagarithrac.h | 6 -- > 3 files changed, 21 insertions(+), 19 deletions(-) Ok -- Anton Khirnov

Re: [libav-devel] [PATCH 03/12] indeo: Convert to the new bitstream reader

2016-12-03 Thread Anton Khirnov
Quoting Diego Biurrun (2016-11-27 17:32:34) > From: Alexandra Hájková > > --- > libavcodec/indeo2.c | 16 +++--- > libavcodec/indeo3.c | 19 --- > libavcodec/indeo4.c | 158 > +++- > libavcodec/indeo5.c | 127 +---

Re: [libav-devel] [PATCH 02/12] imc: Convert to the new bitstream reader

2016-12-03 Thread Anton Khirnov
Quoting Diego Biurrun (2016-11-27 17:32:33) > From: Alexandra Hájková > > --- > libavcodec/imc.c | 45 +++-- > 1 file changed, 23 insertions(+), 22 deletions(-) Ok -- Anton Khirnov ___ libav-devel mailing list

Re: [libav-devel] [PATCH 4/5] configure: Simplify MMAL check

2016-12-03 Thread Martin Storsjö
On Sat, 3 Dec 2016, Diego Biurrun wrote: --- configure | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/configure b/configure index f838a53..44fc147 100755 --- a/configure +++ b/configure @@ -4677,14 +4677,12 @@ enabled libx265 && require_pkg_config x26

Re: [libav-devel] [PATCH 3/5] configure: Simplify OMX check

2016-12-03 Thread Diego Biurrun
On Sat, Dec 03, 2016 at 11:00:01AM +0100, Luca Barbato wrote: > On 03/12/2016 07:49, Diego Biurrun wrote: > > +enabled omx && { check_header OMX_Core.h || die "ERROR: > > OpenMAX IL headers not found"; } > > +enabled omx_rpi && { ! enabled cross_compile && add_cflags > > -

Re: [libav-devel] [PATCH 4/5] configure: Simplify MMAL check

2016-12-03 Thread Luca Barbato
On 03/12/2016 07:49, Diego Biurrun wrote: > --- > configure | 14 ++ > 1 file changed, 6 insertions(+), 8 deletions(-) > Seems fine. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 3/5] configure: Simplify OMX check

2016-12-03 Thread Luca Barbato
On 03/12/2016 07:49, Diego Biurrun wrote: > +enabled omx && { check_header OMX_Core.h || die "ERROR: > OpenMAX IL headers not found"; } > +enabled omx_rpi && { ! enabled cross_compile && add_cflags > -isystem/opt/vc/include/IL > + check_header

Re: [libav-devel] [PATCH 2/5] configure: Add misssing qpeldsp dependency to mss2 decoder

2016-12-03 Thread Luca Barbato
On 03/12/2016 07:49, Diego Biurrun wrote: > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > One too many S. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 1/5] configure: Add missing cuda dependency to nvenc

2016-12-03 Thread Hendrik Leppkes
On Sat, Dec 3, 2016 at 7:49 AM, Diego Biurrun wrote: > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index 70d645c..d6aba02 100755 > --- a/configure > +++ b/configure > @@ -2186,7 +2186,7 @@ wmv3_vaapi_hwaccel_select="vc1_vaapi_hw