New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make HcalDDDGeometry thread-safe #4167
Make HcalDDDGeometry thread-safe #4167
Conversation
Made caching of values internal to HcalDDDGeometry thread-safe. Given that the caching is done only once and kept for a long time and only one HcalDDDGeometry tends to be used at a time a std::mutex was used to synchronize the filling of the cache. This minimized the amount of code that had to be changed.
This is a bug fix since we already have stream based modules in CMSSW_7_1_X which use this object and therefore are not thread-safe without the fix. |
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_1_X. Make HcalDDDGeometry thread-safe It involves the following packages: Geometry/HcalTowerAlgo @civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @nclopezo, @Degano, @ktf can you please review it and eventually sign? Thanks. |
+1 |
Make HcalDDDGeometry thread-safe
Made caching of values internal to HcalDDDGeometry thread-safe. Given that the caching is done only once and kept for a long time and only one HcalDDDGeometry tends to be used at a time a std::mutex was used to synchronize the filling of the cache. This minimized the amount of code that had to be changed.