Skip to content

Conversation

@LloydNicholson
Copy link

@LloydNicholson LloydNicholson commented Jul 30, 2025

Added basic Navi44 Support

This is my first time I have written c or c++, so please let me know what else I can do to make this PR more robust.

Also, I am not even sure that I am doing this PR in the right way as well. Please let me know how I can improve it. Thank you. I honestly just wanted to have a better experience with a game I was enjoying and stop enjoying due to Update 4, so I fixed it to the best of my ability and maybe it can help some others to see the changes.

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 with AMDVLK in Indiana Jones instead of throwing a VK_ERROR_INCOMPATIBLE_DRIVER.

@Triang3l
Copy link

Triang3l commented Jul 30, 2025

Hello! Oh wow, this is super interesting, the desperation behind this… Thank you!

Can you please move the pull request to the https://github.com/GPUOpen-Drivers/pal repository where the original files are, so the file changes can be seen, and they potentially can be merged? The AMDVLK repo is used purely for an issue tracker and top-level overview documentation, while the code is located in the pal, llpc, gpurt and xgl repositories.

Are the modifications based on the Mesa drivers?

@LloydNicholson
Copy link
Author

LloydNicholson commented Jul 30, 2025

Oh okay I see. Thanks I will move it now. I have another change for llpc, so I will do that in that repo then seperately as well.

Yes, I ran all of this while having mesa-vulkan-git at Mesa 25.3.0-devel.

Edit:
Also, I ran this on Fedora. Might update the docs if I remember if there were any hiccups.

@LloydNicholson
Copy link
Author

@Triang3l Added the PR to pal repo now -> GPUOpen-Drivers/pal#109

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants