Commit Graph

68454 Commits

Author SHA1 Message Date
cfallows-amd 2e750763e7 Fix file naming for roofline PDF outputs when using --kernel (#803)
Move check for kernel filtering to add to roofline pdfs- was originally only labelling pdfs with the filtered kernel names from --kernel when --kernel-names was called, we want it at all times when kernels are filtered. 

Signed-off-by: Carrie Fallows <Carrie.Fallows@amd.com>
2025-09-06 02:34:51 -04:00
Galantsev, Dmitrii 0cd05bf307 Create run-clang-tidy.sh
Change-Id: I4faa950a59434ba4706da581af51dd8a7e071dcb
Signed-off-by: Galantsev, Dmitrii <dmitrii.galantsev@amd.com>
2025-09-05 17:44:17 -05:00
Galantsev, Dmitrii d0b5e20440 Create run-clang-tidy.sh
Change-Id: I4faa950a59434ba4706da581af51dd8a7e071dcb
Signed-off-by: Galantsev, Dmitrii <dmitrii.galantsev@amd.com>


[ROCm/amdsmi commit: 0cd05bf307]
2025-09-05 17:44:17 -05:00
Galantsev, Dmitrii 85e37bb6ce Add extra element to array for bounds checking
Decrement padding to keep struct size the same

Change-Id: I4bea5d4b4d5c908423c7cc55a7e8c404b4a6b5e8
Signed-off-by: Galantsev, Dmitrii <dmitrii.galantsev@amd.com>
2025-09-05 17:44:17 -05:00
Galantsev, Dmitrii 7bbfc98588 Add extra element to array for bounds checking
Decrement padding to keep struct size the same

Change-Id: I4bea5d4b4d5c908423c7cc55a7e8c404b4a6b5e8
Signed-off-by: Galantsev, Dmitrii <dmitrii.galantsev@amd.com>


[ROCm/amdsmi commit: 85e37bb6ce]
2025-09-05 17:44:17 -05:00
Galantsev, Dmitrii 20bc3aeeef Ignore more warnings in clang and clang-tidy
Change-Id: I6f7c7e478f0f176da550d5bccf833dae1a4f1878
Signed-off-by: Galantsev, Dmitrii <dmitrii.galantsev@amd.com>
2025-09-05 17:44:17 -05:00
Galantsev, Dmitrii 6797de3ed5 Ignore more warnings in clang and clang-tidy
Change-Id: I6f7c7e478f0f176da550d5bccf833dae1a4f1878
Signed-off-by: Galantsev, Dmitrii <dmitrii.galantsev@amd.com>


[ROCm/amdsmi commit: 20bc3aeeef]
2025-09-05 17:44:17 -05:00
Galantsev, Dmitrii aba1c792b4 Clean up clang-tidy warnings and unused variables
Change-Id: I1365edf8926908b3a49652fb87f079f8fbf1f56b
2025-09-05 17:44:17 -05:00
Galantsev, Dmitrii 74efdc57a7 Clean up clang-tidy warnings and unused variables
Change-Id: I1365edf8926908b3a49652fb87f079f8fbf1f56b


[ROCm/amdsmi commit: aba1c792b4]
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 66eb189396 Remove an impossible check
amdsmi/tests/amd_smi_test/functional/memorypartition_read_write.cc:453:32: warning: the address of ‘orig_memory_partition’ will never be NULL [-Waddress]
  453 |     if ((orig_memory_partition == nullptr) ||
      |          ~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~

Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 3a7b4a283a Remove an impossible check
amdsmi/tests/amd_smi_test/functional/memorypartition_read_write.cc:453:32: warning: the address of ‘orig_memory_partition’ will never be NULL [-Waddress]
  453 |     if ((orig_memory_partition == nullptr) ||
      |          ~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~

Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>


[ROCm/amdsmi commit: 66eb189396]
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) eacec681dd Use nested namespace for amd::smi
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 0c7c849c42 Use nested namespace for amd::smi
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>


[ROCm/amdsmi commit: eacec681dd]
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 4a863b27ab Drop an unnecessary NULL comparison
warning: the address of ‘amdsmi_asic_info_t::vendor_name’ will never be NULL [-Waddress]

Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 47b2e80ab8 Drop an unnecessary NULL comparison
warning: the address of ‘amdsmi_asic_info_t::vendor_name’ will never be NULL [-Waddress]

Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>


[ROCm/amdsmi commit: 4a863b27ab]
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) a15bad1c9e Fix a comparison between signed and unsigned integer
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 02b357526b Fix a comparison between signed and unsigned integer
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>


[ROCm/amdsmi commit: a15bad1c9e]
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) a99e827d97 Drop unused variables
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 08eec3c675 Drop unused variables
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>


