 |
 |
Well, because at the time the code was written (oh about 18 years ago) pthread_timed_lock didn’t exist in Solaris. From: Brian Bockelman Sent: Tuesday, August 28, 2018 6:43 PM To: xrootd/xrootd Cc: Subscribed Subject: Re: [xrootd/xrootd] XrdHttp loses requests under modest concurrency (#810) The ofs_Stall is not coming from the OSS but from here: https://github.com/xrootd/xrootd/blob/master/src/XrdOfs/XrdOfsHandle.cc#L203 It appears there's a modest contention on the file descriptor table -- one that does not particularly play well with what appears to be an ad-hoc implementation of a timed lock: https://github.com/xrootd/xrootd/blob/master/src/XrdOfs/XrdOfsHandle.cc#L504 It's not obvious why one would utilize that instead of a wrapper around pthread_mutex_timedlock; it appears the hand-rolled version has similar guarantees as the standard function but worse performance. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/xrootd/xrootd/issues/810#issuecomment-416797323 ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the XROOTD-DEV list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1