Repository navigation
[SYCL] Fix get mem error - #28227
[SYCL] Fix get mem error#28227
Conversation
Co-authored-by: Titaniumtown <titaniumtown@proton.me>
|
This does not compile for me. The base b10743 does compile. Compiler log |
The root cause is you didn't install the level-zero dev package. Two methods: Disable using level-zero by Thank you! |
|
Thanks, I ran |
I hit the same abort as #28134 on an Arc 140T, so I tested this PR merged onto master 1. The PR does not compile when
|
|
@manderss99 I will update the PR as soon. Thank you! |
|
@manderss99 Thank you! |
|
Retested at Build
With the development headers missing, configure now prints the new message and continues instead of failing: (Reproduced by forcing Runtime — all four combinations fine, no
|
|
@manderss99 They are fixed:
Thank you! |
|
when will it merge ? |
|
@ahmetem If there are more feedback to verify this PR, the merge will be speeded up. Thank you! |
|
@Titaniumtown @ggerganov Thank you! |
* fix for unsupport zes API * optimize the code * adjust the log level * rm unused head files * Update docs/backend/SYCL.md Co-authored-by: Titaniumtown <titaniumtown@proton.me> * fix the error to detect level zero SDK/dev package, stop build after detect the error * update the message * fix the build error when missed to install level zero dev package * rm GGML_SYCL_DEV_DEBUG, mv read env vars in all entry functions --------- Co-authored-by: Neo Zhang Jianyu <jianyu.zhang@intel.com> Co-authored-by: Titaniumtown <titaniumtown@proton.me> Co-authored-by: Neo Zhang <NA>
* fix for unsupport zes API * optimize the code * adjust the log level * rm unused head files * Update docs/backend/SYCL.md Co-authored-by: Titaniumtown <titaniumtown@proton.me> * fix the error to detect level zero SDK/dev package, stop build after detect the error * update the message * fix the build error when missed to install level zero dev package * rm GGML_SYCL_DEV_DEBUG, mv read env vars in all entry functions --------- Co-authored-by: Neo Zhang Jianyu <jianyu.zhang@intel.com> Co-authored-by: Titaniumtown <titaniumtown@proton.me> Co-authored-by: Neo Zhang <NA>
* fix for unsupport zes API * optimize the code * adjust the log level * rm unused head files * Update docs/backend/SYCL.md Co-authored-by: Titaniumtown <titaniumtown@proton.me> * fix the error to detect level zero SDK/dev package, stop build after detect the error * update the message * fix the build error when missed to install level zero dev package * rm GGML_SYCL_DEV_DEBUG, mv read env vars in all entry functions --------- Co-authored-by: Neo Zhang Jianyu <jianyu.zhang@intel.com> Co-authored-by: Titaniumtown <titaniumtown@proton.me> Co-authored-by: Neo Zhang <NA>
* fix for unsupport zes API * optimize the code * adjust the log level * rm unused head files * Update docs/backend/SYCL.md Co-authored-by: Titaniumtown <titaniumtown@proton.me> * fix the error to detect level zero SDK/dev package, stop build after detect the error * update the message * fix the build error when missed to install level zero dev package * rm GGML_SYCL_DEV_DEBUG, mv read env vars in all entry functions --------- Co-authored-by: Neo Zhang Jianyu <jianyu.zhang@intel.com> Co-authored-by: Titaniumtown <titaniumtown@proton.me> Co-authored-by: Neo Zhang <NA>
Overview
Fixed the issue #28134 involved in #27968.
Additional information
The PR [SYCL] Enhance to get the free memory of Intel GPU #27968 use level zero API to get free memory size to support more GPUs. But it can't work on WSL on windows, because level zero Sysman doesn't support it.
The solution in this PR fallback to SYCL API when it's not supported.
Add GGML_SYCL_DEV_DEBUG() for development debug.
Verified on Linux (Arc770,UHD Graphics 770 in 13700K), Windows (140T) and WSL (Ubuntu) on Windows.
Requirements