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: Remove zclip hackery #11268

Merged
merged 4 commits into from Dec 22, 2021
Merged

rsx: Remove zclip hackery #11268

merged 4 commits into from Dec 22, 2021

Conversation

kd-11
Copy link
Contributor

@kd-11 kd-11 commented Dec 19, 2021

  • Calculates precise Z value as requested by the game.
  • Works properly if the underlying Z format matches the PS3 1:1 but may cause minor problems otherwise. This is a problem on AMD where integer formats are non-existent for depth wider than 16 bits.

Fixes #8950

AMD is still broken, but that is because of a hardware limitation. It was always broken, so no regression on that front.
Needs testing as it interacts with a lot of titles that have weird clip volumes.

- Calculates precise Z value as requested by the game
- Works properly if the underlying Z format matches the PS3 1:1 but may cause minor problems otherwise
@Linear524
Copy link

Interesting PR... , will test all my titles tonight :)

@DaishinSilviaGT
Copy link

It regressed on AMD, GT6 has broken skyboxes
GPU: RX 470 4 GB
No_Skybox.log.gz
no_more_skyboxes

@kd-11
Copy link
Contributor Author

kd-11 commented Dec 20, 2021

It regressed on AMD, GT6 has broken skyboxes GPU: RX 470 4 GB No_Skybox.log.gz no_more_skyboxes

I feared as much. I'll restore the unprecision hack for GPUs without integer Z buffers. This feature is completely useless in modern titles but it really helps with emulation :(

@ribolovac510
Copy link

The flickering is gone in Deadly Premonition, shadows looks the same as in the build before the regression.

PR:

rpcs3-v0.0.19-add0cb3d_win64.mp4

Master:

rpcs3-v0.0.19-13106-18f59d3b_win64.mp4

Old build:

rpcs3-v0.0.12-10844-2e88924c_win64.mp4

@MarioSonic2987
Copy link
Contributor

Fixes broken candles and overbloom in Saints Row: Gat out of Hell (only in OpenGL).

Before:
image
After:
image

With Vulkan, it looks like this. It's not caused by this PR.
image

@kd-11
Copy link
Contributor Author

kd-11 commented Dec 20, 2021

@DaishinSilviaGT Please try with the new build when you can.

@Megamouse Megamouse added the RSX label Dec 20, 2021
@Linear524
Copy link

Tested GT6 on my GF1080 - skybox is broken and it is not AMD issue... Nvidia shows the same bug:

BCES01893_screenshot_2021_12_20_18_59_02
BCES01893_screenshot_2021_12_20_18_59_12
BCES01893_screenshot_2021_12_20_18_59_14
BCES01893_screenshot_2021_12_20_18_56_37
BCES01893_screenshot_2021_12_20_18_57_14
BCES01893_screenshot_2021_12_20_18_58_37

@Linear524
Copy link

Other testing results:

  • GoW Ascension = no changes compared to master branch
  • GoW 3 = no changes
  • Gran Turismo 6 = broken skybox, screenshots above. (tried all combinations with or without WDB,RDB, WCB,RCB... And CPU Blit) ... Maybe this skybox has too big Z value ?
  • Journey = no changes
  • Killzone 2 = no changes
  • Killzone 3 = no changes
  • Persona 5 = no changes
  • The Last of Us = no changes

@coolllman
Copy link

I had the same, broken skybox GT6 on GTX 1660

@kd-11 kd-11 marked this pull request as draft December 21, 2021 08:54
- If near and far clip are too close together, the API will not distinguish between them leading to out of bounds values
@kd-11
Copy link
Contributor Author

kd-11 commented Dec 21, 2021

GT6 should be ok now. Please check for other regressions.

@Linear524
Copy link

GT6 should be ok now. Please check for other regressions.

Will test GT6 and other titles tonight :)

( P.S. - GT6 have some serious issues with hardware blitting, and only CPU blit currently working without constant flashing and particle bugs and only without WCB/WDB/RCB/RDB usage... MLAA antialiasing is broken too as well as bloom postprocessing. )

@coolllman
Copy link

Gt6 fixed! In Uncharted 3 strange stutters, in master all ok

@kd-11
Copy link
Contributor Author

kd-11 commented Dec 21, 2021

Gt6 fixed! In Uncharted 3 strange stutters, in master all ok

It would be helpful to get a short video clip demonstrating the problem.

@Linear524
Copy link

Confirm - GT6 skybox is fixed. All other titles I have shows no regression (^_^)

@coolllman
Copy link

Gt6 fixed! In Uncharted 3 strange stutters, in master all ok

It would be helpful to get a short video clip demonstrating the problem.
Ok, i will try tonight

@coolllman
Copy link

coolllman commented Dec 22, 2021

base-profile-20211222-18315901-online-video-cuttercom-1_Uk8jol60.online-video-cutter.com.2.mp4

Gt6 stutters too

@coolllman
Copy link

In master all good, test exe in master folder rpcs3, settings same

@Darkhost1999
Copy link
Contributor

I don't see anything different in that 8 second clip than just how the game has performed for me since I got it.
Essentially saying that looks exactly the same as how the game performs on master. Can you edit and provide a comparison?

@coolllman
Copy link

No, not the same, 3 stutters in video, in master all clean, video upload limit 10mb big problem

@kd-11
Copy link
Contributor Author

kd-11 commented Dec 22, 2021

Unrelated, this code change changes shader accuracy and has nothing to do with rendering pipeline or performance. I thought the report was a regression causing graphical glitches but if its just performance stutters I'm confident it is caused by something else.

@kd-11 kd-11 marked this pull request as ready for review December 22, 2021 17:06
@kd-11 kd-11 changed the title [TESTERS NEEDED] rsx: Remove zclip hackery rsx: Remove zclip hackery Dec 22, 2021
@kd-11 kd-11 merged commit 56dd09f into RPCS3:master Dec 22, 2021
@coolllman
Copy link

Unrelated, this code change changes shader accuracy and has nothing to do with rendering pipeline or performance. I thought the report was a regression causing graphical glitches but if its just performance stutters I'm confident it is caused by something else.

I don't know why, but test all settings and this pr and Thread Scheduler: RPCS3 Alternative Sheduler give stutters, in master all good

@coolllman
Copy link

I go to Operation System scheduler, test with Alternative please

@coolllman
Copy link

Screenshot_20211223-021423_VNC Viewer
Cod World at War black skybox too

@kd-11
Copy link
Contributor Author

kd-11 commented Dec 22, 2021

The PR is already merged. Please open a regression ticket and provide RSX captures for analysis.

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.

Deadly Premonition: The Director's Cut | Flickering Shadows
8 participants