search

ROCm / rocm_bandwidth_test

Bandwidth test for ROCm
MIT License
51 stars 28 forks source link

SWDEV-345870 - Correct include paths for new directory layout #85

Closed raramakr closed 2 years ago

raramakr commented 2 years ago

With file reorganization, the header files are moved to /opt/rocm-xyz/include/comp_name/ The hsa header files referred are wrapper files and will generate a warning on compilation. Changes made to refer actual header files

raramakr commented 2 years ago

How are you building the executable. I see changes for header file input directory and how can this build without making cmake changes. Could I get clarification on this?

In docker(ubuntu or centos), the script build_rocm_bandwidth_test.sh is used for creating the package. The hsa file reorganization cmake changes are already in mainline. All header files for rocm will be in /opt/rocm-ver/include/comp-name/ . For backward compatibility , the older path /opt/rocm-ver/hsa/include/hsa is still there , but will be removed in future.