Skip to content

sync nvml.h with CUDA version 13.1.2#182

Open
tariq1890 wants to merge 1 commit into
mainfrom
upd-nvml-h-cuda-13.1
Open

sync nvml.h with CUDA version 13.1.2#182
tariq1890 wants to merge 1 commit into
mainfrom
upd-nvml-h-cuda-13.1

Conversation

@tariq1890
Copy link
Copy Markdown
Contributor

This PR synchronises the nvml.h file with that of the CUDA 13.1 toolkit.

The nvml header file can be found here

Comment thread pkg/nvml/device.go Outdated
return device.ReadPRMCounters_v1(counterList)
}

func (device nvmlDevice) ReadPRMCounters_v1(counterList *PRMCounterList_v1) Return {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@klueska Acknowledged that I need to rework the input params of this method. Before I do that, I think we need to convert the *PRMCounterList_v1 pointer type in C to a go slice. Does c-for-go support this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

never mind. I think I got the idea of the *PRMCounterList struct now. Please let me know what you think of the latest iteration of ReadPRMCounters_v1

Comment thread pkg/nvml/device.go
return device.SetRusdSettings_v1(settings)
}
func (device nvmlDevice) SetRusdSettings_v1(settings RusdSettings_v1) Return {
settings.Version = STRUCT_VERSION(settings, 1)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

setting the STRUCT_VERSION as requested in the earlier PR

@tariq1890 tariq1890 force-pushed the upd-nvml-h-cuda-13.1 branch from c553002 to 4029ba3 Compare May 14, 2026 01:20
Signed-off-by: Tariq Ibrahim <tibrahim@nvidia.com>
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

Successfully merging this pull request may close these issues.

1 participant