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

Making papi.spec better support RPM builds with cuda, rocm, and rocm_smi components #215

Open
wcohen opened this issue Aug 5, 2024 · 2 comments

Comments

@wcohen
Copy link
Contributor

wcohen commented Aug 5, 2024

Would like to make it easier for rpms to be built with cuda, rocm, and rocm_smi components enabled. The current WIP SRPM can be found as a fedora rawhide scratch build at https://koji.fedoraproject.org/koji/taskinfo?taskID=121528450. Because cuda and rocm support are not in the koji build system, the build does not have the cuda, rocm, rocm_smi support enabled by default. However, assuming the amd rocm and nvidia cuda are installed one can download the srpm and build it with the cuda, rocm, and rocm_smi support on RHEL9 with:

rpm -Uvh papi-7.1.0-5.fc39cuda.src.rpm
cd ~/rpmbuild/SPECS/
rpmbuild -with cuda -with rocm -with rocm_smi -ba papi.spec

The https://fedoraproject.org/wiki/SIGs/HC describes progress in getting the rocm support into Fedora Rawhide. There are some changes needed to get the srpm to build the rocm_smi component on Fedora Rawhide as the rpms names are different and PAPI_ROCMSMI_ROOT will need to be changed from the default.

@adanalis
Copy link
Contributor

Will, I'm not sure what we need to do to accommodate this effort. What needs to be changed wrt PAPI_ROCMSMI_ROOT?

@wcohen
Copy link
Contributor Author

wcohen commented Aug 13, 2024

For this issue the changes would be in papi.spec to have the correct dependencies and set PAPI_*_ROOT appropriately. Ideally, it would be good if someone could use the spec file in the tarball and build papi with those additional components. The comment about PAPI_ROCMSMI_ROOT is just mentioning that locations for rocm includes etc are going to change in the future at the papi.spec will need to take that into account.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants