[SYCL] Adds support for device UUID as a SYCL extension. #3696

rbegam · 2021-05-05T21:53:03Z

This includes support for only level_zero. The
llvm/sycl/docs/extensions/IntelGPU/IntelGPUDeviceInfo.md doc has
been modified with a brief description of the extension. A new aspect
is added to indicate if the support is available.

Signed-off-by: rbegam [email protected]

sycl/doc/extensions/IntelGPU/IntelGPUDeviceInfo.md

smaslov-intel · 2021-05-05T22:05:21Z

sycl/doc/extensions/IntelGPU/IntelGPUDeviceInfo.md

If we wanted this to be Level-Zero only then we'd add this extension to https://github.com/intel/llvm/blob/sycl/sycl/doc/extensions/LevelZeroBackend/LevelZeroBackend.md. My understanding is that the intention is to have it supported at least with OpenCL too, so let's not have this description be too Level-Zero centric.

ok, will modify.

sycl/plugins/level_zero/pi_level_zero.cpp

smaslov-intel · 2021-05-05T22:19:13Z

sycl/include/CL/sycl/detail/pi.h

I realized that this is not a good idea to use STL in PI API. The goal was to keep it C-only such that there are no C++ compatibility issues. Let's instead have the first call to know the size, have users allocate a C-array, and have the second call to populate it.

sycl/include/CL/sycl/detail/pi.h

sycl/include/CL/sycl/feature_test.hpp

sycl/source/detail/device_impl.cpp

smaslov-intel

Please address review comments

sycl/doc/extensions/IntelGPU/IntelGPUDeviceInfo.md

sycl/include/CL/sycl/feature_test.hpp

s-kanaev · 2021-05-11T14:40:20Z

sycl/source/detail/device_impl.cpp

In the end this memory is going to leak, isn't it?
Hence, with a long running application, RAM consumption is going to increase this tiny bit at a time.
I think, this should be mitigated somehow.

thanks for pointing this out.

tagging @smaslov-intel @gmlueck

I assume that we will change the DPC++ interface back to std::array (see my other comment). In that case, this code here will be a memory leak, which is bad. If the DPC++ interface assumes that the UUID is always 16-bytes, then there is no point in trying to support an arbitrary size here. Therefore, you could just allocate a fixed 16-byte array to receive the results from the PI query. Something like:

assert(return_size <= 16); std::byte buf[16]; /* read contents of "buf" from PI layer */

If UUID is always 16-bytes (which it is), then do we even need 2 calls here?

@smaslov-intel

Since PI interface readily supports receiving the size then I'd still go with 2 calls. You can additionally assert here that the returned size is 16 since you'd need to truncate it to fit the array of 16-bytes returned in SYCL API.

Sorry, could somebody please clarify the resolution here? Will return value of the SYCL device query be std::array<uint8_t, 16>?

it's std::array<std::byte, 16>

s-kanaev · 2021-05-11T14:41:40Z

sycl/source/detail/device_info.hpp

NIT: It's better to move this kind of changes into a distinct patch.

gmlueck · 2021-05-11T14:58:28Z

sycl/doc/extensions/IntelGPU/IntelGPUDeviceInfo.md

Why did you change the return type here? Returning std::array<std::byte, 16> was much better, I think. With uint8_t *, it's unclear whether the caller is supposed to deallocate the memory.

@smaslov-intel pointed out that it's preferable to not use STLs to maintain C compatibility. Changing this does include the requirement for deallocation.

My understanding was that @smaslov-intel was concerned about the PI interface being C compatible. This interface here is the SYCL / DPC++ interface, which is assumed to be C++. There are other existing device descriptors that return STL types (e.g. info::device::sub_group_sizes returns an std::vector), so I think it's OK if this descriptor returns an std::array.

@gmlueck is right, I only wanted PI interface to stay C-only

my bad, will fix it.

sycl/include/CL/sycl/aspects.hpp

sycl/include/CL/sycl/detail/pi.h

romanovvlad · 2021-05-12T14:22:25Z

sycl/source/detail/device_impl.cpp

