Re: [libav-devel] [PATCH] NVENC Input Surface Optimization

2017-06-22 Thread Luca Barbato
On 6/23/17 1:47 AM, Ben Chang wrote: > Hi, > > Please help review this nvenc patch for input surface optimization. > > This patch aims to reduce the number of input/output surfaces NVENC > allocates per session. Previous default sets allocated surfaces to 32 > (unless there is user specified

[libav-devel] [PATCH] NVENC Input Surface Optimization

2017-06-22 Thread Ben Chang
Hi, Please help review this nvenc patch for input surface optimization. This patch aims to reduce the number of input/output surfaces NVENC allocates per session. Previous default sets allocated surfaces to 32 (unless there is user specified param or lookahead involved). Having large number of

Re: [libav-devel] [PATCH] bink: Cast runtime-generated quant tables to const

2017-06-22 Thread Diego Biurrun
On Thu, Jun 22, 2017 at 11:24:50PM +0300, Rémi Denis-Courmont wrote: > On jeudi 22 juin 2017 21:12:00 EEST Diego Biurrun wrote: > > On Wed, Jun 21, 2017 at 07:10:55PM +0300, Rémi Denis-Courmont wrote: > > > Le keskiviikkona 21. kesäkuuta 2017, 18.07.05 EEST Diego Biurrun a écrit : > > > > On Wed,

Re: [libav-devel] [PATCH] bink: Cast runtime-generated quant tables to const

2017-06-22 Thread Diego Biurrun
On Wed, Jun 21, 2017 at 07:10:55PM +0300, Rémi Denis-Courmont wrote: > Le keskiviikkona 21. kesäkuuta 2017, 18.07.05 EEST Diego Biurrun a écrit : > > On Wed, Jun 21, 2017 at 07:05:45PM +0300, Rémi Denis-Courmont wrote: > > > Le keskiviikkona 21. kesäkuuta 2017, 13.10.42 EEST Diego Biurrun a écrit

Re: [libav-devel] [PATCH 6/8] hwcontext_d3d11va: add option to enable debug mode

2017-06-22 Thread Luca Barbato
On 6/22/17 3:15 PM, wm4 wrote: > Right. Can whoever applies this fix it? Amended and put on oracle. If it survives I'll push it with the 7/8 with the fate test updated. lu ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 7/8] hevcdec: fix cropping with new hwaccel API

2017-06-22 Thread Luca Barbato
On 6/22/17 2:52 PM, wm4 wrote: > Affects in particular DXVA2/D3D11VA, which asks for 128 pixels > alignment. The AVHWFramesContext will return AVFrames with that larger > size set. We need to crop it back in the decoder. > --- > libavcodec/hevc_refs.c | 3 +++ > 1 file changed, 3 insertions(+) >

Re: [libav-devel] [PATCH 6/8] hwcontext_d3d11va: add option to enable debug mode

2017-06-22 Thread wm4
On Thu, 22 Jun 2017 16:02:10 +0300 (EEST) Martin Storsjö wrote: > On Thu, 22 Jun 2017, wm4 wrote: > > > Basically copied from VLC (LGPL): > > > >

Re: [libav-devel] [PATCH 6/8] hwcontext_d3d11va: add option to enable debug mode

2017-06-22 Thread Martin Storsjö
On Thu, 22 Jun 2017, wm4 wrote: Basically copied from VLC (LGPL): http://git.videolan.org/?p=vlc.git;a=blob;f=modules/video_output/win32/direct3d11.c;h=e9fcb83dcabfe778f26e63d19f218caf06a7c3ae;hb=HEAD#l1482

[libav-devel] [PATCH 8/8] dxva: verbose-log decoder GUID list

2017-06-22 Thread wm4
Helpful for debugging. --- libavcodec/dxva2.c | 47 +++ 1 file changed, 47 insertions(+) diff --git a/libavcodec/dxva2.c b/libavcodec/dxva2.c index 79b77150d0..a60604a84f 100644 --- a/libavcodec/dxva2.c +++ b/libavcodec/dxva2.c @@ -196,12 +196,59 @@

[libav-devel] [PATCH 7/8] hevcdec: fix cropping with new hwaccel API

2017-06-22 Thread wm4
Affects in particular DXVA2/D3D11VA, which asks for 128 pixels alignment. The AVHWFramesContext will return AVFrames with that larger size set. We need to crop it back in the decoder. --- libavcodec/hevc_refs.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/hevc_refs.c

[libav-devel] [PATCH 2/8] dxva: fix some warnings

2017-06-22 Thread wm4
Some existed since forever, some are new. The cast in get_surface() is silly, but unless we change the av_log function signature, or all callers of ff_dxva2_get_surface_index(), it's needed to remove the const warning. --- libavcodec/dxva2.c | 16 +++- 1 file changed, 7

[libav-devel] [PATCH 3/8] hwcontext_d3d11va: fix crash on frames_init failure

2017-06-22 Thread wm4
It appears in this case, frames_ininit is called twice (once by av_hwframe_ctx_init(), and again by unreffing the frames ctx ref). --- libavutil/hwcontext_d3d11va.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavutil/hwcontext_d3d11va.c b/libavutil/hwcontext_d3d11va.c index

[libav-devel] [PATCH 5/8] dxva: support DXGI_FORMAT_420_OPAQUE decoding

2017-06-22 Thread wm4
Some devices (some phones, apparently) will support only this opaque format. Of course this won't work with CLI, because copying data directly is not supported. Automatic frame allocation (setting AVCodecContext.hw_device_ctx) does not support this mode, even if it's the only supported mode. But

[libav-devel] [PATCH 1/8] dxva: add declarative profile checks

2017-06-22 Thread wm4
Make supported codec profiles part of each dxva_modes entry. Every DXVA2 mode is representative for a codec with a subset of supported profiles, so reflecting that in dxva_modes seems appropriate. In practice, this will more strictly check MPEG2 profiles, will stop relying on the surface format

[libav-devel] [PATCH 4/8] hwcontext_d3d11va: allocate staging texture lazily

2017-06-22 Thread wm4
Makes dealing with formats that can not be used for staging textures easier (DXGI_FORMAT_420_OPAQUE). It also saves memory if the staging texture is never needed, so this is a good thing. --- libavutil/hwcontext_d3d11va.c | 46 --- 1 file changed, 34

[libav-devel] [PATCH 6/8] hwcontext_d3d11va: add option to enable debug mode

2017-06-22 Thread wm4
Basically copied from VLC (LGPL): http://git.videolan.org/?p=vlc.git;a=blob;f=modules/video_output/win32/direct3d11.c;h=e9fcb83dcabfe778f26e63d19f218caf06a7c3ae;hb=HEAD#l1482

Re: [libav-devel] [PATCH] configure: Try adding -Werror=unknown-warning-option before testing for -Werror=partial-availability

2017-06-22 Thread Luca Barbato
On 6/21/17 9:32 PM, Martin Storsjö wrote: > Using check_cflags with -Werror parameters in clang is normally > futile, since the tests always pass, but add new warnings instead. > > Since the check for -Werror=partial-availability is in the OS section > of configure, we need to try adding this