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

Vulkan:Add more Dirty flag #13762

Closed
wants to merge 1 commit into from
Closed

Conversation

sum2012
Copy link
Collaborator

@sum2012 sum2012 commented Dec 13, 2020

Fix #13741
Fix Princess Maker 5 Portable half screen in Vulkan

edit2: Not sure line 839 also need more dirty flag.

@sum2012 sum2012 added the Vulkan label Dec 13, 2020
@sum2012 sum2012 added this to the v1.11.0 milestone Dec 13, 2020
@hrydgard
Copy link
Owner

Interesting, I'm somewhat surprised that line 594 is not enough ... I'll see what I can figure out...

@sum2012
Copy link
Collaborator Author

sum2012 commented Dec 14, 2020

In line 594,Remove DIRTY_TEXTURE_IMAGE | DIRTY_TEXTURE_PARAMS flag,the game still work

@sum2012
Copy link
Collaborator Author

sum2012 commented Dec 20, 2020

Re-test,just need DIRTY_TEXTURE_IMAGE or DIRTY_TEXTURE_PARAMS is okay
Don't need both

@unknownbrackets
Copy link
Collaborator

I feel like this implies something else is not properly dirtying texture params...

-[Unknown]

@sum2012
Copy link
Collaborator Author

sum2012 commented Dec 25, 2020

@unknownbrackets ,@hrydgard said that my frame dump is okay
https://github.com/hrydgard/ppsspp/files/5647911/recording.zip
Maybe problem is more complicated.
It is okay to move this to Milestone v1.12

@hrydgard hrydgard modified the milestones: v1.11.0, v1.12.0 Dec 26, 2020
unknownbrackets added a commit to unknownbrackets/ppsspp that referenced this pull request Feb 14, 2021
This logs the impacts of hrydgard#13762 so we can see why that helps, even though
it doesn't look like a correct fix.
unknownbrackets added a commit to unknownbrackets/ppsspp that referenced this pull request Feb 28, 2021
This logs the impacts of hrydgard#13762 so we can see why that helps, even though
it doesn't look like a correct fix.
@sum2012
Copy link
Collaborator Author

sum2012 commented Feb 28, 2021

Fix in #14233 instead

@sum2012 sum2012 closed this Feb 28, 2021
@unknownbrackets unknownbrackets removed this from the v1.12.0 milestone Feb 28, 2021
unknownbrackets added a commit to unknownbrackets/ppsspp that referenced this pull request Oct 13, 2021
This logs the impacts of hrydgard#13762 so we can see why that helps, even though
it doesn't look like a correct fix.
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.

Princess Maker 5 Portable half screen in Vulkan
3 participants