What extensions need to be reported through CL_PLATFORM_EXTENSIONS

[karolherbst]

One sentence in the spec which I'm not entirely sure how to parse is "Each extension that is supported by all devices associated with this platform must be reported here.".

As of now in rusticl/mesa we only report extensions which don't rely on any hardware support and only those we promise to support on every possibly supported device.

However one could also read this as "extensions supported by all devices by this platform need to be listed there". "all devices" as in all the currently available and listed ones.

We are probably one of the few OpenCL implementations where distinguishing between those two interpretations might even matter and I was wondering if it makes sense to clarify this or what the actual intention here was.

[bashbaug]

I remember being confused about this also.

I read "Each extension that is supported by all devices associated with this platform must be reported here" to mean:

• If an extension is listed in CL_PLATFORM_EXTENSIONS then it also must be listed in CL_DEVICE_EXTENSIONS for each of the devices enumerated by clGetDeviceIDs for that platform.
• In other words, CL_PLATFORM_EXTENSIONS contains the intersection of the set of extensions supported by all of the devices in the platform.

There's a CTS that kind of checks for this. It's a little weird though, in that it only tracks a specific set of extensions. IMHO this test should be expanded so it checks for all of the extensions reported in CL_PLATFORM_EXTENSIONS generally, and not only a specific set.

https://github.com/KhronosGroup/OpenCL-CTS/blob/main/test_conformance/api/test_platform.cpp#L24

[karolherbst]

I think this test looks entirely broken, especially as extensions and extensionsSupported are out of sync, probably because of the // "cl_APPLE_SetMemObjectDestructor", line. But extensionsSupported is all false anyway. And reading the code I don't think it could ever reach the vlog_error("Platform supports extension \"%s\" but device does not\n", check.

But yeah, I think it makes sense to clean this up and to clarify what we want to test here.

I can certainly change the behavior in rusticl to check all devices and report all common extensions, if that's what's expected here anyway.

[karolherbst]

I think my point here is, that we don't know how all the other implementations are behaving and if they'd even pass once that test gets fixed.

Should probably bring it up on the next WG meeting and see what others say.

[bashbaug]

KhronosGroup/OpenCL-CTS#2172 updates the test so it works the way I expect it should work.

It's passing on most of the implementations I've tested, though not all: one implementation is returning cl_khr_icd for the platform but not for the device.