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

Pr80x L1T memory manage XMLConfigReader last 150 mb #16250

Conversation

rekovic
Copy link
Contributor

@rekovic rekovic commented Oct 18, 2016

This is the back-port of the memory management fixes in L1T XMLConfigReader
and fixes for the subsequent worfkflow crashes uncovered.
Already in 81x (#16222)

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @rekovic for CMSSW_8_0_X.

It involves the following packages:

DataFormats/L1Trigger
L1Trigger/L1TCommon
L1Trigger/L1TMuonOverlap

@cmsbuild, @rekovic, @mulhearn, @davidlange6 can you please review it and eventually sign? Thanks.
@Martin-Grunewald this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are listed here #13028

@rekovic rekovic changed the title Pr80x memory manage xml config reader last150 mb Pr80x L1T memory manage XMLConfigReader last 150 mb Oct 18, 2016
@rekovic
Copy link
Contributor Author

rekovic commented Oct 18, 2016

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 18, 2016

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/15794/console

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@davidlange6 davidlange6 merged commit ba23892 into cms-sw:CMSSW_8_0_X Oct 19, 2016
@kkotov
Copy link
Contributor

kkotov commented Oct 19, 2016

Hi @rekovic , I confirm that the L1T O2O payloads generated with this PR are identical to those generated a week earlier with #16198

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