Skip to content
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

Revert "util: Spoof Ada and later GPUs as Ampere for Ghost of Tsushima" #219

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Saancreed
Copy link
Collaborator

This reverts commit f86f56b.

To be merged after DLSS FG support is in place. Let me know if I should actually write a new commit message.

@jp7677
Copy link
Owner

jp7677 commented Nov 2, 2024

Thanks, PR is good ofc. Would be nice if the commit message at least briefly states why we don't need this any longer.

This reverts commit f86f56b.

The issue was caused by DLSS-FG being in theory supported by reported hardware but the system / Wine environment failing required checks (hardware scheduling and nvofapi), making the game enter a half-broken state where it tripped over its own present calls. This should no longer be the case, at least in Proton.
@shelterx
Copy link

shelterx commented Nov 5, 2024

The workaround was made for a reason, are we sure it isn’t needed?

HansKristian-Work/vkd3d-proton#2008

@Saancreed
Copy link
Collaborator Author

@shelterx The cause of this issue will be gone from Proton soon so yes, we are… fairly confident about it.

But feel free to verify this and report back; while I can no longer notice the problem myself, another confirmation would be appreciated. Just don't forget to add Liam's D3DKMT patches to the mix and install nvofapi64.dll into Proton prefix.

@shelterx
Copy link

shelterx commented Nov 5, 2024

@Saancreed seems to work yes from what I could tell, with DLSS FG and Reflex On
I also of course tried FSR...

@Saancreed
Copy link
Collaborator Author

That's great, thanks for checking!

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.

3 participants