Skip to content

[sycl-rel] Cherry-pick patches for sycl_ext_intel_device_info #18886

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

Conversation

KornevNikita
Copy link
Contributor

Cherry-pick commits that reached the internal branch between intel/llvm cutoff and release branch pulldown.

@KornevNikita KornevNikita requested review from a team as code owners June 10, 2025 11:56
@KornevNikita KornevNikita requested a review from npmiller June 10, 2025 11:56
@npmiller
Copy link
Contributor

This will likely be needed as well:

againull added 3 commits June 12, 2025 12:25
Tests were not added because there are existing conformance tests which
cover this functionality.
…ntel#17719)

Our pre-commit CI uses CUDA 12.6 but nightly uses CUDA 12.1, it turns
out nvml which is part of CUDA 12.6 has
nvmlDeviceGetCurrentClocksEventReasons API, but nvml which is part of
CUDA 12.1 doesn't have it, but only supports older deprecated
nvmlDeviceGetCurrentClocksThrottleReasons.

NVML doesn't provide a version macro to check the support for that API,
so use new API nvmlDeviceGetCurrentClocksEventReasons based on cuda
version.
@KornevNikita KornevNikita force-pushed the cherry-pick-sycl_ext_intel_device_info branch from 8596557 to 3ffe48b Compare June 12, 2025 10:31
@KornevNikita
Copy link
Contributor Author

There was an already cherry-picked commit, so I had to force push.

@KornevNikita
Copy link
Contributor Author

KornevNikita commented Jun 12, 2025

Ignore the pre-commit results, it's stale. Actual test run - https://github.com/intel/llvm/actions/runs/15608181761

@intel/llvm-reviewers-cuda @intel/unified-runtime-reviewers take a look please.

@KornevNikita
Copy link
Contributor Author

@intel/unified-runtime-reviewers friendly ping

@KornevNikita
Copy link
Contributor Author

@kbenzie could you please take a look?

@AlexeySachkov AlexeySachkov merged commit 65dc2df into intel:sycl-rel-6_2 Jun 18, 2025
21 of 31 checks passed
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.

6 participants