Re: [libav-devel] [PATCH 5/6] lavc: Delete all fake hwaccels

2017-09-07 Thread Luca Barbato
On 07/09/2017 21:49, Mark Thompson wrote:> If it is decided that they do actually need to persist, they should probably all get moved into a separate file to separate them from actually-used code. I'd just drop them and be done with that. lu ___ lib

Re: [libav-devel] [PATCH 3/6] webp: Fix alpha initialisation

2017-09-07 Thread Luca Barbato
On 07/09/2017 21:49, Mark Thompson wrote: ff_get_format() in the next patch will reject formats which aren't in the offered list, so the hack in 7cb9296db872c4221453e5411f242ebcfca62664 is no longer valid. Change the hack by adding a new field in the VP8 decoder context to indicate that it's act

Re: [libav-devel] [PATCH 1/6] lavc: Add codec metadata to indicate hardware support

2017-09-07 Thread Luca Barbato
On 07/09/2017 21:49, Mark Thompson wrote: + * The array is terminated by a configuration with a pix_fmt of + * AV_PIX_FMT_NONE. why not NULL-terminating it? The rest looks good. ___ libav-devel mailing list libav-devel@libav.org https://lists

Re: [libav-devel] [PATCH] qsv: Load the hw hevc plugin by default on Linux

2017-09-07 Thread Luca Barbato
On 07/09/2017 22:14, Mark Thompson wrote: In principle I'm very much in favour, except last time I looked at this there was (and earlier thread). Are you able to test it on Broadwell? (Or with an older Media SDK version?

Re: [libav-devel] [PATCH] qsv: Load the hw hevc plugin by default on Linux

2017-09-07 Thread Maxym Dmytrychenko
old Media SDK might be considered as outdated, it would be good to support the latest one, at first. On Thu, Sep 7, 2017 at 10:14 PM, Mark Thompson wrote: > On 06/09/17 14:07, Luca Barbato wrote: > > Only on Windows the software plugin is available. > > --- > > > > libavcodec/qsvenc_hevc.c | 8

Re: [libav-devel] [PATCH 1/6] lavc: Add codec metadata to indicate hardware support

2017-09-07 Thread Mark Thompson
On 07/09/17 21:47, wm4 wrote: > On Thu, 7 Sep 2017 20:49:04 +0100 > Mark Thompson wrote: > >> --- >> (Also version bump.) >> >> Written intending to also be useful for encoders, but there won't actually >> be any implementations. >> >> >> doc/APIchanges | 3 +++ >> libavcodec/avcodec.h

Re: [libav-devel] [PATCH 1/6] lavc: Add codec metadata to indicate hardware support

2017-09-07 Thread wm4
On Thu, 7 Sep 2017 20:49:04 +0100 Mark Thompson wrote: > --- > (Also version bump.) > > Written intending to also be useful for encoders, but there won't actually be > any implementations. > > > doc/APIchanges | 3 +++ > libavcodec/avcodec.h | 69 > ++

Re: [libav-devel] [PATCH 11/14] lavc/vaapi: Support context reuse

2017-09-07 Thread wm4
On Thu, 7 Sep 2017 21:04:16 +0100 Mark Thompson wrote: > On 06/09/17 21:28, wm4 wrote: > > On Wed, 6 Sep 2017 21:07:53 +0100 > > Mark Thompson wrote: > > > >> On 06/09/17 17:27, wm4 wrote: > >>> On Wed, 6 Sep 2017 16:49:11 +0100 > >>> Mark Thompson wrote: > >>> > On 06/09/17 15:3

Re: [libav-devel] [PATCH] qsv: Load the hw hevc plugin by default on Linux

2017-09-07 Thread Mark Thompson
On 06/09/17 14:07, Luca Barbato wrote: > Only on Windows the software plugin is available. > --- > > libavcodec/qsvenc_hevc.c | 8 +++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/qsvenc_hevc.c b/libavcodec/qsvenc_hevc.c > index 27ff0c6feb..efed4d98f4 100644

Re: [libav-devel] [PATCH 11/14] lavc/vaapi: Support context reuse

2017-09-07 Thread Mark Thompson
On 06/09/17 21:28, wm4 wrote: > On Wed, 6 Sep 2017 21:07:53 +0100 > Mark Thompson wrote: > >> On 06/09/17 17:27, wm4 wrote: >>> On Wed, 6 Sep 2017 16:49:11 +0100 >>> Mark Thompson wrote: >>> On 06/09/17 15:33, wm4 wrote: > On Tue, 5 Sep 2017 23:59:30 +0100 > Mark Thompson wro

[libav-devel] [PATCH 5/6] lavc: Delete all fake hwaccels

2017-09-07 Thread Mark Thompson
They are now unused. --- They are technically externally visible (via av_hwaccel_next()), but hopefully noone is insane enough to actually use that. If it is decided that they do actually need to persist, they should probably all get moved into a separate file to separate them from actually-used

[libav-devel] [PATCH 4/6] lavc: Use hardware config information in ff_get_format()

2017-09-07 Thread Mark Thompson
This removes the dependency that hardware pixel formats previously had on AVHWAccel instances, meaning only those which actually do something need exist after this patch. Also updates avcodec_default_get_format() to be able to choose hardware formats if either a matching device has been supplied o

[libav-devel] [PATCH 6/6] avconv: Use codec hardware config to configure hwaccels

2017-09-07 Thread Mark Thompson
Removes specific support for all hwaccels supported by the generic code (CUVID, DXVA2, D3D11VA, VAAPI and VDPAU). --- Also restores "-hwaccel auto" support. avtools/avconv.c | 77 +++- avtools/avconv.h | 9 +- avtools/avconv_hw.c | 245 ++--

[libav-devel] [PATCH 2/6] lavc: Add hardware config metadata for decoders supporting hardware output

2017-09-07 Thread Mark Thompson
--- Mostly mechanical to define the possible configurations. Patch could also be split if that is preferred. libavcodec/h263dec.c | 10 ++ libavcodec/h264dec.c | 19 +++ libavcodec/hevcdec.c | 19 +++ libavcodec/hwaccel.h | 26

[libav-devel] [PATCH 3/6] webp: Fix alpha initialisation

2017-09-07 Thread Mark Thompson
ff_get_format() in the next patch will reject formats which aren't in the offered list, so the hack in 7cb9296db872c4221453e5411f242ebcfca62664 is no longer valid. Change the hack by adding a new field in the VP8 decoder context to indicate that it's actually WebP and don't call ff_get_format() at

[libav-devel] [PATCH 1/6] lavc: Add codec metadata to indicate hardware support

2017-09-07 Thread Mark Thompson
--- (Also version bump.) Written intending to also be useful for encoders, but there won't actually be any implementations. doc/APIchanges | 3 +++ libavcodec/avcodec.h | 69 2 files changed, 72 insertions(+) diff --git a/doc/APIchan