Skip to content

[SYCL][Doc] Update restrictions in extension #10281

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

Merged
merged 2 commits into from
Jul 10, 2023

Conversation

gmlueck
Copy link
Contributor

@gmlueck gmlueck commented Jul 10, 2023

Update the description of the limitations in the
"sycl_ext_oneapi_device_architecture" extension. The newly added host APIs in that extension do not have the same restrictions as the if_architecture_is function.

Update the description of the limitations in the
"sycl_ext_oneapi_device_architecture" extension.  The newly added host
APIs in that extension do not have the same restrictions as the
`if_architecture_is` function.
@gmlueck gmlueck requested a review from a team as a code owner July 10, 2023 14:48
@gmlueck
Copy link
Contributor Author

gmlueck commented Jul 10, 2023

@dm-vodopyanov are there any other restrictions we should capture for this extension? For the new host-side APIs, do they work with all backends or only with Level Zero?

@dm-vodopyanov
Copy link
Contributor

@dm-vodopyanov are there any other restrictions we should capture for this extension? For the new host-side APIs, do they work with all backends or only with Level Zero?

Currently host-side APIs work for Level Zero and OpenCL backends only, and so only Intel GPU architectures are covered.
There might be a limitation related to querying CPU architectures: we currently report that the architecture is just x86_64, we don't specify the exact CPU architecture like SKL, SPR, etc.
For device-side APIs, no other restrictions.

@gmlueck
Copy link
Contributor Author

gmlueck commented Jul 10, 2023

Currently host-side APIs work for Level Zero and OpenCL backends only, and so only Intel GPU architectures are covered.
There might be a limitation related to querying CPU architectures: we currently report that the architecture is just x86_64, we don't specify the exact CPU architecture like SKL, SPR, etc.

Thanks. I noted the limitations with Nvidia and AMD devices in f97f299.

I think the CPU issue you describe does not need to be noted as a limitation because the current API does not provide any way to get more detail about the CPU device architecture. We might decide to add this granularity in the future, but for now it is not part of the extension.

@dm-vodopyanov dm-vodopyanov merged commit a69a54f into intel:sycl Jul 10, 2023
@gmlueck gmlueck deleted the gmlueck/device-arch-restrictions branch July 10, 2023 21:54
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.

3 participants