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
Bugfix new CaloLayer1 unpacker #21486
Conversation
Fixes cms-sw#21067 l1t::Block::payload() returns a copy of internal std::vector which we grabbed a bare pointer to, but then it is destructed since it is an rvalue, so the things we do with that pointer are rather naughty. Fixed by returning const reference, as all access patterns are read-only anyway. Propagated const-ness to bare pointer in the UCTCTP7RawData utility.
The code-checks are being triggered in jenkins. |
+code-checks |
A new Pull Request was created by @nsmith- (Nicholas Smith) for master. It involves the following packages: EventFilter/L1TRawToDigi @cmsbuild, @rekovic, @mulhearn can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test Let's see if this works. |
The tests are being triggered in jenkins. |
+1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
Comparison job queued. |
+1 |
@thomreis only in that they will be faster now. |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar (and backports should be raised in the release meeting by the corresponding L2) |
Follow-up: no other invalid reads in cmsDriver.py digi -s RAW2DIGI --eventcontent=FEVTDEBUGHLT --filein=/store/data/Run2017G/HLTPhysics/RAW/v1/000/306/584/00000/1277E19C-D2C8-E711-AA4F-02163E01A5C7.root --data -n 10 --conditions=auto:run2_data --no_exec
time valgrind --tool=memcheck --suppressions=$CMSSW_RELEASE_BASE/src/Utilities/ReleaseScripts/data/cms-valgrind-memcheck.supp --num-callers=20 --xml=yes --xml-file=valgrind.xml cmsRun digi_RAW2DIGI.py although there seem to be plenty of other leaks, |
Comparison is ready There are some workflows for which there are errors in the baseline: Comparison Summary:
|
+1 |
Fixes #21067
l1t::Block::payload() returns a copy of internal std::vector
which we grabbed a bare pointer to, but then it is destructed since it
is an rvalue, so the things we do with that pointer are rather naughty.
Fixed by returning const reference, as all access patterns are read-only
anyway. Propagated const-ness to bare pointer in the UCTCTP7RawData
utility.
p.s. introduced in #20914, backport to 94x necessary I suppose?