-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
GS: Default to OpenGL for RDNA 2 and 3 on windows. #12144
base: master
Are you sure you want to change the base?
Conversation
When someone buys an RDNA 3 GPU please debug the driver timeouts on windows to figure out what's causing the crashing, seems to be related to barriers. RDNA 2 and higher support feature level 12.2 so use that to detect RDNA 3.
d2d64fd
to
840342b
Compare
Why are we including RDNA2 in this? |
Exactly what im thinking. I haven't had any issues on my 6700XT RDNA2 and I use VK exclusively |
Because otherwise we'd have to hardcode every rdna3 and possibly future gpu to default to GL instead and I don't like that, you can still switch from Auto to Vulkan for rdna2. |
Probably would be a good idea to make an issue page about this, just in the case an external dev with a RDNA3 GPU sees it and might be able to debug it or just for the sake to keep track on the issue. |
In my link, shader model 6.8 is specifically singled out. Speaking of which, I guess resource binding could also be used to distinguish between fermi, kepler and maxwell. |
Ok so, I checked 6700 XT vs 7700S (and also a 780M) with dx caps viewer.. And indeed aside of reading tea leaves and then hoping for the best guess I couldn't spot night and day differences. Thankfully I then found D3d12info and D3d12infoDB and I figured I had been pretty stupid because the last compiled dxcaps exe is from before RDNA3 was even released (i.e. it doesn't have any of the fanciest new stuff). |
There's a number of differences in Vulkan caps as well, though if you go that route, it's probably easiest to just check |
I don't think so, that seems pretty much identical too (besides model-specific variations like compute units per array or total).
Conversely indeed there are still many smaller ones in vulkan (including different supported extensions and sub-properties). |
I was looking at VGPRs per SIMD but it looks like AMD only raised that for the bigger RDNA3 cards, and not the smaller ones. |
Description of Changes
When someone buys an RDNA 3 GPU please debug the driver timeouts on windows to figure out what's causing the crashes on Vulkan, seems to be related to barriers.
RDNA 2 and higher support feature level 12.2 so use that to detect RDNA 3.
Rationale behind Changes
Avoid driver timeouts on RDNA3 on Vulkan.
We need someone with an RDNA3 GPU to debug and fix the issue which we don't have.
Suggested Testing Steps
Test to make sure rdna 2 and 3 default to gl, older vulkan capable to vulkan, legacy to dx11.