You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@trafficserver.apache.org by shinrich <gi...@git.apache.org> on 2015/06/26 20:16:57 UTC
[GitHub] trafficserver pull request: TS-3486: Crashes due to race condition...
GitHub user shinrich opened a pull request:
https://github.com/apache/trafficserver/pull/235
TS-3486: Crashes due to race condition on server sessions moving betw…
…een threads.
Addressing crash issues disucssed in TS-3266 and TS-3486. This fix has been running on Y! 5.3.x on one production machine for over a day and on three other production machines for several hours.
There is a race condition when server sessions reused from a global pool are moved between threads. This patch addresses the issue by holding both the session pool lock and the activating state machine lock while setting up the first read.vio on the reactivated server. This prevents conflicting IO using one lock or the other on another thread.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/shinrich/trafficserver ts-3486
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/trafficserver/pull/235.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #235
----
commit 8c1a8631b92f8c7b1bbe2e47835575524f1fba8c
Author: shinrich <sh...@yahoo-inc.com>
Date: 2015-06-26T18:11:40Z
TS-3486: Crashes due to race condition on server sessions moving between threads.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] trafficserver pull request: TS-3486: Crashes due to race condition...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/trafficserver/pull/235
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---