Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Conversation

@LloydNicholson
Copy link

@LloydNicholson LloydNicholson commented Jul 30, 2025

This is my first time I have written c or c++, so please let me know what else I can do to make this code more robust or any refactors that are required.

Please let me know how I can improve the code. I am under no illusion that this not everything needed for this to work across different 9060 series (gfx1200/navi44) cards but I have it working on mine and that was my goal from the start anyway.

I honestly just wanted to have a better experience with a game I was enjoying and stop enjoying due to an update, so I fixed it to the best of my ability and maybe it can help to get this change into dev.

I did it to the point where I can run Indiana Jones and the Great Circle because it was bugging me that I couldn't and there we some serious performance issues around the save points and I knew AMDVLK could fix the issue.

I haven't done any other meaningful testing other than the fact that my ASRock Challenger 9060 XT 16GB works perfectly so far with AMDVLK in Indiana Jones instead of throwing a VK_ERROR_INCOMPATIBLE_DRIVER.

I added a complimentary change that works together to the llpc repo as well: GPUOpen-Drivers/llpc#2917

@KeyserSoze1
Copy link

Well if this isn't what open source is about I don't know what is. You had an issue and was able to resolve it yourself.
While I can't comment on the quality of the code I can say I was able to get this to compile/install (along with the associated llpc branch) on my Ubuntu 25.04 machine with Mesa 25.1.6 installed. vulkaninfo is reporting successfully but I haven't tried anything beyond that.

I'm using a XFX Swift 9060 XT 16GB to add to the list of what appears to be working variations of 9060's.

Just want to say thanks for taking the effort to work this and push it out for others to be able to use.

@jinjianrong
Copy link
Member

@LloydNicholson Thanks for your contribution. However, the AMDVLK open source project is discontinued as announced here. You are welcome to contribute to RADV.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants