Thanks Lukasz ... we will try this tomorrow then. Where do we insert the barrier, before
a) the if in https://github.com/xrootd/xrootd/blob/master/src/XrdSys/XrdSysPthread.hh#L341
or really before
b) https://github.com/xrootd/xrootd/blob/master/src/XrdCl/XrdClMessageUtils.hh#L66
? In case b) it will also be there for the libc semaphores that seem to be ok now.
With glibc-2.22 things work ok, the helgrind reports about SyncHandler are gone.
A question: why are semaphores better than condition variables in this case?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/xrootd/xrootd","title":"xrootd/xrootd","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/xrootd/xrootd"}},"updates":{"snippets":[{"icon":"PERSON","message":"@osschar in #404: Thanks Lukasz ... we will try this tomorrow then. Where do we insert the barrier, before\r\na) the if in https://github.com/xrootd/xrootd/blob/master/src/XrdSys/XrdSysPthread.hh#L341\r\nor really before\r\nb) https://github.com/xrootd/xrootd/blob/master/src/XrdCl/XrdClMessageUtils.hh#L66\r\n? In case b) it will also be there for the libc semaphores that seem to be ok now.\r\n\r\nWith glibc-2.22 things work ok, the helgrind reports about SyncHandler are gone.\r\n\r\nA question: why are semaphores better than condition variables in this case?"}],"action":{"name":"View Issue","url":"https://github.com/xrootd/xrootd/issues/404#issuecomment-240942773"}}}
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