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
Converted MultiTrackSelector from global to stream #5324
Converted MultiTrackSelector from global to stream #5324
Conversation
Helgrind identified a race condition in MultiTrackSelector. The problem was StringCutObjectSelector is not const thread safe (i.e. it has a const function which modifies its internal state) and therefore only one thread at a time can call it. Therefore MultiTrackSelector can not simultaneously talk to multiple events. However, converting the module to a stream module is thread safe.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_2_X. Converted MultiTrackSelector from global to stream It involves the following packages: RecoTracker/FinalTrackSelectors @cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks. |
too bad... |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine). @nclopezo can you please take care of it? |
…odule Converted MultiTrackSelector from global to stream
Helgrind identified a race condition in MultiTrackSelector. The problem
was StringCutObjectSelector is not const thread safe (i.e. it has a const
function which modifies its internal state) and therefore only one thread
at a time can call it. Therefore MultiTrackSelector can not simultaneously
talk to multiple events which means it can not be a global module. However,
converting the module to a stream module is thread safe.