Raster, On Fri, Aug 30, 2013 at 5:11 AM, Carsten Haitzler <ras...@rasterman.com> wrote: > On Thu, 29 Aug 2013 10:26:50 -0300 Ulisses Furquim <uliss...@gmail.com> said: > >> Raster, >> >> On Thu, Aug 29, 2013 at 9:18 AM, Carsten Haitzler - Enlightenment Git >> <no-re...@enlightenment.org> wrote: >> > raster pushed a commit to branch master. >> > >> > commit 42a46214c4f9b35c0e1f5a84c56ea76ba2235eae >> > Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com> >> > Date: Thu Aug 29 21:18:04 2013 +0900 >> > >> > other async render issue - sync ALL rendering canvases, not just one >> > --- >> > src/lib/evas/canvas/evas_render.c | 2 ++ >> > src/lib/evas/common/evas_thread_render.c | 21 ++++++++++++++++++++- >> > src/lib/evas/include/evas_common_private.h | 4 +++- >> > 3 files changed, 25 insertions(+), 2 deletions(-) >> > >> > diff --git a/src/lib/evas/canvas/evas_render.c >> > b/src/lib/evas/canvas/evas_render.c index b04d606..4fbe9e2 100644 >> > --- a/src/lib/evas/canvas/evas_render.c >> > +++ b/src/lib/evas/canvas/evas_render.c >> > @@ -2235,6 +2235,7 @@ _canvas_render_dump(Eo *eo_e EINA_UNUSED, void *_pd, >> > va_list *list EINA_UNUSED) Evas_Public_Data *e = _pd; >> > Evas_Layer *lay; >> > >> > + evas_thread_queue_block(); >> > evas_render_rendering_wait(e); >> > evas_cache_async_freeze(); >> >> This might deadlock. If the queue is no empty and we have there the >> commands for e then we'll wait forever in _rendering_wait() in the >> main thread and the render thread will sleep forever trying to acquire >> the evas_thread_block_lock. Right? > > fixed. i didnt notice evas->rendering was set in evas_render as opposed to the > rendering handle in the async thread. :)
Well, yes, your change solves the deadlock. But I want to know what you want, really. Do you want a call to sync all canvases? This lock is not doing that. We'd need to wait all canvases rendering flags to be set to false and then return. This way we wouldn't have any pending commands in the render thread. -- Ulisses ------------------------------------------------------------------------------ Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more! Discover the easy way to master current and previous Microsoft technologies and advance your career. Get an incredible 1,500+ hours of step-by-step tutorial videos with LearnDevNow. Subscribe today and save! http://pubads.g.doubleclick.net/gampad/clk?id=58040911&iu=/4140/ostg.clktrk _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