Probably I'm missing something, but won't this call write UUID values to the device_type variable? If so, why?
Shouldn't 3rg argument be 0 and 4th nullptr?

here trying to get the size.

It seems we should pass 0 and nullptr as 3rd and 4th arguments. Otherwise low level runtime will try to write N bytes to the memory pointed by &device_type.

@rbegam Could you please address this comment?

did you mean sizeof(pi_device_type), &device_type these two parameters? I mistakenly thought &device_type and &return_size. I think you are right. I am fixing this.

sycl/source/detail/device_impl.cpp

romanovvlad · 2021-05-12T14:26:10Z

sycl/source/detail/device_impl.cpp

Sorry, could somebody please clarify the resolution here? Will return value of the SYCL device query be std::array<uint8_t, 16>?

sycl/include/CL/sycl/info/info_desc.hpp

sycl/include/CL/sycl/detail/pi.h

smaslov-intel

LGTM overall. just few small comments.

rbegam · 2021-05-25T19:49:30Z

ping @smaslov-intel

smaslov-intel

LGTM. Please add a test too.

sycl/include/CL/sycl/detail/type_traits.hpp

sycl/source/detail/device_impl.cpp

sycl/test/on-device/extensions/intel-ext-device.cpp

Signed-off-by: rbegam <[email protected]>

Signed-off-by: rehana begam <[email protected]>

bader

@rbegam, please, avoid forced pushed. It makes tracking conversations difficult.

romanovvlad

LGTM

Signed-off-by: rehana begam <[email protected]>

bader · 2021-07-01T09:20:21Z

I still see a bunch of unaddressed comments. What are we going to do with them?

rbegam · 2021-07-01T17:59:46Z

@bader all those comments were discussed offline and resolved. I've marked them as so now.

rbegam · 2021-07-01T18:17:53Z

ping @gmlueck

* upstream/sycl: (649 commits) [SYCL][Driver][NFC] Update integration footer test for 32-bit host (intel#4039) [SYCL][L0] Initialize descriptor .stype and .pNext (intel#4032) [SYCL] Add sycl::kernel::get_kernel_bundle method (intel#3855) [SYCL] Add support for device UUID as a SYCL extension. (intel#3696) [SYCL][Matrix] Add spec document for the matrix extension interface and its first implementation for AMX (intel#3551) Fix debug build mangler test after PR#3992 (8f38045). (intel#4033) [Driver][SYCL] Restrict user -include file in final integration footer step (intel#4036) [SYCL] [Tests] Do not copy device binary image mocks (intel#4023) [SYCL][Doc] Update docs to reflect new compiler features (intel#4030) [SYCL][CUDA] cl_khr_fp16 extension connected to cuda PI. (intel#4029) [SYCL][NFC] Refactor RT unit tests (intel#4021) [SYCL] Switch to using integration footer by default (intel#3777) [SYCL][CUDA] Add the Use Default Stream property (intel#4004) Uplift GPU RT version for Linux to 21.24.20098 (intel#4003) [SYCL][CUDA] atomic_ref.fetch_add used for fp64 reduction if device.has(atomic64) (intel#3950) [Driver][SYCL] Differentiate host dependency link from regular host link (intel#4002) [SYCL][ESIMD] Support device half type in intrinsics. (intel#4024) [SYCL] Allow fpga_reg only for PODs and Trivially-copyable structs (intel#3643) [SYCL][FPGA] Restore legacy debug info version for the hardware (intel#3991) [SYCL][PI][L0] Force reset of memcpy command-list. (intel#4001) ...

zejun-chen · 2024-07-05T05:39:30Z

Hi, @rbegam

I have a question. For the device UUID, is there any platform, whose backend is sycl::backend::ext_oneapi_level_zero, but doesn't support device UUID? Or can i say all sycl devices support the device UUID.

Thank you.

Bumps [urllib3](https://github.com/urllib3/urllib3) from 2.5.0 to 2.6.0. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/urllib3/urllib3/releases">urllib3's releases</a>.</em></p> <blockquote> <h2>2.6.0</h2> <h2>🚀 urllib3 is fundraising for HTTP/2 support</h2> <p><a href="https://sethmlarson.dev/urllib3-is-fundraising-for-http2-support">urllib3 is raising ~$40,000 USD</a> to release HTTP/2 support and ensure long-term sustainable maintenance of the project after a sharp decline in financial support. If your company or organization uses Python and would benefit from HTTP/2 support in Requests, pip, cloud SDKs, and thousands of other projects <a href="https://opencollective.com/urllib3">please consider contributing financially</a> to ensure HTTP/2 support is developed sustainably and maintained for the long-haul.</p> <p>Thank you for your support.</p> <h2>Security</h2> <ul> <li>Fixed a security issue where streaming API could improperly handle highly compressed HTTP content ("decompression bombs") leading to excessive resource consumption even when a small amount of data was requested. Reading small chunks of compressed data is safer and much more efficient now. (CVE-2025-66471 reported by <a href="https://github.com/Cycloctane"><code>@Cycloctane</code></a>, 8.9 High, GHSA-2xpw-w6gg-jr37)</li> <li>Fixed a security issue where an attacker could compose an HTTP response with virtually unlimited links in the <code>Content-Encoding</code> header, potentially leading to a denial of service (DoS) attack by exhausting system resources during decoding. The number of allowed chained encodings is now limited to 5. (CVE-2025-66418 reported by <a href="https://github.com/illia-v"><code>@illia-v</code></a>, 8.9 High, GHSA-gm62-xv2j-4w53)</li> </ul> <blockquote> <p>[!IMPORTANT]</p> <ul> <li>If urllib3 is not installed with the optional <code>urllib3[brotli]</code> extra, but your environment contains a Brotli/brotlicffi/brotlipy package anyway, make sure to upgrade it to at least Brotli 1.2.0 or brotlicffi 1.2.0.0 to benefit from the security fixes and avoid warnings. Prefer using <code>urllib3[brotli]</code> to install a compatible Brotli package automatically.</li> <li>If you use custom decompressors, please make sure to update them to respect the changed API of <code>urllib3.response.ContentDecoder</code>.</li> </ul> </blockquote> <h2>Features</h2> <ul> <li>Enabled retrieval, deletion, and membership testing in <code>HTTPHeaderDict</code> using bytes keys. (<a href="https://github.com/urllib3/urllib3/issues/3653">#3653</a>)</li> <li>Added host and port information to string representations of <code>HTTPConnection</code>. (<a href="https://github.com/urllib3/urllib3/issues/3666">#3666</a>)</li> <li>Added support for Python 3.14 free-threading builds explicitly. (<a href="https://github.com/urllib3/urllib3/issues/3696">#3696</a>)</li> </ul> <h2>Removals</h2> <ul> <li>Removed the <code>HTTPResponse.getheaders()</code> method in favor of <code>HTTPResponse.headers</code>. Removed the <code>HTTPResponse.getheader(name, default)</code> method in favor of <code>HTTPResponse.headers.get(name, default)</code>. (<a href="https://github.com/urllib3/urllib3/issues/3622">#3622</a>)</li> </ul> <h2>Bugfixes</h2> <ul> <li>Fixed redirect handling in <code>urllib3.PoolManager</code> when an integer is passed for the retries parameter. (<a href="https://github.com/urllib3/urllib3/issues/3649">#3649</a>)</li> <li>Fixed <code>HTTPConnectionPool</code> when used in Emscripten with no explicit port. (<a href="https://github.com/urllib3/urllib3/issues/3664">#3664</a>)</li> <li>Fixed handling of <code>SSLKEYLOGFILE</code> with expandable variables. (<a href="https://github.com/urllib3/urllib3/issues/3700">#3700</a>)</li> </ul> <h2>Misc</h2> <ul> <li>Changed the <code>zstd</code> extra to install <code>backports.zstd</code> instead of <code>zstandard</code> on Python 3.13 and before. (<a href="https://github.com/urllib3/urllib3/issues/3693">#3693</a>)</li> <li>Improved the performance of content decoding by optimizing <code>BytesQueueBuffer</code> class. (<a href="https://github.com/urllib3/urllib3/issues/3710">#3710</a>)</li> <li>Allowed building the urllib3 package with newer setuptools-scm v9.x. (<a href="https://github.com/urllib3/urllib3/issues/3652">#3652</a>)</li> <li>Ensured successful urllib3 builds by setting Hatchling requirement to ≥ 1.27.0. (<a href="https://github.com/urllib3/urllib3/issues/3638">#3638</a>)</li> </ul> </blockquote> </details> <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/urllib3/urllib3/blob/main/CHANGES.rst">urllib3's changelog</a>.</em></p> <blockquote> <h1>2.6.0 (2025-12-05)</h1> <h2>Security</h2> <ul> <li>Fixed a security issue where streaming API could improperly handle highly compressed HTTP content ("decompression bombs") leading to excessive resource consumption even when a small amount of data was requested. Reading small chunks of compressed data is safer and much more efficient now. (<code>GHSA-2xpw-w6gg-jr37 <https://github.com/urllib3/urllib3/security/advisories/GHSA-2xpw-w6gg-jr37></code>__)</li> <li>Fixed a security issue where an attacker could compose an HTTP response with virtually unlimited links in the <code>Content-Encoding</code> header, potentially leading to a denial of service (DoS) attack by exhausting system resources during decoding. The number of allowed chained encodings is now limited to 5. (<code>GHSA-gm62-xv2j-4w53 <https://github.com/urllib3/urllib3/security/advisories/GHSA-gm62-xv2j-4w53></code>__)</li> </ul> <p>.. caution::</p> <ul> <li> <p>If urllib3 is not installed with the optional <code>urllib3[brotli]</code> extra, but your environment contains a Brotli/brotlicffi/brotlipy package anyway, make sure to upgrade it to at least Brotli 1.2.0 or brotlicffi 1.2.0.0 to benefit from the security fixes and avoid warnings. Prefer using <code>urllib3[brotli]</code> to install a compatible Brotli package automatically.</p> </li> <li> <p>If you use custom decompressors, please make sure to update them to respect the changed API of <code>urllib3.response.ContentDecoder</code>.</p> </li> </ul> <h2>Features</h2> <ul> <li>Enabled retrieval, deletion, and membership testing in <code>HTTPHeaderDict</code> using bytes keys. (<code>[#3653](urllib3/urllib3#3653) <https://github.com/urllib3/urllib3/issues/3653></code>__)</li> <li>Added host and port information to string representations of <code>HTTPConnection</code>. (<code>[#3666](urllib3/urllib3#3666) <https://github.com/urllib3/urllib3/issues/3666></code>__)</li> <li>Added support for Python 3.14 free-threading builds explicitly. (<code>[#3696](urllib3/urllib3#3696) <https://github.com/urllib3/urllib3/issues/3696></code>__)</li> </ul> <h2>Removals</h2> <ul> <li>Removed the <code>HTTPResponse.getheaders()</code> method in favor of <code>HTTPResponse.headers</code>. Removed the <code>HTTPResponse.getheader(name, default)</code> method in favor of <code>HTTPResponse.headers.get(name, default)</code>. (<code>[#3622](urllib3/urllib3#3622) <https://github.com/urllib3/urllib3/issues/3622></code>__)</li> </ul> <h2>Bugfixes</h2> <ul> <li>Fixed redirect handling in <code>urllib3.PoolManager</code> when an integer is passed for the retries parameter. (<code>[#3649](urllib3/urllib3#3649) <https://github.com/urllib3/urllib3/issues/3649></code>__)</li> <li>Fixed <code>HTTPConnectionPool</code> when used in Emscripten with no explicit port. (<code>[#3664](urllib3/urllib3#3664) <https://github.com/urllib3/urllib3/issues/3664></code>__)</li> <li>Fixed handling of <code>SSLKEYLOGFILE</code> with expandable variables. (<code>[#3700](urllib3/urllib3#3700) <https://github.com/urllib3/urllib3/issues/3700></code>__)</li> </ul>  </blockquote> <p>... (truncated)</p> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/urllib3/urllib3/commit/720f484b605f18887a48eef448d0084e2b76902d"><code>720f484</code></a> Release 2.6.0</li> <li><a href="https://github.com/urllib3/urllib3/commit/24d7b67eac89f94e11003424bcf0d8f7b72222a8"><code>24d7b67</code></a> Merge commit from fork</li> <li><a href="https://github.com/urllib3/urllib3/commit/c19571de34c47de3a766541b041637ba5f716ed7"><code>c19571d</code></a> Merge commit from fork</li> <li><a href="https://github.com/urllib3/urllib3/commit/816fcf04528bc0f89672e13398eb813dcc892490"><code>816fcf0</code></a> Bump actions/setup-python from 6.0.0 to 6.1.0 (<a href="https://github.com/urllib3/urllib3/issues/3725">#3725</a>)</li> <li><a href="https://github.com/urllib3/urllib3/commit/18af0a10efc4c99dd028f7ad5a461470b9a8b0fd"><code>18af0a1</code></a> Improve speed of <code>BytesQueueBuffer.get()</code> by using memoryview (<a href="https://github.com/urllib3/urllib3/issues/3711">#3711</a>)</li> <li><a href="https://github.com/urllib3/urllib3/commit/1f6abac3e6d426c3939b8a17cf4afa099e691ab2"><code>1f6abac</code></a> Bump versions of pre-commit hooks (<a href="https://github.com/urllib3/urllib3/issues/3716">#3716</a>)</li> <li><a href="https://github.com/urllib3/urllib3/commit/1c8fbf787b8e6ed151842c5d6874c9d5bdbf1d0b"><code>1c8fbf7</code></a> Bump actions/checkout from 5.0.0 to 6.0.0 (<a href="https://github.com/urllib3/urllib3/issues/3722">#3722</a>)</li> <li><a href="https://github.com/urllib3/urllib3/commit/7784b9eee95b7c90802c02b111e98df70259ae4f"><code>7784b9e</code></a> Add Python 3.15 to CI (<a href="https://github.com/urllib3/urllib3/issues/3717">#3717</a>)</li> <li><a href="https://github.com/urllib3/urllib3/commit/0241c9e7286d3008e3cce18effc13b40dc633385"><code>0241c9e</code></a> Updated docs to reflect change in optional zstd dependency from <code>zstandard</code> t...</li> <li><a href="https://github.com/urllib3/urllib3/commit/7afcabb6489d9a8ea95a40e5afcb46463af17351"><code>7afcabb</code></a> Expand environment variable of SSLKEYLOGFILE (<a href="https://github.com/urllib3/urllib3/issues/3705">#3705</a>)</li> <li>Additional commits viewable in <a href="https://github.com/urllib3/urllib3/compare/2.5.0...2.6.0">compare view</a></li> </ul> </details> <br /> [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=urllib3&package-manager=pip&previous-version=2.5.0&new-version=2.6.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) You can disable automated security fix PRs for this repo from the [Security Alerts page](https://github.com/intel/llvm/network/alerts). </details> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

rbegam requested review from a team and smaslov-intel as code owners May 5, 2021 21:53

smaslov-intel reviewed May 5, 2021

View reviewed changes

sycl/doc/extensions/IntelGPU/IntelGPUDeviceInfo.md Outdated Show resolved Hide resolved

smaslov-intel reviewed May 5, 2021

View reviewed changes

sycl/plugins/level_zero/pi_level_zero.cpp Outdated Show resolved Hide resolved

smaslov-intel reviewed May 5, 2021

View reviewed changes

sycl/include/CL/sycl/detail/pi.h Outdated Show resolved Hide resolved

smaslov-intel reviewed May 5, 2021

View reviewed changes

sycl/include/CL/sycl/feature_test.hpp Outdated Show resolved Hide resolved

smaslov-intel reviewed May 5, 2021

View reviewed changes

sycl/source/detail/device_impl.cpp Outdated Show resolved Hide resolved

smaslov-intel approved these changes May 5, 2021

View reviewed changes

smaslov-intel suggested changes May 5, 2021

View reviewed changes

gmlueck reviewed May 10, 2021

View reviewed changes

sycl/doc/extensions/IntelGPU/IntelGPUDeviceInfo.md Outdated Show resolved Hide resolved

bader requested review from gmlueck, s-kanaev and smaslov-intel May 11, 2021 11:21

s-kanaev reviewed May 11, 2021

View reviewed changes

gmlueck reviewed May 11, 2021

View reviewed changes

romanovvlad reviewed May 12, 2021

View reviewed changes

rbegam force-pushed the uuid branch from b64dd90 to 367d322 Compare May 14, 2021 22:04

smaslov-intel reviewed May 15, 2021

View reviewed changes

sycl/include/CL/sycl/info/info_desc.hpp Outdated Show resolved Hide resolved

smaslov-intel reviewed May 15, 2021

View reviewed changes

sycl/include/CL/sycl/detail/pi.h Outdated Show resolved Hide resolved

smaslov-intel suggested changes May 15, 2021

View reviewed changes

smaslov-intel previously approved these changes May 25, 2021

View reviewed changes

romanovvlad reviewed May 26, 2021

View reviewed changes

sycl/include/CL/sycl/detail/type_traits.hpp Outdated Show resolved Hide resolved

romanovvlad reviewed May 26, 2021

View reviewed changes

sycl/source/detail/device_impl.cpp Outdated Show resolved Hide resolved

romanovvlad reviewed May 26, 2021

View reviewed changes

sycl/source/detail/device_impl.cpp Outdated Show resolved Hide resolved

romanovvlad reviewed May 26, 2021

View reviewed changes

sycl/test/on-device/extensions/intel-ext-device.cpp Outdated Show resolved Hide resolved

rbegam added 8 commits June 28, 2021 12:56

fix has aspect.

1468a29

Signed-off-by: rbegam <[email protected]>

add guard with C++17 check.

7256444

Signed-off-by: rbegam <[email protected]>

remove unnecessary code from the test.

a7de8de

Signed-off-by: rbegam <[email protected]>

changes uuid type and adds a unittest.

4160272

Signed-off-by: rbegam <[email protected]>

clang formatted.

2d1d711

Signed-off-by: rbegam <[email protected]>

remove old code.

244a082

Signed-off-by: rbegam <[email protected]>

modified the unittest.

b344f6d

Signed-off-by: rbegam <[email protected]>

minor corrections.

bddc7bc

Signed-off-by: rehana begam <[email protected]>

rbegam dismissed stale reviews from s-kanaev, smaslov-intel, and gmlueck via bddc7bc June 28, 2021 20:35

rbegam force-pushed the uuid branch from 29db260 to bddc7bc Compare June 28, 2021 20:35

bader previously approved these changes Jun 29, 2021

View reviewed changes

romanovvlad previously approved these changes Jun 30, 2021

View reviewed changes

rearranged ext.

22fa8f6

Signed-off-by: rehana begam <[email protected]>

rbegam dismissed stale reviews from romanovvlad and bader via 22fa8f6 June 30, 2021 22:20

Merge branch 'sycl' into uuid

8ed9861

romanovvlad approved these changes Jul 1, 2021

View reviewed changes

smaslov-intel approved these changes Jul 1, 2021

View reviewed changes

gmlueck approved these changes Jul 1, 2021

View reviewed changes

bader merged commit 25aee28 into intel:sycl Jul 2, 2021

[SYCL] Adds support for device UUID as a SYCL extension. #3696

[SYCL] Adds support for device UUID as a SYCL extension. #3696

Uh oh!

Conversation

rbegam commented May 5, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

smaslov-intel left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

smaslov-intel left a comment

Choose a reason for hiding this comment

Uh oh!

rbegam commented May 25, 2021

Uh oh!

smaslov-intel left a comment

Choose a reason for hiding this comment

Uh oh!

rbegam commented May 5, 2021 •

edited

Loading

zejun-chen commented Jul 5, 2024 •

edited

Loading