[FFmpeg-devel] [PATCH 0/5] avutil/qsv add hwcontext_qsv

[nablet developer]

> On 10 Jun 2016, at 00:48, Michael Niedermayer <michael at niedermayer.cc> wrote:
> 
> On Mon, Jun 06, 2016 at 05:25:41PM +0700, nablet developer wrote:
>> 
>>> On 06 Jun 2016, at 16:40, Mark Thompson <sw at jkqxz.net> wrote:
>>> 
>>> On 06/06/16 09:25, nablet developer wrote:
>>>> ping. any feedback?
>>> 
>>> Perhaps you could comment on the merits and functionality of your version as
>>> compared with the one here
>>> <https://lists.libav.org/pipermail/libav-devel/2016-May/077126.html> (with
>>> associated series) posted at roughly the same time?
>>> 
>> 
>> as I have understood so far, hwcontext in general handles two tasks:
>> 1. context management (context initialisation and cleanup)
>> 2. hardware frame management (initialisation, cleanup and transfer data from/to system memory)
>> 
>> patch from libav handles both 1) and 2).
>> my patches just handle 1) - context management.
>> frame management can be added as well as additional patch set, that's no problem.
>> for my task (VPP filter) just context management was enough for now.
>> 
>> so yes, changes from libav are more complete in term of feature set.
> 
> if the code is overall better than it makes sense to use it instead
> 
> are you interrested in testing and updating our code to that ?
> (aka submit a patchset that switches to the overall better code)

yes, that definitely makes sense for me, I will handle testing/updating and reply back with new patch-set next week then.

> 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> The real ebay dictionary, page 2
> "100% positive feedback" - "All either got their money back or didnt complain"
> "Best seller ever, very honest" - "Seller refunded buyer after failed scam"
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org <mailto:ffmpeg-devel at ffmpeg.org>
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel <http://ffmpeg.org/mailman/listinfo/ffmpeg-devel>