Re: [Mesa-dev] [PATCH] clover: Fix bug with computing hard_event status

2015-07-16 Thread Tom Stellard
On Sat, Jul 11, 2015 at 02:35:53PM +0300, Francisco Jerez wrote: > Tom Stellard writes: > > > pipe_context::flush() can return a NULL fence if the queue is already > > empty, so we should not assume that an event with a NULL fence > > has the status of CL_QUEUED. > > > > This seems suspicious...

Re: [Mesa-dev] [PATCH] clover: Fix bug with computing hard_event status

2015-07-11 Thread Francisco Jerez
Tom Stellard writes: > pipe_context::flush() can return a NULL fence if the queue is already > empty, so we should not assume that an event with a NULL fence > has the status of CL_QUEUED. > This seems suspicious... On the one hand it doesn't seem to be a documented "feature" of pipe_context::f

[Mesa-dev] [PATCH] clover: Fix bug with computing hard_event status

2015-07-10 Thread Tom Stellard
pipe_context::flush() can return a NULL fence if the queue is already empty, so we should not assume that an event with a NULL fence has the status of CL_QUEUED. CC: 10.6 --- src/gallium/state_trackers/clover/core/event.cpp | 7 --- src/gallium/state_trackers/clover/core/event.hpp | 1 + 2 f