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

rsx: Do not require ZCULL buffer binding to enable ZPASS counting #12432

Merged
merged 2 commits into from
Jul 31, 2022

Conversation

kd-11
Copy link
Contributor

@kd-11 kd-11 commented Jul 31, 2022

  • ZPASS data is still accessible in unbuffered mode.
    The only thing that buffered ZCULL enables is something closer to early-Z where large blocks of pixels can be dicarded earlier.
    It is strictly a performance optimization and not required for ZPASS to work.
  • Update ZCULL stat calculations to take into account unbuffered Z

Fixes #10869
Fixes #7741

- ZPASS data is still accessible in unbuffered mode.
  The only thing that buffered ZCULL enables is something closer to early-Z where large blocks of pixels can be dicarded earlier.
  It is strictly a performance optimization and not required for ZPASS to work.
- Update ZCULL stat calculations to take into account unbuffered Z
@Emulator-Team-2
Copy link

Emulator-Team-2 commented Jul 31, 2022

Fixes: #10869.

ZCULL Accuracy setting: precise.

Deadpool:

(Before/After Comparisons)

Before - (RPCS3-0.0.23-13986):
1

After - (PR):
2


Transformers: Fall of Cybertron:

(Before/After Comparisons)

Before - (RPCS3-0.0.23-13986):
3

After - (PR):
4


Shaun White Skateboarding:

(Before/After Comparisons)

Before - (RPCS3-0.0.23-13986 - OpenGL):
5

After - (PR - OpenGL):
6

@Megamouse Megamouse added the RSX label Jul 31, 2022
@Emulator-Team-2
Copy link

Emulator-Team-2 commented Jul 31, 2022

Fixes: #7741.

Dante's Inferno:

ZCULL Accuracy setting: precise.

(Before/After Comparisons)

Before - (RPCS3-0.0.23-13986):
1

After - (PR):
2

@Darkhost1999
Copy link
Contributor

How about Lucifer is he rendering in cutscenes?

@Jonathan44062
Copy link

Still an issue
image
Also, I tested with Read Color Buffers to fix black boxes, but is the same
Maybe is caused by a different RSX bug

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

Successfully merging this pull request may close these issues.

Bug: Broken graphics in Deadpool - [BLUS31146] dantes inferno graphical errors
5 participants