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-tc: propagate texture shuffle format on readback #4887

Merged
merged 1 commit into from
Oct 9, 2021

Conversation

tadanokojin
Copy link
Member

@tadanokojin tadanokojin commented Oct 8, 2021

Description of Changes

Read back PSMCT32 when the texture being blit is the result of a channel effect. This is not exactly right but there's no harm in having the extra data in cpu memory since it's supposed to be there anyway. The only issue is it gets unscaled.

fixes #1326
fixes #2732

Rationale behind Changes

Fixes silent hill flashlight.
image

Suggested Testing Steps

If you plan on testing silent hill, the game still needs fast texture invalidation.

@prafullpcsx2
Copy link
Contributor

Great work. This also fixes Silent Hill shattered memories issue with flashlight and also shadows.

@MrCK1
Copy link
Member

MrCK1 commented Oct 8, 2021

Ran through a bunch of games, didn't notice any new bugs/changes.

@refractionpcsx2
Copy link
Member

Wasn't sure what games this might fix (can't think of any with broken lighting/shadows off the top of my head), but I tested a bunch of games I know do GS downloads and they were all still peachy (except Star Wars Racer which is still broken, but probably outside the scope of this PR since it's depth download).

so yeah, couldn't find anything else fixed I'm afraid, but also nothing broken, so LGTM

@lightningterror lightningterror changed the title tc: propagate texture shuffle format on readback gs-tc: propagate texture shuffle format on readback Oct 8, 2021
@lightningterror lightningterror added this to the Release 1.8 milestone Oct 8, 2021
@lightningterror
Copy link
Contributor

Would be nice to adjust the commit prefix to gs-tc:

Copy link
Contributor

@lightningterror lightningterror left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ran through the dump list which trigger the code, seemed fine.

@tadanokojin tadanokojin merged commit ef2c7ac into PCSX2:master Oct 9, 2021
@tadanokojin tadanokojin deleted the silent-hill branch October 9, 2021 01:43
inactive123 added a commit to libretro/LRPS2 that referenced this pull request Oct 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants