On Tue, 9 Mar 2010, Christian Brunner wrote:

> > > Yehuda is taking a look at it now.  The signals are used to wake up 
> > > blocking read()/write()/connect(), but we should be able to use poll() or 
> > > select() instead (and clean things up in the process).
> > 
> > Great! - That was really a quick response. I will check out the current
> > git version tomorrow and will give it a try.
> 
> I couldn't resist to try it today. I had to make a minor change regarding
> pthread handling. Now everything seems to work well. An updated patch
> is attached. - Thanks for the quick solution.

Oh, I see the issue.  I changed the Ceph Thread class to mask signals on 
any new threads it creates, so you shouldn't need to worry about it all 
anymore.  (I think you got lucky before: librados does create threads for 
connecting to servers as it needs them, and those signals weren't getting 
masked.)

> I haven't changed the naming scheme yet. I will do this in a few days.

Sounds good.

Thanks!
sage

------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Ceph-devel mailing list
Ceph-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ceph-devel

Reply via email to