[libav-devel] [PATCH 0/6] D3D11VA in avconv

2017-01-03 Thread Steve Lhomme
Update on this patchset with a lot of fixes that was discussed during the last weeks. The patches have been split more. The public structures in libavutil are more documented and require less things set by the caller. The GUIDs declared in the template are defined in a separate avconv_guid.c

[libav-devel] [PATCH 5/6] avconv: add avconv_d3d11va

2017-01-03 Thread Steve Lhomme
From: Steve Lhomme The code is similar to avconv_dxva2. The decoded output needs to be copied into a staging texture that can be accessed by the CPU as the decoder texture can't be accessed by the CPU. edit: move the actual GUID definitions in a shared C file --- Changelog

[libav-devel] [PATCH 3/6] avconv: dxva2: move the DXVA GUID definitions outside of the template

2017-01-03 Thread Steve Lhomme
From: Steve Lhomme --- Makefile | 2 +- avconv_dxva2.c | 1 + avconv_dxva_template.c | 7 ++- avconv_guid.c | 20 4 files changed, 28 insertions(+), 2 deletions(-) create mode 100644 avconv_guid.c diff --git

[libav-devel] [PATCH 4/6] avconv_dxva2: remove unused initial values

2017-01-03 Thread Steve Lhomme
From: Steve Lhomme --- avconv_dxva2.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/avconv_dxva2.c b/avconv_dxva2.c index 4c6ef8d..0edfe55 100644 --- a/avconv_dxva2.c +++ b/avconv_dxva2.c @@ -125,8 +125,8 @@ static int

[libav-devel] [PATCH 1/6] libavutil: add support for AV_HWDEVICE_TYPE_D3D11VA

2017-01-03 Thread Steve Lhomme
From: Steve Lhomme --- doc/APIchanges | 3 + libavutil/Makefile | 3 + libavutil/hwcontext.c | 3 + libavutil/hwcontext.h | 1 + libavutil/hwcontext_d3d11va.c | 460 +

[libav-devel] [PATCH 6/6] avconv: use the typedefs more to make comparison between dxva2 and d3d11va

2017-01-03 Thread Steve Lhomme
From: Steve Lhomme --- avconv_dxva2.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/avconv_dxva2.c b/avconv_dxva2.c index 0edfe55..99f608f 100644 --- a/avconv_dxva2.c +++ b/avconv_dxva2.c @@ -69,7 +69,7 @@

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

2017-01-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

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

2017-01-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

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

2017-01-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 6063993..6fc4f97 100644 --- a/libavcodec/dxva2.c +++

[libav-devel] [PATCH 0/4] DXVA2 cleaning

2017-01-03 Thread Steve Lhomme
This patchset was already sent but various cleaning has been done and it has been rebased to the latest master. The main goal is to have patch 3/4 that allows more flexibility on the decoder pool, which we need in VLC for better efficiency. Patch 4/4 is the one that enables this flexibility.

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

2017-01-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

Re: [libav-devel] [PATCH 2/5] libautil: add support for AV_HWDEVICE_TYPE_D3D11VA

2017-01-03 Thread Steve Lhomme
Hi, On Mon, Dec 19, 2016 at 1:04 PM, Anton Khirnov wrote: > Quoting Steve Lhomme (2016-12-16 11:05:40) >> From: Steve Lhomme >> >> --- >> libavutil/Makefile | 3 + >> libavutil/hwcontext.c | 3 + >> libavutil/hwcontext.h |

Re: [libav-devel] [PATCH] avconv: Do not pass NULL to avio_tell

2017-01-03 Thread Vittorio Giovara
On Tue, Jan 3, 2017 at 1:41 PM, Vittorio Giovara wrote: > On Tue, Jan 3, 2017 at 1:33 PM, Luca Barbato wrote: >> On 03/01/2017 11:51, Vittorio Giovara wrote: >>> Should it? What happens here when you do pass NULL? >> >> It spams a pointless message

Re: [libav-devel] [PATCH] avconv: Do not pass NULL to avio_tell

2017-01-03 Thread Vittorio Giovara
On Tue, Jan 3, 2017 at 1:33 PM, Luca Barbato wrote: > On 03/01/2017 11:51, Vittorio Giovara wrote: >> Should it? What happens here when you do pass NULL? > > It spams a pointless message on verbose. mention it in the commit log please total_size is used without being

Re: [libav-devel] [PATCH] avconv: Do not pass NULL to avio_tell

2017-01-03 Thread Luca Barbato
On 03/01/2017 11:51, Vittorio Giovara wrote: > Should it? What happens here when you do pass NULL? It spams a pointless message on verbose. lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] crypto: consistently use size_t as type for length parameters

