- Create a local copy for ROCm/rccl-tests for our examples.
- Update argument parsing to no longer use getopt_long.
- Workaround for Dyninst instrumentation.
---------
Co-authored-by: David Galiffi <David.Galiffi@amd.com>
* Add custom rocprofiler-sdk counter definitions file for MI 100
* Update CHANGELOG to mention that accumulation counters will not be
collected when profiling on MI 100 using rocprofiler-sdk/rocprofv3
* Migrate accum_counters.yaml to code
Change when Roofline PDFs are generated- during general profiling and --roof-only profiling (skip only when --no-roof option is present)
---------
Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
* Replace `cmake-format` with `gersemi`
Signed-off-by: David Galiffi <David.Galiffi@amd.com>
* Remove .cmake-format.yaml
Signed-off-by: David Galiffi <David.Galiffi@amd.com>
* Update workflow to use gersemi
Signed-off-by: David Galiffi <David.Galiffi@amd.com>
* Update CONTRIBUTING.md
* Update helper scripts
* Don't include `*/external/*` in workflows
---------
Signed-off-by: David Galiffi <David.Galiffi@amd.com>
* Updating clang-format to v18
- Updates the pre-commit-config
- Formats source files according to the utility
Signed-off-by: David Galiffi <David.Galiffi@amd.com>
* Update format source workflow
Signed-off-by: David Galiffi <David.Galiffi@amd.com>
* Update CONTRIBUTING
* Update comment in .clang-format
* Update CONTRIBUTING.md
* Update helper script
---------
Signed-off-by: David Galiffi <David.Galiffi@amd.com>
* add paths-ignore for both push and PR to rhel-8.yml and ubuntu-jammy.yml
* add paths-ignore for PR only in tarball.yml
* add rocm-docs-core linting.yml workflow to formatting.yml
* fix small mistake in github directory path
* remove rocm-docs-core linting
On AMD-SMI, in rocm 7.0, vcn_activity and jpeg_activity will not be reported when XCP (partition) stats, vcn_busy and jpeg_busy, are available. This causes the activity tracking to fail. The fix is to read the busy values when activity values are not supported.
For issue: SWDEV-536439
---------
Co-authored-by: David Galiffi <David.Galiffi@amd.com>
* added progress printout for profiler
* added comments and fixed readability
* removed redundant newlines
* moved format_time helper function to utils
* removed tqdm and redundant time calc
* enable roofline plot in cli.
* Add roofline to analysis config.
* Unify global variables.
* Disable roofline for baseline comparison and gfx908.
* Add check for roofline.csv
- Refactor elfutils build folders. Move build artifacts to the "external" subfolders
- Remove reference to unused DYNINST configuration
- Declare Dyninst interfaces with other external declarations
- Updated cmake warning message
- Update the deprecation warning for "DYNINST_BUILD_${dep}" to make it clearer that the option is being translated to the new option, "ROCPROFSYS_BUILD_${dep}"
* Fix: Add explicit includes for assert.h and thread in sample/vcopy.cpp
The HIP/CLR change e3cb5399c removed transitive inclusion
of standard headers like assert.h from hip_runtime.h.
This caused build failures in rocprof-compute samples.
This commit explicitly includes <assert.h> and <thread>
in vcopy.cpp to resolve potential missing definitions.
Signed-off-by: jamessiddeley-amd <James.Siddeley@amd.com>
* Update vsequential_access.cpp
Added assert and thread imports
Signed-off-by: jamessiddeley-amd <James.Siddeley@amd.com>
* Update vrandom_access.cpp
Added assert import
Signed-off-by: jamessiddeley-amd <James.Siddeley@amd.com>
* Update vsequential_access.cpp
Signed-off-by: jamessiddeley-amd <James.Siddeley@amd.com>
---------
Signed-off-by: jamessiddeley-amd <James.Siddeley@amd.com>
* Revert of https://github.com/ROCm/rocprofiler-compute/pull/738
* Change default rocprof backend interface to rocprofv3
* Add MI 350 support in documentation
* Added known issue that MI 100 profiling will not work unless rocprofv1
is explicitly opted in
* Remove MI 50 soc gfx python class since MI 50 is not supported
Test is missing from rocm-7.0 stack because of a HIP version check.
In these builds, hip_version.h is still reporting 6.5.0.
This check was originally put in to skip the test on older versions
of ROCm, which should no longer be required
- For SWDEV-537718
Signed-off-by: David Galiffi <David.Galiffi@amd.com>
Pranjal Swarup
Fei Zheng
vedithal-amd
anujshuk-amd
David Galiffi
dependabot[bot]
Kian Cossettini
Kunal Malviya
cfallows-amd
xuchen-amd
Jason Bonnell
Sajina PK
ajanicijamd
jamessiddeley-amd
Pratik Basyal
Daniel Su
Vladimir Indic
ywang103-amd