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

Implement a first draft of the ROCmServices #40642

Merged
merged 1 commit into from Feb 1, 2023

Conversation

fwyzard
Copy link
Contributor

@fwyzard fwyzard commented Jan 30, 2023

PR description:

Implement a first draft of the ROCmService and ROCmMonitoringService.

While the ROCmService is not on par with the CUDAService, it is already usable, especially for the ROCm library tests and as the basis to implement more ROCm utilities.

The commented out code is left as a reminder of the CUDAService features that are still missing.

The PR builds on top of and includes:

PR validation:

The new unit tests pass.

@cmsbuild
Copy link
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40642/33945

ERROR: Build errors found during clang-tidy run.

HeterogeneousCore/ROCmServices/src/ROCmService.cc:20:10: error: 'HeterogeneousCore/ROCmUtilities/interface/hipCheck.h' file not found [clang-diagnostic-error]
#include "HeterogeneousCore/ROCmUtilities/interface/hipCheck.h"
         ^
Suppressed 517 warnings (516 in non-user code, 1 with check filters).
--
HeterogeneousCore/ROCmServices/plugins/ROCmMonitoringService.cc:15:10: error: 'HeterogeneousCore/ROCmUtilities/interface/hipCheck.h' file not found [clang-diagnostic-error]
#include "HeterogeneousCore/ROCmUtilities/interface/hipCheck.h"
         ^
Suppressed 548 warnings (547 in non-user code, 1 with check filters).
--
gmake: *** [config/SCRAM/GMake/Makefile.coderules:129: code-checks] Error 2
gmake: *** [There are compilation/build errors. Please see the detail log above.] Error 2

@fwyzard
Copy link
Contributor Author

fwyzard commented Jan 30, 2023

This PR requires #40619 .

@fwyzard
Copy link
Contributor Author

fwyzard commented Jan 31, 2023

please test with cms-sw/cmsdist#8273,#40619

@fwyzard
Copy link
Contributor Author

fwyzard commented Jan 31, 2023

please test

@fwyzard
Copy link
Contributor Author

fwyzard commented Jan 31, 2023

+heterogeneous

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 1, 2023

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40642/33987

ERROR: Build errors found during clang-tidy run.

HeterogeneousCore/ROCmServices/src/ROCmService.cc:20:10: error: 'HeterogeneousCore/ROCmUtilities/interface/hipCheck.h' file not found [clang-diagnostic-error]
#include "HeterogeneousCore/ROCmUtilities/interface/hipCheck.h"
         ^
Suppressed 518 warnings (517 in non-user code, 1 with check filters).
--
HeterogeneousCore/ROCmServices/plugins/ROCmMonitoringService.cc:15:10: error: 'HeterogeneousCore/ROCmUtilities/interface/hipCheck.h' file not found [clang-diagnostic-error]
#include "HeterogeneousCore/ROCmUtilities/interface/hipCheck.h"
         ^
Suppressed 549 warnings (548 in non-user code, 1 with check filters).
--
gmake: *** [config/SCRAM/GMake/Makefile.coderules:129: code-checks] Error 2
gmake: *** [There are compilation/build errors. Please see the detail log above.] Error 2

@fwyzard
Copy link
Contributor Author

fwyzard commented Feb 1, 2023

Ah... so the code checks use the latest IB as-is, without the PRs that have already been merged :-(

@fwyzard
Copy link
Contributor Author

fwyzard commented Feb 1, 2023

please test

@fwyzard
Copy link
Contributor Author

fwyzard commented Feb 1, 2023

+heterogeneous

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 1, 2023

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40642/33991

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 1, 2023

Pull request #40642 was updated. can you please check and sign again.

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 1, 2023

-1

Failed Tests: RelVals-INPUT
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-f8bc51/30326/summary.html
COMMIT: 4f5ff35
CMSSW: CMSSW_13_0_X_2023-01-31-2300/el8_amd64_gcc11
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/40642/30326/install.sh to create a dev area with all the needed externals and cmssw changes.

RelVals-INPUT

The relvals timed out after 4 hours.

Comparison Summary

Summary:

  • You potentially added 8 lines to the logs
  • Reco comparison results: 7 differences found in the comparisons
  • DQMHistoTests: Total files compared: 49
  • DQMHistoTests: Total histograms compared: 3555495
  • DQMHistoTests: Total failures: 6
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 3555467
  • DQMHistoTests: Total skipped: 22
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 48 files compared)
  • Checked 211 log files, 162 edm output root files, 49 DQM output files
  • TriggerResults: no differences found

@perrotta
Copy link
Contributor

perrotta commented Feb 1, 2023

+1

@perrotta
Copy link
Contributor

perrotta commented Feb 1, 2023

merge

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