Regenerate bindings for NVML 12.9.79. Update device tests for new NVML_FI_DEV fields#133
Regenerate bindings for NVML 12.9.79. Update device tests for new NVML_FI_DEV fields#133blthayer wants to merge 1 commit into
Conversation
|
Hey folks, a quick update: I'm not planning to pursue this much further, as instead I need CUDA 13 support (which has some backwards incompatible changes). But I'll leave this open for now in case it's useful. See also: #134 |
| pub const NVML_FI_DEV_NVLINK_COUNT_FEC_HISTORY_13: u32 = 248; | ||
| pub const NVML_FI_DEV_NVLINK_COUNT_FEC_HISTORY_14: u32 = 249; | ||
| pub const NVML_FI_DEV_NVLINK_COUNT_FEC_HISTORY_15: u32 = 250; | ||
| pub const NVML_FI_PWR_SMOOTHING_ENABLED: u32 = 251; |
There was a problem hiding this comment.
Note the current v12 fields in main don't match the latest NVML v12 header...
|
It's a pity that this is a breaking change… |
@TheElectronWill - thanks for weighing in. I agree, this incompatibility should probably be handled via a runtime version check and match statement or something handled by this wrapper. That being said, the CUDA versioning is a little weird/complex and the breaking change occurred between versions of CUDA 13.0 (see #134). As far as I can tell (although I would need to dig deeper), some v13 field numbers slipped into this wrapper's v12 bindings about a year ago? I'm not a maintainer of this repo, just opened a couple draft PRs to help illustrate the differences and give folks branches to use if they want. |
|
FYI, the maintainers are working on a solution: #137 |
Latest v12 bindings (12.9.79). Note this is a breaking change. See also: #134 and #135