[SYCL] Add UUID and sub-, sub-sub- device info in sycl-ls --verbose#13999
[SYCL] Add UUID and sub-, sub-sub- device info in sycl-ls --verbose#13999steffenlarsen merged 4 commits intointel:syclfrom
sycl-ls --verbose#13999Conversation
| if (Device.has(aspect::ext_intel_device_info_uuid)) { | ||
| auto UUID = Device.get_info<sycl::ext::intel::info::device::uuid>(); | ||
| std::cout << Prepend << "UUID : "; | ||
| for (int i = 0; i < 16; i++) { |
There was a problem hiding this comment.
Should you be using ZE_MAX_DEVICE_UUID_SIZE instead of assuming size is 16
There was a problem hiding this comment.
That can be done. But, Isn't the UUID size fixed to 16: https://github.com/intel/llvm/blob/sycl/sycl/doc/extensions/supported/sycl_ext_intel_device_info.md#device-uuid?
There was a problem hiding this comment.
If it is documented as 16 then ok.
There was a problem hiding this comment.
Or just use a range-based for loop on UUID
| @@ -0,0 +1,15 @@ | |||
| /* Test to check that sycl-ls is outputting UUID and number of sub and sub-sub | |||
| * devices. */ | |||
| // REQUIRES: gpu, level_zero | |||
There was a problem hiding this comment.
will this work on Gen12 ? Or is it PVC only?
There was a problem hiding this comment.
It should work on Gen12 as well. The test passes on Gen12 precommit: https://github.com/intel/llvm/actions/runs/9343215312/job/25714501971
|
Pre commit failure in Linux Gen12 is in |
|
@intel/llvm-gatekeepers The PR is ready! |
Example
sycl-ls --verboseoutput: