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.

Sign up for GitHub

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

Jump to bottom

70X memory leak in DTMeantimerPatternReco #22

Closed
slava77 opened this issue Jan 22, 2014 · 1 comment
Closed

70X memory leak in DTMeantimerPatternReco #22

slava77 opened this issue Jan 22, 2014 · 1 comment
Labels
memory

Comments

@slava77
Copy link
Owner

slava77 commented Jan 22, 2014

Visible in CMSSW_7_0_X_2014-01-17-0200 (should be in 700pre12) in TTBar 40PU25ns test run

Concise report in
https://slava77sk.web.cern.ch/slava77sk/reco/valgrind_70X-201412170200.txt

    139,776 (36,864 direct, 102,912 indirect) bytes in 256 blocks are definitely lost in loss record 151,393 of 151,598
      operator new(unsigned long)
      DTSegmentCand::operator DTChamberRecSegment2D*() const
      DTMeantimerPatternReco4D::reconstruct()
      DTRecSegment4DProducer::produce(edm::Event&, edm::EventSetup const&)

superphi is never deleted in DTMeantimerPatternReco4D::reconstruct()

    76,352 bytes in 1,193 blocks are definitely lost in loss record 151,244 of 151,598
      operator new(unsigned long)
      DTMeantimerPatternReco::initHits(DTSuperLayer const*, std::vector<DTRecHit1DPair...
      DTMeantimerPatternReco4D::buildPhiSuperSegmentsCandidates()
      DTMeantimerPatternReco4D::reconstruct()
      DTRecSegment4DProducer::produce(edm::Event&, edm::EventSetup const&)

pairPhi2 and pairPhi1 should be cleaned up

    72,192 bytes in 1,128 blocks are definitely lost in loss record 151,232 of 151,598
      operator new(unsigned long)
      DTMeantimerPatternReco::initHits(DTSuperLayer const*, std::vector<...
      DTMeantimerPatternReco4D::buildPhiSuperSegmentsCandidates()
      DTMeantimerPatternReco4D::reconstruct()
      DTRecSegment4DProducer::produce(edm::Event&, edm::EventSetup const&)

pairPhi2 and pairPhi1 should be cleaned up

    61,184 bytes in 956 blocks are definitely lost in loss record 151,198 of 151,598
      operator new(unsigned long)
      DTMeantimerPatternReco::initHits(DTSuperLayer const*, std::vector<DTRecHit1DPair... 
      DTMeantimerPatternReco::reconstruct(DTSuperLayer const*, std::vector<DTRecHit1DPair,...      DTMeantimerPatternReco4D::reconstruct()
      DTRecSegment4DProducer::produce(edm::Event&, edm::EventSetup const&)

hitsForFit should be cleaned up in the end
Full report in
https://slava77sk.web.cern.ch/slava77sk/reco/valgrind_70X-201412170200.xml
@ptraczyk ptraczyk mentioned this issue Jan 29, 2014
Merged
@slava77
Copy link
Owner Author

slava77 commented Feb 6, 2014

fixed in cms-sw#2221 492a05f
sign303

@slava77 slava77 closed this as completed Feb 6, 2014
slava77 pushed a commit that referenced this issue May 29, 2014
@jshlee
Merge pull request #22 from calabria/CMSSW_6_2_X_SLHC
b43f4b7 
Update GEMEtaPartitionMask.h
slava77 pushed a commit that referenced this issue May 29, 2014
Merge pull request #22 from dildick/from-CMSSW_6_2_X_SLHC_2014-03-08-…
ae4400d 
…1400

From cmssw 6 2 x slhc 2014 03 08 1400
slava77 pushed a commit that referenced this issue Apr 12, 2015
Merge pull request #22 from cms-l1t-offline/EtSumAlg-devel-CMSSW_7_4_…
3b347ba 
…0_pre5

Stage 1 ET sum algorithm development finished except dijet phi (iPhi HT)
slava77 pushed a commit that referenced this issue Feb 11, 2016
@isobelojalvo
Merge pull request #22 from cgalloni/80ReMiniAODwBoostedTaus_forPull_…
8f10923 
…v800pre4

missing keep statement added
slava77 pushed a commit that referenced this issue Apr 20, 2016
@lgray
Merge pull request #22 from kpedro88/workflowsp2_81X_localreco
007b5de 
restore tilted tracker in 2023dev
slava77 pushed a commit that referenced this issue Oct 9, 2017
@gpetruc
Merge pull request #29 from cms-nanoAOD/nano_94X_fixes
a8aba11 
PR cms-sw#20626 code review + #22
slava77 pushed a commit that referenced this issue Oct 9, 2021
@cerati
Merge pull request #22 from cerati/mk-build-improve
9db272a 
Mk build improve
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
memory
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@slava77