Skip to content
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

Disable the triggerCache in the multithreaded environment. #1939

Merged
merged 1 commit into from Jan 3, 2014

Conversation

wmtan
Copy link
Contributor

@wmtan wmtan commented Jan 3, 2014

There are two separate caches used by PoolSource for reading events in input files. The secondary one, called the trigger cache, is used only in unusual circumstances. However, in the multithreaded environment, having the trigger cache enabled can cause occasional fatal asserts due to a yet undiscovered bug in its implementation. Since there is doubt that the trigger cache is useful in a multithreaded environment, this pull request simply disables it for now. This eliminates the asserts.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 3, 2014

A new Pull Request was created by @wmtan for CMSSW_7_0_THREADED_X.

Disable the triggerCache in the multithreaded environment.

It involves the following packages:

IOPool/Input

@cmsbuild, @Dr15Jones, @ktf, @nclopezo can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.

@Dr15Jones
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 3, 2014

This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests.

ktf added a commit that referenced this pull request Jan 3, 2014
Multithreading fixes -- Disable the triggerCache in the multithreaded environment.
@ktf ktf merged commit edbf40b into cms-sw:CMSSW_7_0_THREADED_X Jan 3, 2014
@wmtan wmtan deleted the DisableTriggerCache branch January 7, 2014 04:30
ggovi pushed a commit to ggovi/cmssw that referenced this pull request Jan 11, 2017
Updated root to tip of v6-02-00-patches
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants