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

Various fixes and cleanup #101

Merged

Conversation

fwyzard
Copy link

@fwyzard fwyzard commented Jul 24, 2018

Fix errors found by cuda-memcheck:

  • properly initialise device memory
  • fix various cudaMemcpy calls

Remove unused debug variables and function declarations, and #ifdef some debug printouts.

Call cudaDeviceReset() before exiting, via the destructor of CUDAService. This explicitly destroys and cleans up all resources associated with the current device, and is useful to check for memory leaks with cuda-memcheck --tool memcheck --leak-check full.

Explicitly destroys and cleans up all resources associated with the
current device in the destructor of the CUDAService.
Useful to check for memory leaks with

  cuda-memcheck --tool memcheck --leak-check full
@fwyzard
Copy link
Author

fwyzard commented Jul 24, 2018

Validation summary

Reference release CMSSW_10_2_0_pre6 at a674e1f
Development branch CMSSW_10_2_X_Patatrack at fa24ba1
Testing PRs:

makeTrackValidationPlots.py plots

/RelValTTbar_13/CMSSW_10_2_0_pre6-PU25ns_102X_upgrade2018_realistic_v7-v1/GEN-SIM-DIGI-RAW

/RelValZMM_13/CMSSW_10_2_0_pre6-102X_upgrade2018_realistic_v7-v1/GEN-SIM-DIGI-RAW

DQM GUI plots

/RelValTTbar_13/CMSSW_10_2_0_pre6-PU25ns_102X_upgrade2018_realistic_v7-v1/GEN-SIM-DIGI-RAW

/RelValZMM_13/CMSSW_10_2_0_pre6-102X_upgrade2018_realistic_v7-v1/GEN-SIM-DIGI-RAW

  • reference DQM plots for reference release, workflow 10824.5
  • DQM plots for development release, workflow 10824.5
  • DQM plots for development release, workflow 10824.8 are missing
  • DQM plots for development release, workflow 10824.7
  • DQM plots for development release, workflow 10824.9
  • DQM plots for testing release, workflow 10824.5
  • DQM plots for testing release, workflow 10824.8 are missing
  • DQM plots for testing release, workflow 10824.7
  • DQM plots for testing release, workflow 10824.9
  • DQM comparison for reference workflow 10824.5
  • DQM comparison for workflow 10824.8
  • DQM comparison for workflow 10824.7
  • DQM comparison for workflow 10824.9

logs and nvprof/nvvp profiles

/RelValTTbar_13/CMSSW_10_2_0_pre6-PU25ns_102X_upgrade2018_realistic_v7-v1/GEN-SIM-DIGI-RAW

/RelValZMM_13/CMSSW_10_2_0_pre6-102X_upgrade2018_realistic_v7-v1/GEN-SIM-DIGI-RAW

Logs

The full log is available at https://fwyzard.web.cern.ch/fwyzard/patatrack/pulls/8705365de812944ff66ec95e683e9ef5088cddc6/log .

@fwyzard
Copy link
Author

fwyzard commented Jul 24, 2018

While this does not fix the 10824.8 Zmumu workflow, it should fix some minor bugs and do some clean up - and it does not seem to introduce any regressions.

@fwyzard
Copy link
Author

fwyzard commented Jul 24, 2018

@VinInn @makortel any objections ?

@VinInn
Copy link

VinInn commented Jul 24, 2018

ok for me....

@makortel
Copy link

+1

@fwyzard fwyzard merged commit e61eb06 into cms-patatrack:CMSSW_10_2_X_Patatrack Jul 24, 2018
@fwyzard fwyzard deleted the various_fixes_and_cleanup branch July 24, 2018 15:19
fwyzard added a commit that referenced this pull request Oct 8, 2020
Fix errors found by cuda-memcheck:
  - properly initialise device memory
  - fix various cudaMemcpy calls

Remove unused debug variables and function declarations, and #ifdef some debug printouts.

Call cudaDeviceReset() before exiting, via the destructor of CUDAService. This explicitly destroys and cleans up all resources associated with the current device, and is useful to check for memory leaks with cuda-memcheck --tool memcheck --leak-check full.
fwyzard added a commit that referenced this pull request Oct 19, 2020
Fix errors found by cuda-memcheck:
  - properly initialise device memory
  - fix various cudaMemcpy calls

Remove unused debug variables and function declarations, and #ifdef some debug printouts.

Call cudaDeviceReset() before exiting, via the destructor of CUDAService. This explicitly destroys and cleans up all resources associated with the current device, and is useful to check for memory leaks with cuda-memcheck --tool memcheck --leak-check full.
fwyzard added a commit that referenced this pull request Oct 20, 2020
Fix errors found by cuda-memcheck:
  - properly initialise device memory
  - fix various cudaMemcpy calls

Remove unused debug variables and function declarations, and #ifdef some debug printouts.

Call cudaDeviceReset() before exiting, via the destructor of CUDAService. This explicitly destroys and cleans up all resources associated with the current device, and is useful to check for memory leaks with cuda-memcheck --tool memcheck --leak-check full.
fwyzard added a commit that referenced this pull request Oct 23, 2020
Fix errors found by cuda-memcheck:
  - properly initialise device memory
  - fix various cudaMemcpy calls

Remove unused debug variables and function declarations, and #ifdef some debug printouts.

Call cudaDeviceReset() before exiting, via the destructor of CUDAService. This explicitly destroys and cleans up all resources associated with the current device, and is useful to check for memory leaks with cuda-memcheck --tool memcheck --leak-check full.
fwyzard added a commit that referenced this pull request Nov 6, 2020
Fix errors found by cuda-memcheck:
  - properly initialise device memory
  - fix various cudaMemcpy calls

Remove unused debug variables and function declarations, and #ifdef some debug printouts.

Call cudaDeviceReset() before exiting, via the destructor of CUDAService. This explicitly destroys and cleans up all resources associated with the current device, and is useful to check for memory leaks with cuda-memcheck --tool memcheck --leak-check full.
fwyzard added a commit that referenced this pull request Nov 16, 2020
Fix errors found by cuda-memcheck:
  - properly initialise device memory
  - fix various cudaMemcpy calls

Remove unused debug variables and function declarations, and #ifdef some debug printouts.

Call cudaDeviceReset() before exiting, via the destructor of CUDAService. This explicitly destroys and cleans up all resources associated with the current device, and is useful to check for memory leaks with cuda-memcheck --tool memcheck --leak-check full.
fwyzard pushed a commit that referenced this pull request Dec 25, 2020
Fix errors found by cuda-memcheck:
  - properly initialise device memory
  - fix various cudaMemcpy calls

Remove unused debug variables and function declarations, and #ifdef some debug printouts.

Call cudaDeviceReset() before exiting, via the destructor of CUDAService. This explicitly destroys and cleans up all resources associated with the current device, and is useful to check for memory leaks with cuda-memcheck --tool memcheck --leak-check full.
fwyzard added a commit that referenced this pull request Dec 29, 2020
Fix errors found by cuda-memcheck:
  - properly initialise device memory
  - fix various cudaMemcpy calls

Remove unused debug variables and function declarations, and #ifdef some debug printouts.

Call cudaDeviceReset() before exiting, via the destructor of CUDAService. This explicitly destroys and cleans up all resources associated with the current device, and is useful to check for memory leaks with cuda-memcheck --tool memcheck --leak-check full.
fwyzard added a commit that referenced this pull request Dec 29, 2020
Fix errors found by cuda-memcheck:
  - properly initialise device memory
  - fix various cudaMemcpy calls

Remove unused debug variables and function declarations, and #ifdef some debug printouts.

Call cudaDeviceReset() before exiting, via the destructor of CUDAService. This explicitly destroys and cleans up all resources associated with the current device, and is useful to check for memory leaks with cuda-memcheck --tool memcheck --leak-check full.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants