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

Check for closest render-to-tex, ignore high bits of fb addr #6383

Merged
merged 4 commits into from
Jun 21, 2014

Conversation

unknownbrackets
Copy link
Collaborator

Tested on a real PSP:

sceGuDepthBuffer((void *)((intptr_t)dbp0 | 0x99000000), 512);
sceGuDrawBuffer(GU_PSM_8888, (void *)((intptr_t)fbp0 | 0x99000000), 512);

Still draws exactly the same place. Tried a couple other combinations to be sure. I think we should just ignore the high bits entirely.

@solarmystic does this (specifically the closest framebuffer) help Breath of Fire 3?

-[Unknown]

There could be multiple, or even an exact match too.  Make sure to take
the closest one.
And depth buffer.  On a PSP, these bits don't affect where it renders to
even slightly.
@solarmystic
Copy link
Contributor

@unknownbrackets Yes, the flickering in BOF3 is resolved by that specific commit.

But the "halved framerate" issue re-introduced by 0b9db17 still remains.

Last completely unaffected build is still 822f936 , just for the record.

@unknownbrackets
Copy link
Collaborator Author

Well, the halved framerate should've caused both issues. The second change just made it not wrap anymore on the framebuffer (incorrectly anyway.) This just meant it was selecting the wrong framebuffer.

Can you upload a debug log (even a few frames while the framerate is halved)?

-[Unknown]

@solarmystic
Copy link
Contributor

@unknownbrackets Debug log (zipped for size):- https://www.mediafire.com/?aelg3g81k2cfnac

@unknownbrackets
Copy link
Collaborator Author

How about this?

-[Unknown]

@solarmystic
Copy link
Contributor

@unknownbrackets Perfection. No issues remain with the game now. No more flickering and it's full speed ahead.

@hrydgard
Copy link
Owner

Makes sense, thanks for testing it!

hrydgard added a commit that referenced this pull request Jun 21, 2014
Check for closest render-to-tex, ignore high bits of fb addr
@hrydgard hrydgard merged commit 5f57e7e into hrydgard:master Jun 21, 2014
@unknownbrackets unknownbrackets deleted the gpu-minor branch June 21, 2014 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants