Conversation
|
@LegNeato Please hold off on this PR for now — Jimver/cuda-toolkit does not yet support CUDA 13.2.0 (Error: Version not available: 13.2.0), so the Windows CI matrix entry can't run, see CI output on my fork. I'll revisit once the action adds 13.2 support. |
|
We have docker images for cuda 13 in https://github.com/Rust-GPU/rust-cuda/tree/e91b9cefc78258000f92d0978f327c907c515831/container for CI. They currently check 13.0...should we add 13.2? The matrix is starting to get big 🤔 |
…A 13.2 Made-with: Cursor
Made-with: Cursor
ed6fad8 to
db5be61
Compare
Thx for checking. I think you definitely should: this repo aims to be the de facto full port CUDA kit/ecosystem of the Rust world 🦀, so any new official CUDA updates that introduce breaking changes shall be tested (new changes & regression). The Windows build dependency recently adds support of CUDA 13.2. I tested with their latest 0.2.35 release on Windows and it's passing. For now, there're 2 pieces of 13.2 update: 1) this branch
|
Summary
idfield ofCUmemLocation_stwas placed inside an anonymous union. Bindgen exposes this as__bindgen_anon_1: CUmemLocation_st__bindgen_ty_1instead of a directidfield on the struct.2623e21, "Fix CUDA 13 API issues") setiddirectly, which compiled on CUDA 13.0 but fails on 13.2.cuMemLocation_anon_idcfg inbuild.rs(gated ondriver_version >= 13020) and uses field-level#[cfg]at all 5CUmemLocationconstruction sites so both CUDA 13.0 and 13.2 compile correctly.Test plan
cargo build -p custpasses on CUDA 13.2 (Windows, RTX 3060, driver 595.79)cargo build -p custstill passes on CUDA 13.0 (Windows, Linux CI)Attribution
This bugfix was implemented with the help of Claude Sonnet 4.6 and Cursor IDE