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

Package request: ZLUDA #288392

Closed
errnoh opened this issue Feb 12, 2024 · 10 comments
Closed

Package request: ZLUDA #288392

errnoh opened this issue Feb 12, 2024 · 10 comments

Comments

@errnoh
Copy link
Contributor

errnoh commented Feb 12, 2024

Project description

ZLUDA lets you run unmodified CUDA applications with AMD GPUs.

There was a recent good writeup of the capabilities at phoronix so I tried compiling it and running some cuda samples. Seemed to work fine.

Dependencies for building:

    rocmPackages.clr
    rocmPackages.miopen
    rocmPackages.rocm-smi
    rocmPackages.rocsparse
    rocmPackages.rocsolver
    rocmPackages.rocblas
    rocmPackages.hipblas
    rocmPackages.rocm-cmake
    rocmPackages.hipfft
    python3
    cargo
    rustc
    cmake
    clang
    zlib
    libxml2
    libedit

Additional build notes:

  • as hipBLASLt is not available on nixpkgs I did comment out zluda_blaslt from Cargo.toml to be able to successfully build.

If no-one is interested I might end up packaging it myself at later date, but as I'm not that active on the ML space it might benefit from more active maintainer 🤷

Metadata


Add a 👍 reaction to issues you find important.

@errnoh
Copy link
Contributor Author

errnoh commented Feb 13, 2024

Created a draft PR with my current progress. For some reason creating a nix-shell with the same buildinputs and running the cargo command manually builds fine. If someone has good ideas check the PR and the comment I added there.

@rastarr
Copy link

rastarr commented Apr 9, 2024

is there any progress on this package request?

@Atemu
Copy link
Member

Atemu commented Apr 10, 2024

Actually, yes. @errnoh created a PR that was linked above: #288644

(Please create a comment in the feature request issue if you've a PR implementing it because else the people watching this don't get pinged!)

@rastarr
Copy link

rastarr commented Apr 10, 2024

Actually, yes. @errnoh created a PR that was linked above: #288644

(Please create a comment in the feature request issue if you've a PR implementing it because else the people watching this don't get pinged!)

OK thank you. I'm a bit lost with when it will get finalised but great it's in progress. Thanks for your quick reply too

@errnoh
Copy link
Contributor Author

errnoh commented Apr 10, 2024

Actually, yes. @errnoh created a PR that was linked above: #288644

(Please create a comment in the feature request issue if you've a PR implementing it because else the people watching this don't get pinged!)

Good point, thought that just having the cross-mention does the trick but didn't consider watchers. Thanks for handling the reply though!

@Atemu
Copy link
Member

Atemu commented Apr 11, 2024

I'm a bit lost with when it will get finalised

It's done when it's done ;)

If you want to help speed up the process, it'd be great if you could test the PR and report your findings.

@rastarr
Copy link

rastarr commented Apr 11, 2024

I'm a bit lost with when it will get finalised

It's done when it's done ;)
If you want to help speed up the process, it'd be great if you could test the PR and report your findings.

I'm happy to help. I have no clue about testing and I only use github for nix config things.
Is there some testing documentation you can point me to?

@Atemu
Copy link
Member

Atemu commented Apr 11, 2024

Just https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#how-to-review-pull-requests but the gist is to build the package locally and then test that build.

@rastarr
Copy link

rastarr commented Apr 11, 2024

Just https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#how-to-review-pull-requests but the gist is to build the package locally and then test that build.

OK thanks for that. I read through it :)

@errnoh
Copy link
Contributor Author

errnoh commented Apr 14, 2024

ZLUDA v3 has now been merged. Worth mentioning that zluda_blaslt is not supported on this release and will be added later once hipblaslt package is available in nixpkgs. Everything else supported by ZLUDA should work fine though.

@errnoh errnoh closed this as completed Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

4 participants