[libav-devel] [PATCH 3/4] vp9: Add bsf to fix reordering in raw streams

2017-01-09 Thread Mark Thompson
Takes a raw input stream containing frames with correct timestamps but possibly out of order and inserts additional show-existing-frame packets to correct the ordering. --- More specifically, it fixes up the output of VAAPI VP9 with B-frames enabled to show the frames in the right order. It may

[libav-devel] [PATCH 2/4] vp9: Add bsf to merge superframes

2017-01-09 Thread Mark Thompson
From: "Ronald S. Bultje" From ffmpeg commit 2e6636aa87303d37b112e79f093ca39500f92364. --- Exactly as in the other tine (except licence header). It might be nicer to call it vp9_superframe_merge, but maintaining identical behaviour probably trumps that?

[libav-devel] [PATCH 4/4] vaapi_encode: Add VP9 support

2017-01-09 Thread Mark Thompson
--- Uses global_quality without accounting for any multiplier rather than adding a private option. B-frames are not enabled by default (without running it through the vp9_reorder filter the output will not show the right frames). configure | 3 + libavcodec/Makefile

[libav-devel] [PATCH 1/4] vaapi_encode: Pass framerate parameters to driver

2017-01-09 Thread Mark Thompson
--- The i965 driver has been updated so that this works consistently. libavcodec/vaapi_encode.c | 14 ++ libavcodec/vaapi_encode.h | 4 2 files changed, 18 insertions(+) diff --git a/libavcodec/vaapi_encode.c b/libavcodec/vaapi_encode.c index 246b76abc..953a6bae0 100644 ---

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

2017-01-09 Thread Anton Khirnov
Quoting Vittorio Giovara (2017-01-03 11:43:22) > 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 |

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

2017-01-09 Thread Luca Barbato
On 09/01/2017 12:38, Anton Khirnov wrote: > Quoting Luca Barbato (2017-01-09 11:36:47) >> On 09/01/2017 10:47, Anton Khirnov wrote: >>> The patch is fine with me, but more generally I'd say that as long >>> as the muxer is open, the AVIOContext belongs to it and the caller >>> touching it in any

Re: [libav-devel] [PATCH] hevcdec: add P010 support for D3D11VA

2017-01-09 Thread Anton Khirnov
Quoting Steve Lhomme (2017-01-04 09:39:31) > Given it's the same API than DVXA2 I don't know why the same output was not > enabled for both. > --- > libavcodec/hevcdec.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c >

[libav-devel] [PATCH] Changelog: mention the new avbuild/ directory

2017-01-09 Thread Anton Khirnov
Especially config.log is often read by users trying to compile Libav, so its move should be documented. --- Changelog | 1 + 1 file changed, 1 insertion(+) diff --git a/Changelog b/Changelog index e17ef20..70d6a4a 100644 --- a/Changelog +++ b/Changelog @@ -6,6 +6,7 @@ version : - Intel

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

2017-01-09 Thread Luca Barbato
On 09/01/2017 10:47, Anton Khirnov wrote: > The patch is fine with me, but more generally I'd say that as long > as the muxer is open, the AVIOContext belongs to it and the caller > touching it in any way is invalid. Shall we add an avformat_report_progress() or similar? lu

Re: [libav-devel] [PATCH] hmac: Explicitly convert types at function pointer assignment

2017-01-09 Thread Anton Khirnov
Quoting Diego Biurrun (2016-12-15 13:27:02) > Fixes a number of warnings of the type > libavutil/hmac.c:61:21: warning: assignment from incompatible pointer type > --- > > Ifdeffed the function pointer signature for hmac_update to match API, > according to Martin's review. > > libavutil/hmac.c

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

2017-01-09 Thread wm4
On Tue, 3 Jan 2017 17:34:59 +0100 Steve Lhomme wrote: > From: Steve Lhomme > > --- > doc/APIchanges | 3 + > libavutil/Makefile | 3 + > libavutil/hwcontext.c | 3 + > libavutil/hwcontext.h | 1 + >

[libav-devel] [PATCH] qsvdec: do not sync PIX_FMT_QSV surfaces

2017-01-09 Thread Anton Khirnov
Introducing enforced sync points in arbitrary places is bad for performance. Since the vast majority of receiving code (QSV VPP or encoders, retrieving frames through hwcontext) will do the syncing, this change should not be visible to most callers. But bumping micro just in case. This is also

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

2017-01-09 Thread Anton Khirnov
Quoting Luca Barbato (2016-12-15 19:01:03) > The null demuxer does not have a backing AVIOContext. > --- > avconv.c | 23 --- > 1 file changed, 12 insertions(+), 11 deletions(-) > > diff --git a/avconv.c b/avconv.c > index 5c31332..fe60625 100644 > --- a/avconv.c > +++

Re: [libav-devel] [PATCH 3/3] vaapi_h264: Scale log2_max_pic_order_cnt_lsb with max_b_frames

2017-01-09 Thread Anton Khirnov
Quoting Mark Thompson (2017-01-05 01:10:09) > Before this change, it was possible to overflow pic_order_cnt_lsb and > generate a stream with invalid POC numbering. This makes sure that > the field is large enough that a single IDR B* P sequence uses fewer > than half the available POC lsb values.

Re: [libav-devel] [PATCH] Changelog: mention the new avbuild/ directory

2017-01-09 Thread Luca Barbato
On 09/01/2017 11:10, Anton Khirnov wrote: > Especially config.log is often read by users trying to compile Libav, so > its move should be documented. > --- > Changelog | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Changelog b/Changelog > index e17ef20..70d6a4a 100644 > --- a/Changelog

Re: [libav-devel] [PATCH] qsvdec: do not sync PIX_FMT_QSV surfaces

2017-01-09 Thread Luca Barbato
On 09/01/2017 10:48, Anton Khirnov wrote: > Introducing enforced sync points in arbitrary places is bad for > performance. Since the vast majority of receiving code (QSV VPP or > encoders, retrieving frames through hwcontext) will do the syncing, this > change should not be visible to most

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

2017-01-09 Thread wm4
On Tue, 3 Jan 2017 17:35:03 +0100 Steve Lhomme wrote: > 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

Re: [libav-devel] [PATCH 4/4] build: Move cli tools to a separate subdirectory

2017-01-09 Thread Vittorio Giovara
On Fri, Jan 6, 2017 at 2:45 PM, Diego Biurrun wrote: > This unclutters the top-level directory and groups related files together. > --- > .gitignore | 3 -- > Makefile | 47 > >

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

2017-01-09 Thread Anton Khirnov
Quoting Diego Biurrun (2017-01-03 12:08:10) > 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 +- >

Re: [libav-devel] [PATCH 2/6] avconv: dxva2: factorize some code that can be common with d3d11va

2017-01-09 Thread wm4
On Tue, 3 Jan 2017 17:35:00 +0100 Steve Lhomme wrote: > From: Steve Lhomme > > avconv_dxva.h has to be included and misc. typedefs have to be set to use the > proper DXVA2 structures. > > initguid.h is included in avconv_dxva.h so any includes after

Re: [libav-devel] [PATCH] Changelog: mention the new avbuild/ directory

2017-01-09 Thread Diego Biurrun
On Mon, Jan 09, 2017 at 11:10:24AM +0100, Anton Khirnov wrote: > Especially config.log is often read by users trying to compile Libav, so > its move should be documented. > --- > Changelog | 1 + > 1 file changed, 1 insertion(+) OK Diego ___

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

2017-01-09 Thread Anton Khirnov
Quoting wm4 (2016-12-30 10:41:58) > On Tue, 27 Dec 2016 19:31:15 +0100 > 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

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

2017-01-09 Thread Vittorio Giovara
On Mon, Jan 9, 2017 at 1:18 PM, Anton Khirnov wrote: > Quoting Vittorio Giovara (2017-01-03 11:43:22) >> > @@ -450,6 +451,106 @@ int attribute_align_arg >> > avcodec_send_packet(AVCodecContext *avctx, const AVPacke >> > return 0; >> > } >> > >> > +static int

Re: [libav-devel] [PATCH 4/4] build: Move cli tools to a separate subdirectory

2017-01-09 Thread Luca Barbato
On 09/01/2017 12:11, Vittorio Giovara wrote: > On Fri, Jan 6, 2017 at 2:45 PM, Diego Biurrun wrote: >> This unclutters the top-level directory and groups related files together. >> --- >> .gitignore | 3 -- >> Makefile

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

2017-01-09 Thread Anton Khirnov
Quoting Luca Barbato (2017-01-09 11:36:47) > On 09/01/2017 10:47, Anton Khirnov wrote: > > The patch is fine with me, but more generally I'd say that as long > > as the muxer is open, the AVIOContext belongs to it and the caller > > touching it in any way is invalid. > > Shall we add an

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

