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

Star Ocean: Second Evolution: map errors with Vulkan #17386

Closed
4 of 5 tasks
leoxxx opened this issue May 2, 2023 · 19 comments
Closed
4 of 5 tasks

Star Ocean: Second Evolution: map errors with Vulkan #17386

leoxxx opened this issue May 2, 2023 · 19 comments
Labels
GE emulation Backend-independent GPU issues
Milestone

Comments

@leoxxx
Copy link
Contributor

leoxxx commented May 2, 2023

Game or games this happens in

ULJM-05325 Star Ocean: Second Evolution

What area of the game / PPSSPP

As shown in the picture, map error in battle and world map with Vulkan in v1.15. I'm not sure if there are other map errors.
V

www

It's O.K in v1.14.

What should happen

It's O.K. with OpenGL.
O

Logs

No response

Platform

Android

Mobile phone model or graphics card

Helio X20 (MT6797+Mali-T880 MP4) Android Tablet

PPSSPP version affected

v1.15

Last working version

v1.14

Graphics backend (3D API)

Vulkan

Checklist

  • Test in the latest git build in case it's already fixed.
  • Search for other reports of the same issue.
  • Try resetting settings or older versions and include if the issue is related.
  • Try without any cheats and without loading any save states.
  • Include logs or screenshots of issue.
@hrydgard
Copy link
Owner

hrydgard commented May 2, 2023

Interesting! Can you get a GE frame capture ?

@hrydgard hrydgard added this to the v1.15.1 milestone May 2, 2023
@hrydgard hrydgard added the GE emulation Backend-independent GPU issues label May 2, 2023
@leoxxx
Copy link
Contributor Author

leoxxx commented May 2, 2023

Interesting! Can you get a GE frame capture ?

I will try it later. Now the tablet is charging.

@leoxxx leoxxx changed the title Star Ocean: Second Evolution: map error in battle with Vulkan Star Ocean: Second Evolution: map errors in battle with Vulkan May 2, 2023
@leoxxx leoxxx changed the title Star Ocean: Second Evolution: map errors in battle with Vulkan Star Ocean: Second Evolution: map errors with Vulkan May 2, 2023
@leoxxx
Copy link
Contributor Author

leoxxx commented May 2, 2023

@hrydgard
Here is the new map error dump, it's in world map.
1569474716.zip

Here is the battle map error dump.
1034509260.zip

@hrydgard
Copy link
Owner

hrydgard commented May 2, 2023

Thank you @leoxxx, will have a look.

@hrydgard
Copy link
Owner

hrydgard commented May 2, 2023

Both render OK on PC, gonna have to test on some phones..

Both also render fine on Mali G71 (Galaxy S8). Will try on more Malis.

@hrydgard
Copy link
Owner

hrydgard commented May 2, 2023

Both render fine on:

  • Mali G71 (Galaxy S8).
  • Mali G78 (Galaxy S21).

T880 is a fairly old Mali GPU and the drivers are likely not very up to date, so I'm guessing we're hitting some depth-related driver bug. I don't think I'll be able to fix this for 1.15.1 as I might need to get a device, so re-tagging to the potential 1.15.2.

Though, there's one remaining chance to figure it out: Try various builds from https://buildbot.orphis.net/ppsspp/index.php?m=fulllist and see what change broke it. If you start by large steps and go narrower and narrower, you should be able to find what change broke it within 10-11 tries (bisection).

@hrydgard hrydgard modified the milestones: v1.15.1, v1.15.2 May 2, 2023
@leoxxx
Copy link
Contributor Author

leoxxx commented May 3, 2023

v1.14.4-188-g350cb55c6 It's last working version.

v1.14.4-191-g8d3a3285d No bird. Only a shadow.
no bird

v1.14.4-197-g30586be75 Map error is the same as v1.15.

v1.14.4-772-gdc7f461bv1.14.4-790-g938230e7d All game crash when start.

And I found OpenGL also have map error.
no shadow
No shadow under the bird.

1111
Have shadow with Vulkan.

@hrydgard
Copy link
Owner

hrydgard commented May 3, 2023

Thanks, that's useful. Can you explain what you mean by "map error"? I don't see something specifically wrong with the map, what did it look like before?

@hrydgard
Copy link
Owner

hrydgard commented May 3, 2023

Alright, so this is definitely a case of an old GPU driver not handling even pretty simple shader code correctly, these bugs have later been fixed obviously since on newer phones this works correctly.

The two changes are:

The first one we've previously seen similar problems with in Raspberry Pi drivers. Might try to apply the same workaround.

The second one I'm surprised makes a difference.

Can you go into Settings/Tools/System Information and check the exact Vulkan driver version and device name (take a screenshot where it's visible)?

@leoxxx
Copy link
Contributor Author

leoxxx commented May 3, 2023

O.K.
222
I marked the picture.
No shadow with OpenGL.
Have shadow with Vulkan.

@hrydgard
Copy link
Owner

hrydgard commented May 3, 2023

Oh right, I thought you meant by "map" the overlay map on the bottom right:

image

Didn't see anything wrong with that. Now, I need a picture of the driver version and stuff on the System Information screen.

@leoxxx
Copy link
Contributor Author

leoxxx commented May 3, 2023

Alright, so this is definitely a case of an old GPU driver not handling even pretty simple shader code correctly, these bugs have later been fixed obviously since on newer phones this works correctly.

The two changes are:

The first one we've previously seen similar problems with in Raspberry Pi drivers. Might try to apply the same workaround.

The second one I'm surprised makes a difference.

Can you go into Settings/Tools/System Information and check the exact Vulkan driver version and device name (take a screenshot where it's visible)?

#15526 (comment)
You can see them in this issue.

@hrydgard
Copy link
Owner

hrydgard commented May 3, 2023

perfect, thanks for the link.

@hrydgard
Copy link
Owner

hrydgard commented May 3, 2023

Alright, there should be a new build up in maybe half an hour on the buildbot with the above workaround, please test when available.

@leoxxx
Copy link
Contributor Author

leoxxx commented May 3, 2023

v1.15-42-gec10c73a8 It's O.K.
But still no shadow with OpenGL.

@leoxxx leoxxx closed this as completed May 3, 2023
@hrydgard
Copy link
Owner

hrydgard commented May 3, 2023

Cool, thanks for testing!

We can look into OpenGL later. Can you post your system information page the way it looks if you switch to OpenGL? (The information strings are different than in Vulkan).

@hrydgard hrydgard modified the milestones: v1.15.2, v1.15.1 May 3, 2023
@leoxxx
Copy link
Contributor Author

leoxxx commented May 3, 2023

Screenshot_20230503-191501
Screenshot_20230503-191524

@leoxxx
Copy link
Contributor Author

leoxxx commented May 3, 2023

@hrydgard
So, need I open a new issue?

@hrydgard
Copy link
Owner

hrydgard commented May 3, 2023

Thanks!

Yeah, I guess that would make sense. I'm not likely to work on it for a while though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GE emulation Backend-independent GPU issues
Projects
None yet
Development

No branches or pull requests

2 participants