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: Discard color mask writes with reserved bits #14545

Merged
merged 1 commit into from Aug 25, 2023
Merged

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Aug 24, 2023

Testcase: https://github.com/elad335/myps3tests/tree/master/rsx_tests/write%20output%20mask
This testcase tests what happens on a write to ones to reserved of color mask register
It does this by iterating and ORis valid masks with a single reserved bit at a time, for completeness it also tests it with valid bit masks as well and examines buffer state.
Results: writes with reserved bits set are being ignored by RSX.

Fixes the colors in #13101.
Fixes #10530.
Fixes #10533.

image

@Ordinary205
Copy link
Contributor

Ordinary205 commented Aug 24, 2023

This causes corrupted UI menus graphics on Split/Second.

Before:
Split second non

After:
Split second regression

RPCS3.log.gz

@Ordinary205
Copy link
Contributor

Ordinary205 commented Aug 24, 2023

Apparently this causes a huge regression to almost every game, including Demon's Souls.
Demons souls broken graphics

RPCS3.log.gz

@elad335
Copy link
Contributor Author

elad335 commented Aug 24, 2023

Retest.

@Ordinary205
Copy link
Contributor

The graphics now renders properly.
Demons souls proper graphics
Split second proper graphics
RPCS3.log.gz

@ReLyf47
Copy link

ReLyf47 commented Aug 24, 2023

B4
image

After
image

Also apparently fix the issue of my livery being full black in Nascar: Inside Line

Edit1: I hope i fixed the image not loading

@YuriNator557
Copy link

Fixes #10530
image

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.

Go! Sports Ski™ [NPUA80070] yellow graphics issue UP [BLES00543] blue graphics issue
4 participants