Re: [Mesa-dev] [PATCH] clover: Increment an object's reference count in ret_object()

2012-06-28 Thread Tom Stellard
On Thu, Jun 28, 2012 at 03:47:23PM +0200, Francisco Jerez wrote: > Tom Stellard writes: > > > On Thu, Jun 28, 2012 at 01:09:20AM +0200, Francisco Jerez wrote: > >> Tom Stellard writes: > >> > >> > We need to increment the reference count for objects, like cl_event, > >> > that the user is respo

Re: [Mesa-dev] [PATCH] clover: Increment an object's reference count in ret_object()

2012-06-28 Thread Francisco Jerez
Tom Stellard writes: > On Thu, Jun 28, 2012 at 01:09:20AM +0200, Francisco Jerez wrote: >> Tom Stellard writes: >> >> > We need to increment the reference count for objects, like cl_event, >> > that the user is responsible for destroying when they are returned from >> > the API. Otherwise, the

Re: [Mesa-dev] [PATCH] clover: Increment an object's reference count in ret_object()

2012-06-28 Thread Tom Stellard
On Thu, Jun 28, 2012 at 01:09:20AM +0200, Francisco Jerez wrote: > Tom Stellard writes: > > > We need to increment the reference count for objects, like cl_event, > > that the user is responsible for destroying when they are returned from > > the API. Otherwise, the object will be destroyed when

Re: [Mesa-dev] [PATCH] clover: Increment an object's reference count in ret_object()

2012-06-27 Thread Francisco Jerez
Tom Stellard writes: > We need to increment the reference count for objects, like cl_event, > that the user is responsible for destroying when they are returned from > the API. Otherwise, the object will be destroyed when clover is done with > it, even though the user will still have a reference

[Mesa-dev] [PATCH] clover: Increment an object's reference count in ret_object()

2012-06-27 Thread Tom Stellard
We need to increment the reference count for objects, like cl_event, that the user is responsible for destroying when they are returned from the API. Otherwise, the object will be destroyed when clover is done with it, even though the user will still have a reference to it. For example: 1. clEnq