[vlc-devel] [PATCH 04/17] dxva: split the directx related parts from the pool/va parts

[Steve Lhomme]

On Tue, Jun 20, 2017 at 6:31 PM, Rémi Denis-Courmont <remi at remlab.net> wrote:
> Le tiistaina 20. kesäkuuta 2017, 17.45.36 EEST Steve Lhomme a écrit :
>> ---
>>  modules/codec/Makefile.am                   |   2 +
>>  modules/codec/avcodec/d3d11va.c             |  89 ++++++-------
>>  modules/codec/avcodec/directx_va.c          | 151 +--------------------
>>  modules/codec/avcodec/directx_va.h          |  49 +------
>>  modules/codec/avcodec/dxva2.c               |  76 +++++------
>>  modules/codec/avcodec/va_surface.c          | 198
>> ++++++++++++++++++++++++++++ modules/codec/avcodec/va_surface.h          |
>> 41 ++++++
>>  modules/codec/avcodec/va_surface_internal.h |  91 +++++++++++++
>
> IMO, unfortunate files naming for DX-specific stuff insofar as the directory
> is not DX- nor Windows-specific.

It's on purpose. The goal being that all libavcodec va use this in the
end. The algorithm and logic is the same as what is found in VDPAU.
After refactoring the code I manage to use the same thing.

Still undetermined is whether the context is responsible for freeing
the surfaces and associated resources. Or when closing the module. In
VDPAU it's the former and in my code the latter.

> --
> 雷米‧德尼-库尔蒙
> https://www.remlab.net/
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel