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

GS/Vulkan: Drop feedback loop extension #11124

Merged
merged 1 commit into from
Apr 23, 2024
Merged

GS/Vulkan: Drop feedback loop extension #11124

merged 1 commit into from
Apr 23, 2024

Conversation

stenzek
Copy link
Contributor

@stenzek stenzek commented Apr 22, 2024

Description of Changes

Apparently this causes GPU crashes on RDNA3, and didn't provide any tangible benefit for NVIDIA.

I'll replace this at some point with dynamic rendering local reads, either before or after the GPUDevice transition.

Rationale behind Changes

Might fix the crashes on RDNA3.

Suggested Testing Steps

Compare performance in barrier-heavy games on NVIDIA, make sure it is unaffected.
See if it fixes the GPU crashes on RDNA3.

Apparently this causes GPU crashes on RDNA3, and didn't provide any
tangible benefit for NVIDIA.

I'll replace this at some point with dynamic rendering local reads,
either before or after the GPUDevice transition.
@stenzek stenzek merged commit fe0e71f into PCSX2:master Apr 23, 2024
12 checks passed
@stenzek stenzek deleted the drop-feedback-loop branch April 23, 2024 01:57
@lextra2
Copy link

lextra2 commented Jul 5, 2024

@stenzek
This causes artifacts with Ratchet - Deadlocked water on my RDN2 GPU if:

  1. Internal Resolution is set higher than 3x
    and
  2. Blending Accuracy is set to higher than Minimum

Ratchet - Deadlocked_SCUS-97465_20240705052851

GS Dump:
Ratchet - Deadlocked_SCUS-97465_20240705052851.gs.zip

Tested on Windows 10 19044 with AMD Radeon Graphics Drivers 24.6.1 WHQL

I know y'all don't accept issues related to upscaling but I still wanted to inform you about this regression.

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

Successfully merging this pull request may close these issues.

2 participants