AI/rocm-systems
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Do not use the function part of the location_id

Browse Source 
The function IDs used to be 0 on previous asics but on gfx94x and newer
asics, these bits are set. These bits are used by user applications to
uniquely identify the locations of GPU nodes. These exta bits break
hwloc and are not needed for rocrtst.

Signed-off-by: Mike Li <Tianxinmike.Li@amd.com>
Signed-off-by: David Yat Sin <David.YatSin@amd.com>
Change-Id: I1202f504645b0662d009b9c0926eebb7ddc08d73


[ROCm/ROCR-Runtime commit: d7fa654338]
This commit is contained in:
Mike Li
2022-06-09 12:58:57 -04:00
committed by David Yat Sin
parent dae51188d8
commit e495a4b16a
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
projects/rocr-runtime/rocrtst/suites/performance/memory_async_copy.cc
+4 -1
View File
@@ -657,7 +657,10 @@ static hsa_status_t GetGPUAgents(hsa_agent_t agent, void* data) {
uint8_t bus = (agent_bdf_id & (0xFF << 8)) >> 8;
uint8_t device = (agent_bdf_id & (0x1F << 3)) >> 3;
uint8_t function = (agent_bdf_id & 0x07);
// The function part of the location_id hasn't been used yet
// and may not contain a valid function number.
uint8_t function = 0; //(agent_bdf_id & 0x07);
if (ptr->verbosity() > MemoryAsyncCopy::VERBOSE_STANDARD) {
char name[64];