2017-01-09 Thread Diego Biurrun
On Sun, Nov 27, 2016 at 05:32:39PM +0100, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/motionpixels.c | 77 > --- > 1 file changed, 40 insertions(+), 37 deletions(-) OKed by Luca on IRC. Diego

Re: [libav-devel] [PATCH] qsvdec: do not sync PIX_FMT_QSV surfaces

2017-01-09 Thread Diego Biurrun
On Mon, Jan 09, 2017 at 10:48:32AM +0100, Anton Khirnov wrote: > This is also consistent with that VAAPI hwaccel does. with _W_hat Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 4/4] build: Move cli tools to a separate subdirectory

2017-01-09 Thread Vittorio Giovara
On Mon, Jan 9, 2017 at 12:59 PM, Diego Biurrun wrote: > On Mon, Jan 09, 2017 at 12:11:58PM +0100, Vittorio Giovara wrote: >> On Fri, Jan 6, 2017 at 2:45 PM, Diego Biurrun wrote: >> > This unclutters the top-level directory and groups related files together. >>

Re: [libav-devel] [PATCH 4/4] build: Move cli tools to a separate subdirectory

2017-01-09 Thread Diego Biurrun
On Mon, Jan 09, 2017 at 12:11:58PM +0100, Vittorio Giovara wrote: > On Fri, Jan 6, 2017 at 2:45 PM, Diego Biurrun wrote: > > This unclutters the top-level directory and groups related files together. > > --- > > .gitignore | 3 -- > > Makefile

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

2017-01-09 Thread Anton Khirnov
Quoting Vittorio Giovara (2017-01-09 13:24:21) > On Mon, Jan 9, 2017 at 1:18 PM, Anton Khirnov wrote: > > Quoting Vittorio Giovara (2017-01-03 11:43:22) > >> > @@ -450,6 +451,106 @@ int attribute_align_arg > >> > avcodec_send_packet(AVCodecContext *avctx, const AVPacke > >> >

Re: [libav-devel] [PATCH 4/4] build: Move cli tools to a separate subdirectory

2017-01-09 Thread Vittorio Giovara
On Mon, Jan 9, 2017 at 3:38 PM, Diego Biurrun wrote: > On Mon, Jan 09, 2017 at 01:41:13PM +0100, Luca Barbato wrote: >> On 09/01/2017 12:11, Vittorio Giovara wrote: >> > >> > I don't mind the idea, but imo the main files (avconv.c avprobe.c >> > avserver.c) should be kept in the

Re: [libav-devel] [PATCH 4/4] build: Move cli tools to a separate subdirectory

2017-01-09 Thread Diego Biurrun
On Mon, Jan 09, 2017 at 01:41:13PM +0100, Luca Barbato wrote: > On 09/01/2017 12:11, Vittorio Giovara wrote: > > On Fri, Jan 6, 2017 at 2:45 PM, Diego Biurrun wrote: > >> This unclutters the top-level directory and groups related files together. > >> --- > >> .gitignore

Re: [libav-devel] [PATCH 4/4] build: Move cli tools to a separate subdirectory

2017-01-09 Thread Vittorio Giovara
On Mon, Jan 9, 2017 at 4:06 PM, Diego Biurrun wrote: > On Mon, Jan 09, 2017 at 03:53:44PM +0100, Vittorio Giovara wrote: >> Finally, there are equally-annoying files present in the top level >> directory, such as COPYING.* LICENSE and several others, which hardly >> anybody

Re: [libav-devel] [PATCH 4/4] build: Move cli tools to a separate subdirectory

2017-01-09 Thread Diego Biurrun
On Mon, Jan 09, 2017 at 03:53:44PM +0100, Vittorio Giovara wrote: > Finally, there are equally-annoying files present in the top level > directory, such as COPYING.* LICENSE and several others, which hardly > anybody reads or edits, which could be probably be moved to a separate > directory too.

Re: [libav-devel] [PATCH 4/4] build: Move cli tools to a separate subdirectory

2017-01-09 Thread Luca Barbato
On 09/01/2017 15:38, Diego Biurrun wrote: > IMNSHO this is just a pointless complication for a handful of devs > that run the av* tools from the build dir instead of installing them > or using an installed version. It is not not an handful devs and it is not a complication, please make it so the