Hi Markus,

> The third issue the second patch fixes is something I had quite hard time
> to figure out. It seems that rapi contexts are created and stored into TLS
> so they are per thread. So when the rapi context is created in main thread
> in kernel.py and used later in another thread in synchandler.py, librapi
> method rapi_context_current() returns new, uninitialized context, that
> results in sync-engine crash. I fixed this by creating and using a new rapi
> context in SyncHandler.run() method. This makes the thing work.

Thank you for this patch. I don't have the hal-dccm working, so I can't test 
the other patches. I did apply this patch you provided in the current SVN 
because indeed it was a problem that I was encountering also lately. 

Recently I updated the librapi2 such that it uses TLS because one todo that 
was still open for quite some time was to make the library thread safe. 
Unfortunately, making the lib threadsafe caused segfaults in sync-engine that 
I could not figure out. Thanks to your patch that problem is gone! 

Thank you,

Guido Diepen




-- 
Aviation is proof that given the will, we have the 
capacity to achieve the impossible.
     --Eddie Rickenbacker

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
SynCE-Devel mailing list
SynCE-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/synce-devel

Reply via email to