[FFmpeg-devel] [PATCH 1/2] libavutil/libavfilter: opencl wrapper based on comments on 20130325

Stefano Sabatini stefasab at gmail.com
Tue Mar 26 19:48:48 CET 2013


On date Tuesday 2013-03-26 11:44:30 +0800, Wei Gao encoded:
> Hi,
> 
> Stefano Sabatini, thank you for your reply, some questions and explanations
> as follows
> 
> Thanks
> Best regards
> 
> 2013/3/26 Stefano Sabatini <stefasab at gmail.com>
> 
> > On date Monday 2013-03-25 19:00:24 +0800, Wei Gao encoded:
> > >
> >
> > > From fb53b21efb819336e30eb50bc4408b3c80111e82 Mon Sep 17 00:00:00 2001
> > > From: highgod0401 <highgod0401 at gmail.com>
> > > Date: Mon, 25 Mar 2013 18:52:51 +0800
> > > Subject: [PATCH 1/2] opencl wrapper based on comments on 20130325
> > >
> > > ---
> > >  configure          |   4 +
> > >  libavutil/Makefile |   3 +
> > >  libavutil/opencl.c | 967
> > +++++++++++++++++++++++++++++++++++++++++++++++++++++
> > >  libavutil/opencl.h | 219 ++++++++++++
[...]

Note please, cut parts of the mail which are unrelevant, like done
above, so we have a better signal-to-noise ratio, and save bandwidth.

> > About the whole logic. This is accessing some files in the local
> > path. This should be avoided. We have several choice:
> > - we avoid this access, if it can be avoided, in the first version
> > - we specify the filepath in a sort of global context
> >
> > In the latter case we need to understand how to create such global
> > context.
> >

> can you agree that we just compile every time? I thinks the most important
> thing in this step is make the opencl code run.

Yes, also should simplify review and speed up integration.
-- 
FFmpeg = Fancy & Formidable Mortal Portable Extensive Gem


More information about the ffmpeg-devel mailing list