[ROCm/amdsmi commit: a99e827d97]
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 3d0ea25af3 Remove unnecessary typedef declarations
amd_smi_cper.h:32:1: warning: ‘typedef’ was ignored in this declaration

Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) a8a89db945 Remove unnecessary typedef declarations
amd_smi_cper.h:32:1: warning: ‘typedef’ was ignored in this declaration

Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>


[ROCm/amdsmi commit: 3d0ea25af3]
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 924a06d1e1 Remove unnecessary includes
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) c9eddf75e7 Remove unnecessary includes
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>


[ROCm/amdsmi commit: 924a06d1e1]
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 05f79879c3 Fix a typo
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 5fe413710b Fix a typo
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>


[ROCm/amdsmi commit: 05f79879c3]
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) faca0222f0 Use nested namespace for amd::smi
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>
2025-09-05 17:44:17 -05:00
Mario Limonciello (AMD) 15e335ac3f Use nested namespace for amd::smi
Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>


[ROCm/amdsmi commit: faca0222f0]
2025-09-05 17:44:17 -05:00
Todd tiantuo Li c8ecf77a94 Update dispatch table to move 7.1 new APIs under HIP_RUNTIME_API_TABLE_STEP_VERSION 14 (#790) 2025-09-05 14:14:43 -07:00
Julia Jiang 2889114b4a SWDEV-553031 - Adding Copyright in files for HIP (#827) 2025-09-05 13:48:43 -07:00
Julia Jiang d130c986fc SWDEV-553028 - Adding Coptright in files for HIP-tests (#829) 2025-09-05 16:44:28 -04:00
David Galiffi 6aca76a59f Disable CMAKE_INSTALL_RPATH_USE_LINK_PATH (#864)
Fixing some `check-rpath` errors when generating RPM packages in RHEL 10

```
ERROR 0002: file '/home/gliff/opt/rocprofiler-systems/bin/rocprof-sys-avail' contains an invalid rpath '/opt/rocm-7.0.0/lib' in [$ORIGIN/../lib:$ORIGIN/../lib/rocprofiler-systems:/opt/rocm-7.0.0/lib]
ERROR 0002: file '/home/gliff/opt/rocprofiler-systems/bin/rocprof-sys-causal' contains an invalid rpath '/opt/rocm-7.0.0/lib' in [$ORIGIN/../lib:$ORIGIN/../lib/rocprofiler-systems:/opt/rocm-7.0.0/lib]
ERROR 0002: file '/home/gliff/opt/rocprofiler-systems/bin/rocprof-sys-run' contains an invalid rpath '/opt/rocm-7.0.0/lib' in [$ORIGIN/../lib:$ORIGIN/../lib/rocprofiler-systems:/opt/rocm-7.0.0/lib]
ERROR 0002: file '/home/gliff/opt/rocprofiler-systems/lib/librocprof-sys.so.1.2.0' contains an invalid rpath '/opt/rocm-7.0.0/lib' in [$ORIGIN:$ORIGIN/rocprofiler-systems:/opt/rocm-7.0.0/lib]
```

Signed-off-by: David Galiffi <David.Galiffi@amd.com>
2025-09-05 14:27:31 -04:00
jamessiddeley-amd fa860c5272 [rocprof-compute] Fixed CDash compute code coverage upload (#863)
* Test CDash upload fixes

* Test CDash upload fixes + rename

* fix run-ci.py script

* fix run-ci.py script

* fix run-ci.py script

* fix run-ci.py script

* fixed workflow

* renamed coverage file

* fixed run-ci.py script

* edited workflow file

* edited script desc

* Update Coverage.xml
2025-09-05 11:55:08 -04:00
Joseph Macaranas 5ca7af2d30 Migrate amdgpu-windows-interop to rocm-systems (#808) 2025-09-05 10:32:44 -04:00
Jimbo 3d9d35a1f8 SWDEV-553375 - Allow hipMemAllocationTypeUncached in hipMemGetAllocationGranularity (#847) 2025-09-05 10:31:20 -04:00
Sam Ruscica b2b44de1e7 Removed old perftest directory (#840) 2025-09-04 22:17:31 -04:00
Ajay GunaShekar 2202dcfe80 SWDEV-552613 - Windows: Use Direct Dispatch only on HSA ROCr Backend (#809)
* SWDEV-552613 - Disable Direct Dispatch on Windows

* SWDEV-552613 -  Use Direct Dispatch on HSA backend only

---------

Co-authored-by: GunaShekar <agunashe@amd.com>
Co-authored-by: Christophe Paquot <35546540+chrispaquot@users.noreply.github.com>
2025-09-04 14:07:12 -07:00
habajpai-amd fb6fe518e8 fix(transpose): correct host allocation and GB/s calculation (#860) 2025-09-04 16:08:16 -04:00
harkgill-amd 782dc9214b Fix: Error messages printed to stderr to trigger CMake Error Variable (#743)
This PR intends to cover the edge case seen in https://github.com/ROCm/rocm-systems/issues/694. 

`hip-config-amd.cmake` uses rocm_agent_enumerator to determine which GPU architecture to target when no target is specified.
https://github.com/ROCm/rocm-systems/blob/9a02dae75f8df9d8f08923d34d06d76e96ced7b4/projects/clr/hipamd/hip-config-amd.cmake.in#L86-L95

On WSL, both `readFromKFD` and `readFromLSPCI` are skipped. If `readFromTargetLstFile()` isn't in use, `readFromROCMINFO()` is called on. If rocminfo times out, it prints the following message to stdout.
```
"Timeout querying rocminfo.  Are you compiling with more than 254 threads?"
```
Because this is output and not an explicit error message, `execute_command` in the previous code blocks treats the output as `OUTPUT_VARIABLE` and passes it on as a valid gfx arch which causes these errors in CMake,
```
lang++: error: invalid target ID 'Timeout'; format is a processor name followed by an optional colon-delimited list of features followed by an enable/disable sign (e.g., 'gfx908:sramecc+:xnack-')
    clang++: error: invalid target ID 'querying'; format is a processor name followed by an optional colon-delimited list of features followed by an enable/disable sign (e.g., 'gfx908:sramecc+:xnack-')
    clang++: error: invalid target ID 'rocminfo.'; format is a processor name followed by an optional colon-delimited list of features followed by an enable/disable sign (e.g., 'gfx908:sramecc+:xnack-')
    clang++: error: invalid target ID 'Are'; format is a processor name followed by an optional colon-delimited list of features followed by an enable/disable sign (e.g., 'gfx908:sramecc+:xnack-')
    clang++: error: invalid target ID 'you'; format is a processor name followed by an optional colon-delimited list of features followed by an enable/disable sign (e.g., 'gfx908:sramecc+:xnack-')
    clang++: error: invalid target ID 'compiling'; format is a processor name followed by an optional colon-delimited list of features followed by an enable/disable sign (e.g., 'gfx908:sramecc+:xnack-')
```
The output can be properly pushed to `ERROR_VARIABLE` if rocm_agent_enumerator pushes the output to stderr instead of stdout. This can be done with the changes to the print statement in this PR or using the `logging` module.
2025-09-04 15:12:41 -04:00
vedithal-amd 77ed80f457 Fix Performance (GFLOPs) metric (#843) 2025-09-04 14:30:22 -04:00
jamessiddeley-amd 5a85df8f31 [rocprof-compute] update coverage to 81.1% (#826)
* update coverage to 81.1%

* Update coverage to latest develop
2025-09-04 14:21:03 -04:00
SaleelK c4537e8050 SWDEV-553126 - Improve logging (#835)
* Ability to mask COPY api usage in logs
* Show total graph nodes in logs
* Add another log level for detailed debug
2025-09-04 10:08:41 -07:00
marandje 79bda80049 SWDEV-549686 - Resolve memory leaks in texture unit-tests (#711) 2025-09-04 17:59:18 +02:00
vstojilj 12ad8421bb SWDEV-549700 - Add missing destroy calls (#755) 2025-09-04 17:21:32 +02:00
jamessiddeley-amd 04c6cfec8c [rocprof-compute] updated test_utils mock to be compatible with python3.11 (#839)
* updated test_utils mock to work with python3.11

* fixed python formatting
2025-09-04 10:44:04 -04:00
German Andryeyev 7a1a6682e2 SWDEV-552846 - Unpin memory for hip before exit the copy (#851) 2025-09-04 20:04:01 +05:30
Ioannis Assiouras 7bf7110ae8 SWDEV-550667 - Correct the check for availability of __hip_atomic_fetch_add (#818) 2025-09-04 15:15:34 +01:00
RahulC 9b4c12a357 Revert "[rocprofiler-sdk][SDK] Update to address new API changes for HIP ROCm…" (#850)
This reverts commit 5ac738150a.
2025-09-03 21:52:43 -07:00
systems-assistant[bot] 9f11d73561 SWDEV-541096 - add hipEventWaitDefault and hipEventWaitExternal (#460)
Co-authored-by: Li, Todd tiantuo <Toddtiantuo.Li@amd.com>
2025-09-03 20:19:06 -07:00
estewart08 bc35beafbf rocr: Remove extra LibElf find_package (#767)
This should have been removed when the libelf config search
was added.
2025-09-03 20:04:05 -04:00
David Galiffi f80a2f456c Clean up jammy workflow (#788)
* Clean up jammy workflows

---------

Signed-off-by: David Galiffi <David.Galiffi@amd.com>
2025-09-03 16:25:15 -04:00