2017-01-03 Thread Diego Biurrun
size_t is the correct type to use for sizes. --- Dropped the kind of pointless version bump. doc/APIchanges | 4 libavutil/md5.c | 8 libavutil/md5.h | 8 +++- libavutil/sha.c | 6 +- libavutil/sha.h | 9 +++-- libavutil/version.h | 3 +++ 6 files

Re: [libav-devel] [PATCH] avconv: Do not pass NULL to avio_tell

2017-01-03 Thread Vittorio Giovara
On Thu, Dec 15, 2016 at 7:01 PM, Luca Barbato wrote: > The null demuxer does not have a backing AVIOContext. Should it? What happens here when you do pass NULL? > --- > avconv.c | 23 --- > 1 file changed, 12 insertions(+), 11 deletions(-) > > diff --git

Re: [libav-devel] [PATCH 3/8] hevcdec: export cropping information instead of handling it internally

2017-01-03 Thread Vittorio Giovara
On Tue, Dec 27, 2016 at 7:31 PM, Anton Khirnov wrote: > --- > libavcodec/hevc_parser.c | 6 -- > libavcodec/hevc_ps.c | 33 - > libavcodec/hevc_ps.h | 2 -- > libavcodec/hevc_refs.c | 19 --- >

Re: [libav-devel] [PATCH 2/8] lavc: add an option for exporting cropping information to the caller

2017-01-03 Thread Vittorio Giovara
On Tue, Dec 27, 2016 at 7:31 PM, Anton Khirnov wrote: > Also, add generic code for handling cropping, so the decoders can export > just the cropping size and not bother with the rest. > --- > doc/APIchanges | 4 ++ > libavcodec/avcodec.h | 22 + >

Re: [libav-devel] [PATCH 1/8] frame: add a cropping rectangle to AVFrame

2017-01-03 Thread Vittorio Giovara
On Tue, Dec 27, 2016 at 7:31 PM, Anton Khirnov wrote: > Extend the width/height doy to clarify that it should store coded > values. s/doy/documentation/ ? > --- > doc/APIchanges | 4 > libavutil/frame.c | 4 > libavutil/frame.h | 28

Re: [libav-devel] [PATCH 7/8] h264dec: export cropping information instead of handling it internally

2017-01-03 Thread Vittorio Giovara
On Tue, Dec 27, 2016 at 7:31 PM, Anton Khirnov wrote: > --- > libavcodec/h264_ps.c| 9 - > libavcodec/h264_slice.c | 21 +++-- > libavcodec/h264dec.c| 26 +++--- > libavcodec/h264dec.h| 5 + > 4 files changed, 27

Re: [libav-devel] [PATCH 4/8] h264dec: drop a redundant check

2017-01-03 Thread Vittorio Giovara
On Tue, Dec 27, 2016 at 7:31 PM, Anton Khirnov wrote: > Cropping parameters are already checked for validity during SPS parsing, > no need to check them again. > --- > libavcodec/h264_slice.c | 17 - > 1 file changed, 17 deletions(-) ok -- Vittorio

Re: [libav-devel] [PATCH 4/4] h264_cavlc: check the value of run_before

2017-01-03 Thread Vittorio Giovara
On Wed, Dec 28, 2016 at 1:15 PM, Anton Khirnov wrote: > Section 9.2.3.2 of the spec implies that run_before must not be larger > than zeros_left. > > Fixes invalid reads with corrupted files. > > CC: libav-sta...@libav.org > Bug-Id: 1000 > Found-By: Kamil Frankowicz > --- >

Re: [libav-devel] [PATCH 3/4] h2645_parse: use the bytestream2 API for packet splitting

2017-01-03 Thread Vittorio Giovara
On Wed, Dec 28, 2016 at 1:15 PM, Anton Khirnov wrote: > The code does some nontrivial jumping around in the buffer, so it is > safer to use a checked API rather than do everything manually. > > Fixes a bug in nalff parsing, where the length field is currently not > counted in

Re: [libav-devel] [PATCH 8/8] theora: export cropping information instead of handling it internally

2017-01-03 Thread Vittorio Giovara
On Tue, Dec 27, 2016 at 7:31 PM, Anton Khirnov wrote: > --- > libavcodec/vp3.c | 23 +-- > 1 file changed, 9 insertions(+), 14 deletions(-) > > diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c > index 26374cc..cb8925b 100644 > --- a/libavcodec/vp3.c > +++

Re: [libav-devel] [PATCH 2/4] h264dec: initialize field_started to 0 on each decode call

2017-01-03 Thread Vittorio Giovara
On Wed, Dec 28, 2016 at 1:15 PM, Anton Khirnov wrote: > It might be incorrectly set to 1 if the previous call exited with an > error. > > CC: libav-sta...@libav.org > --- > libavcodec/h264dec.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/h264dec.c