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

[Michael Niedermayer]

On Mon, Apr 01, 2013 at 08:50:42PM -0300, James Almer wrote:
> On 01/04/13 8:37 AM, Michael Niedermayer wrote:
> > On Mon, Apr 01, 2013 at 12:23:51PM +0200, Stefano Sabatini wrote:
> >> On date Monday 2013-04-01 17:52:13 +0800, Wei Gao encoded:
> >>>
> >>
> >>> From 8d7554e834fbb36bdfaaf384905320bce5bc29f0 Mon Sep 17 00:00:00 2001
> >>> From: highgod0401 <highgod0401 at gmail.com>
> >>> Date: Mon, 1 Apr 2013 17:46:11 +0800
> >>> Subject: [PATCH 1/2] opencl wrapper based on comments on 20130401
> >>>
> >>> ---
> >>>  configure          |   4 +
> >>>  libavutil/Makefile |   3 +
> >>>  libavutil/opencl.c | 719 +++++++++++++++++++++++++++++++++++++++++++++++++++++
> >>>  libavutil/opencl.h | 201 +++++++++++++++
> >>>  4 files changed, 927 insertions(+)
> >>>  create mode 100644 libavutil/opencl.c
> >>>  create mode 100644 libavutil/opencl.h
> >>
> >> Missing APIchanges entry and libavutil minor bump, but these can be
> >> added by the committer.
> > 
> > I didnt bump libavutil as i think the API/ABI is not stable yet
> > 
> > [...]
> >> No more comments from me, we can fix it later in case of need as the
> >> API is marked as experimental, and other minor formatting nits can be
> >> fixed when it's already in.
> > 
> > patch applied
> 
> Since unlike in atomic.c we have init() and uninit() functions here, there's no need 
> to have a statically allocated mutex. We could call pthread_mutex_init() and

the init and uninit themselfs must be thread safe, also register is
called before init and must be thread safe

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130402/a4725da5/attachment.asc>