Skip to content
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

[GPU] Try catch for device detection #17011

Conversation

vladimir-paramuzov
Copy link
Contributor

Details:

  • Added try/catch for device creation loop to handle possible exception on getDevices() call which can throw an exception for some weird platforms and prevent usage of properly installed OCL runtime for GPU. So now an exception on device creation leads to silent platform skip

Tickets:

  • 108597

@vladimir-paramuzov vladimir-paramuzov added the category: GPU OpenVINO GPU plugin label Apr 18, 2023
@vladimir-paramuzov vladimir-paramuzov added this to the 2023.0 milestone Apr 18, 2023
@vladimir-paramuzov vladimir-paramuzov requested review from a team as code owners April 18, 2023 07:48
@ilya-lavrenov ilya-lavrenov added the port to 2022.3 Need port from master to 2022.3 LTS label Apr 18, 2023
@p-durandin p-durandin merged commit dbd20ec into openvinotoolkit:master Apr 19, 2023
ilya-lavrenov pushed a commit to ilya-lavrenov/openvino that referenced this pull request May 2, 2023
@vladimir-paramuzov vladimir-paramuzov deleted the try_catch_for_device_detection branch May 3, 2023 04:54
vladimir-paramuzov added a commit to vladimir-paramuzov/openvino that referenced this pull request May 3, 2023
ilya-lavrenov added a commit that referenced this pull request May 19, 2023
* Update ie_common.h (#16860)

* [GPU] Added try/catch for device detection loop to skip platforms which throw an exception (#17011)

* ONNX FE - model loading fix (#17091)

* Path retrieval fix

* More detailed messages in the failing test

* Exe path with model name

---------

Co-authored-by: Michal Lukaszewski <[email protected]>

* Upper-bound for patchelf (#17177)

* Apply suggestions from code review

---------

Co-authored-by: Ian Hunter <[email protected]>
Co-authored-by: Vladimir Paramuzov <[email protected]>
Co-authored-by: Tomasz Dołbniak <[email protected]>
Co-authored-by: Michal Lukaszewski <[email protected]>
Co-authored-by: Alina Kladieva <[email protected]>
@ilya-lavrenov ilya-lavrenov added port to 2022.3 Need port from master to 2022.3 LTS and removed port to 2022.3 Need port from master to 2022.3 LTS labels Jun 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: GPU OpenVINO GPU plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants