(windows) Crashes with `gfx-portability-icd` and `swiftshader-icd` registered

[saschawillems]

(windows) Crashes with `gfx-portability-icd` and `swiftshader-icd` registered

[SaschaWillems]

My patch for swiftshader got approved. If you built again from the current source, you should be able to use it with the caps viewer.

gfx-portability crashes with the same error at vkGetPhysicalDeviceWin32PresentationSupportKHR. I'll also submit a patch for that.

[SaschaWillems]

Not necessary. I already have it up-and-running with gfx-portability and will submit a patch asap ;)

[Hugobros3]

For good measure, here is my vulkaninfo output with both ICDs registered

[SaschaWillems]

How do I register swiftshader as a valid ICD? The loader has a list of known device, and refuses to load the swiftshader ICD for me.

[Hugobros3]

On Windows you go through the registry:

I added a DWORD key in HKEY_LOCAL_MACHINE\SOFTWARE\Khronos\Vulkan\Drivers named
C:\Users\Gobrosse\git\swiftshader\out\build\x64-Release\Windows\vk_swiftshader_icd.json.

The key has to have value zero, otherwise it's ignored. This is following the official vulkan loader documentation.

[SaschaWillems]

Been there done that ;) But at least a current loader will disable that driver because it's not a known icd, see https://github.com/KhronosGroup/Vulkan-Loader/blob/80e9b24ff8e536427b7e99bc0f07792bc9647d10/loader/loader.c#L818. Are you using an older loader?

[Hugobros3]

I have SDK 1.1.101 installed, that sounds like a very restrictive change on the part of the loader indeed 🤔 Not sure how things would work out for projects such as swiftshader, they don't have a "device" to put these keys in...

[SaschaWillems]

Seems like it was recently introduced. I have opened an Issue on this over at the loader repo.

The change also queries DXGI to see if a device for a vendor is present, so it's not even possible to just rename the swiftshader ICD to something else.

Once this is resolved I'll check how to make the caps viewer work with swiftshader.

[Hugobros3]

The docs for the loader mention you can use an environment variable (VK_ICD_FILENAMES) to override the ICD search path

set VK_ICD_FILENAMES=\windows\system32\nv-vk64.json

[SaschaWillems]

That one works. Thanks.

And it looks like it's crashing because swiftshader isn't exporting vkGetPhysicalDeviceWin32PresentationSupportKHR.

I'll take a look at how to work around this.

[Hugobros3]

Both Swiftshader and gfx-portability are in early stages anyway and aren't too usable yet, but it would be really nice exposure and a cool way to track their progress to have them listed on the vulkan.gpuinfo.org DB. If VulkanCapsViewer can't support them right now it's not a big deal, this issue will just track that progress.

[SaschaWillems]

Swiftshader is already able to run most of my samples, and it looks like it's really only about that single function not being exported. Not sure how to handle that yet though.

[Hugobros3]

Works fine, thanks! :)

Will look into fixing portability too one of those days, it's still quite incomplete in it's DX12 backend ( no descriptor set support if i'm correct, which is obviously quite limiting )

[Hugobros3]

Could you handle "VK_PHYSICAL_DEVICE_TYPE_CPU" ? The report shows up as "unknown device" on the site

[SaschaWillems]

My patch for gfx-portability was also merged: gfx-rs/portability@a9edbc0

Will also add device_type_cpu in the next update.