-
Notifications
You must be signed in to change notification settings - Fork 732
[XLA:GPU] [plugin] Adding sycl/oneapi plugin support for Intel GPU #35304
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
base: main
Are you sure you want to change the base?
[XLA:GPU] [plugin] Adding sycl/oneapi plugin support for Intel GPU #35304
Conversation
beckerhe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good to me. Thank you!
| xla::PjRtPlatformId platform_id; | ||
| std::string platform_name; | ||
|
|
||
| std::string plugin_platform = std::string(PJRT_GPU_PLUGIN_PLATFORM_NAME); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you make this an absl::string_view?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
xla/pjrt/c/pjrt_c_api_gpu_test.cc
Outdated
| #ifdef TENSORFLOW_USE_ROCM | ||
| if (allocator_option == "cuda_async") { | ||
| VLOG(1) << "cuda_async allocator not available on ROCm!"; | ||
| VLOG(1) << "cuda_async allocator not available on ROCm! and SYCL!"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: There is an exclamation mark too many.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
…ntel GPU Imported from GitHub PR #35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a2 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd PiperOrigin-RevId: 853258690
…ntel GPU Imported from GitHub PR openxla/xla#35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a21e9e59c38e961ccd68b036a2c52403728 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1abcd5b84fc503fdc6a2502adeebf813d4 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d PiperOrigin-RevId: 853258690
…ntel GPU Imported from GitHub PR #35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a2 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd PiperOrigin-RevId: 853258690
…ntel GPU Imported from GitHub PR openxla/xla#35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a21e9e59c38e961ccd68b036a2c52403728 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1abcd5b84fc503fdc6a2502adeebf813d4 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d PiperOrigin-RevId: 853258690
…ntel GPU Imported from GitHub PR #35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a2 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd PiperOrigin-RevId: 853258690
…ntel GPU Imported from GitHub PR openxla/xla#35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a21e9e59c38e961ccd68b036a2c52403728 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1abcd5b84fc503fdc6a2502adeebf813d4 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d PiperOrigin-RevId: 853258690
…ntel GPU Imported from GitHub PR #35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a2 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd PiperOrigin-RevId: 853258690
…ntel GPU Imported from GitHub PR openxla/xla#35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a21e9e59c38e961ccd68b036a2c52403728 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1abcd5b84fc503fdc6a2502adeebf813d4 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 Reverts 989c58d FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d PiperOrigin-RevId: 853258690
…ntel GPU Imported from GitHub PR #35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a2 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd PiperOrigin-RevId: 853719463
…ntel GPU Imported from GitHub PR openxla/xla#35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a21e9e59c38e961ccd68b036a2c52403728 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1abcd5b84fc503fdc6a2502adeebf813d4 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d PiperOrigin-RevId: 853719463
|
@beckerhe , added a commit that is to fix the JAX failures and also that the bazel query failure is address by the tools PR that has been merged google-ml-infra/rules_ml_toolchain#168 |
…ntel GPU Imported from GitHub PR #35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a2 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd PiperOrigin-RevId: 853719463
…ntel GPU Imported from GitHub PR openxla/xla#35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a21e9e59c38e961ccd68b036a2c52403728 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1abcd5b84fc503fdc6a2502adeebf813d4 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d PiperOrigin-RevId: 853719463
…ntel GPU Imported from GitHub PR #35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a2 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd PiperOrigin-RevId: 853719463
…ntel GPU Imported from GitHub PR openxla/xla#35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a21e9e59c38e961ccd68b036a2c52403728 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1abcd5b84fc503fdc6a2502adeebf813d4 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d PiperOrigin-RevId: 853719463
|
@ashiqimranintel Could you please help take a look at the CI failures? |
…ntel GPU Imported from GitHub PR #35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a2 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd PiperOrigin-RevId: 853719463
…ntel GPU Imported from GitHub PR openxla/xla#35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a21e9e59c38e961ccd68b036a2c52403728 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1abcd5b84fc503fdc6a2502adeebf813d4 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 Reverts bbea049 FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d PiperOrigin-RevId: 853719463
|
@penpornk , the failures are unrelated to this PR. |
|
@penpornk can you check again? |
|
@penpornk , I resolved the conflict, can you check again? |
|
Sorry for the delay. Rerunning the tests now. |
|
@penpornk , the errors are unrelated to my PR, can you check again? |
…ntel GPU Imported from GitHub PR #35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a2 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd PiperOrigin-RevId: 853719463
…ntel GPU Imported from GitHub PR openxla/xla#35304 This PR will enable sycl/oneapi plugin support for Intel GPU Copybara import of the project: -- cca65a21e9e59c38e961ccd68b036a2c52403728 by Ashiq Imran <ashiq.imran@intel.com>: Adding sycl/oneapi plugin support -- a0a9db1abcd5b84fc503fdc6a2502adeebf813d4 by Ashiq Imran <ashiq.imran@intel.com>: addressing review comments -- 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d by ag.ramesh <ag.ramesh@intel.com>: Added if_cuda_is_configured check that was removed and reverted incorrect change. Merging this change closes #35304 FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#35304 from Intel-tensorflow:aimran/oneapi_plugin 8d2f1dd19d4e6ba9d521003b4f047e687feffd3d PiperOrigin-RevId: 853719463
This PR will enable sycl/oneapi plugin support for Intel GPU