-
Notifications
You must be signed in to change notification settings - Fork 44
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
Fixed parser of OpenCL 3.0 for NVIDIA GPU #267
base: v3.5_for_ie_master
Are you sure you want to change the base?
Fixed parser of OpenCL 3.0 for NVIDIA GPU #267
Conversation
@dmitry-gorokhov Could someone to approve CI for this PR ? |
Hi @redradist. Could you please clarify why we need this patch in OpenVINO oneDNN fork? It is used as a backend for OV CPU Plugin and not supposed to be built targeting NVIDIA GPUs. |
@dmitry-gorokhov This patch is needed to be able to compile nvidia_plugin on system with OpenCL 3.0. |
@redradist Does OV nvidia_plugin uses smt from this oneDNN fork? |
It is used in OpenVINO for CPU and GPU plugins, in both places I had to modify this cmake to be able to comiple OpenVINO as dependecy and then NVIDIA Plugin |
Also, one additional comment, reviously this cmake/FindOpenCL.cmake worked properly, but since NVIDIA update they driver with support of OpenCL 3.0, it is broken (I've mentioned it here as well oneapi-src#2001 (comment)) |
@redradist My point is OV CPU plugin set DNNL_GPU_RUNTIME to NONE, so cmake/FindOpenCL.cmake is just not installed in that case. Maybe I miss smt, but couldn't really understand how this fork affected GPU build. Maybe you can share details. |
No description provided.