You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2020/11/18 00:05:49 UTC
[GitHub] [trafficserver] ywkaras opened a new pull request #7330: Add ts::BriefMutex utility class.
ywkaras opened a new pull request #7330:
URL: https://github.com/apache/trafficserver/pull/7330
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] ywkaras commented on pull request #7330: Add ts::AltMutex utility class.
Posted by GitBox <gi...@apache.org>.
ywkaras commented on pull request #7330:
URL: https://github.com/apache/trafficserver/pull/7330#issuecomment-773449672
Note that this video, https://youtu.be/ZQFzMfHIxng , reports good performance using spinlocks. This class is basically a spinlock. It has yields to handle the fact the thread currently holding the mutex may get preempted. It has (micro-)sleeps to handle the possibility that the thread holding the lock gets preempted and has a lower scheduling priority than a thread waiting on the mutex. I think currently all our threads have the same priority, so the sleeps may only be for future safety.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] ywkaras closed pull request #7330: Add ts::AltMutex utility class.
Posted by GitBox <gi...@apache.org>.
ywkaras closed pull request #7330:
URL: https://github.com/apache/trafficserver/pull/7330
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] ywkaras commented on pull request #7330: Add ts::AltMutex utility class.
Posted by GitBox <gi...@apache.org>.
ywkaras commented on pull request #7330:
URL: https://github.com/apache/trafficserver/pull/7330#issuecomment-773449672
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] ywkaras closed pull request #7330: Add ts::AltMutex utility class.
Posted by GitBox <gi...@apache.org>.
ywkaras closed pull request #7330:
URL: https://github.com/apache/trafficserver/pull/7330
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] ywkaras commented on pull request #7330: Add ts::AltMutex utility class.
Posted by GitBox <gi...@apache.org>.
ywkaras commented on pull request #7330:
URL: https://github.com/apache/trafficserver/pull/7330#issuecomment-773638783
Closing this, will reopen if we use it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org