ppsspp icon indicating copy to clipboard operation
ppsspp copied to clipboard

Graphical glitches related to save states and minimizing

Open Panderner opened this issue 4 years ago • 40 comments

Here's what's happens a glitches when the black screen effect bug but when i use the save state it changes the screen to Save State's thumbnail.

Here's a Example for enemy encounter effect in Persona 2: Innocent Sin:

Here's what's looks like without using save states or minimizing: ezgif com-gif-maker

and here's a another one using save states or minimizing: ezgif com-gif-maker (1)

Here's a shatter effect in Danganronpa comparison:

With using save states or minimizing: Screenshot_2020-10-10-23-18-09-47_2f85358b2198d26f8aca533d68bee793

Without using save states or minimizing: Screenshot_2020-10-10-23-52-26-39_2f85358b2198d26f8aca533d68bee793

and what save screen screen and pause screen looks like: Screenshot_2020-10-11-01-52-40-47_2f85358b2198d26f8aca533d68bee793 Screenshot_2020-10-11-01-53-12-06_2f85358b2198d26f8aca533d68bee793

YS Seven: ezgif com-gif-maker (2)

Panderner avatar Oct 10 '20 18:10 Panderner

Games affected: Persona 1 and 2 enemy encounter effect Shatter effects, save screens, pause screen in Danganronpa. YS Seven Danball Senki and Danball Senki Boost (#13338) Gundam AGE Universe Accel/Comsic drive (only happens for block transfer effects turned off) Gundam Battle Series (#13749) Hot Shots Shorties (Everybody's Stress Buster) Portable Island screenshots (#14533)

Games affected for graphical artifacts: Sega Rally (#6979) Juiced 2 (#7295) Colin McRae Rally 2005 (#7810) flickering shadows

Panderner avatar Oct 10 '20 18:10 Panderner

Danball Senki and Danball Senki Boost affected this issue: Screenshot_2020-10-12-20-29-19-57_2f85358b2198d26f8aca533d68bee793 Screenshot_2020-10-13-17-50-25-28_2f85358b2198d26f8aca533d68bee793

Panderner avatar Oct 12 '20 12:10 Panderner

I've mentioned #8708, #13079 and #13338 for using these screenshots that are affected this bug that changes pictures using save states and minimizing

Panderner avatar Oct 12 '20 12:10 Panderner

Thanks for the good reports, this is very weird...

hrydgard avatar Oct 12 '20 13:10 hrydgard

Character Talking backgrounds in Gundam AGE Universe Accel and Cosmic Drive (only happens if you disable simulate block transfer effects) Screenshot_2020-10-14-15-56-50-88_2f85358b2198d26f8aca533d68bee793

Panderner avatar Oct 14 '20 08:10 Panderner

Stuff that only happens if you disable block transfers is not really bugs.

We're nearing a state now I think where we can just remove that option, it's not that many games that benefit without breaking anymore..

hrydgard avatar Oct 14 '20 08:10 hrydgard

Stuff that only happens if you disable block transfers is not really bugs.

even I've enabled simulate block transfer effects for Persona 2, enemy encounter effect glitch still happens. including other games like Danganronpa.

Panderner avatar Oct 14 '20 08:10 Panderner

Then that's a real bug.

hrydgard avatar Oct 14 '20 08:10 hrydgard

This bug does not occur on Software Renderer mode. Screenshot_2020-10-14-16-25-18-65_2f85358b2198d26f8aca533d68bee793 Screenshot_2020-10-14-16-29-48-48_2f85358b2198d26f8aca533d68bee793 Screenshot_2020-10-14-16-30-56-20_2f85358b2198d26f8aca533d68bee793

Panderner avatar Oct 14 '20 08:10 Panderner

These issues are caused by graphical glitches related to framebuffers.

Panderner avatar Oct 25 '20 10:10 Panderner

@hrydgard this also happens on all backends.

Panderner avatar Nov 15 '20 08:11 Panderner

@hrydgard it also happens in all backends can you remove the OpenGL label?

Panderner avatar Dec 07 '20 15:12 Panderner

Regarding this comment https://github.com/hrydgard/ppsspp/issues/13256#issuecomment-691658682 Hot Shots Shorties (Everybody's Stress Buster) it also affected this issue:

Before Minimizing: Screenshot_2020-12-13-22-44-58-421_org ppsspp ppsspp

After Minimizing: Screenshot_2020-12-13-22-40-30-466_org ppsspp ppsspp

Panderner avatar Dec 13 '20 14:12 Panderner

Has this worked in old builds? Possibly back when we had Readback mode in addition to buffered and non-buffered?

At least the Persona 2 issue seems to be texturing from a section of RAM, so this seems to be about the CPU reading back the screen copying it from VRAM to RAM. If you switch tasks or save state, we readback and write the framebuffer to VRAM, which makes it look like it works (kind of - it's still an old frame).

So likely there's a function that needs patching (or we need to implement proper detection of CPU reading from VRAM, which is difficult).

Anyway, postponing to 1.12.

hrydgard avatar Jan 09 '21 11:01 hrydgard

Same happens for older versions of PPSSPP

Panderner avatar Jan 09 '21 11:01 Panderner

Here's what happens when i disable BlockTransferAllowCreateFB for Ys Seven

BlockTransferAllowCreateFB off: ezgif com-gif-maker (1)

BlockTransferAllowCreateFB on: ezgif com-gif-maker (2)

Panderner avatar Jan 18 '21 11:01 Panderner

So the compat hack should be disabled for this title, since it causes issues. It was already written, so just going to paste why rushing to add games to compat (speed)hacks was(is) a bad idea: https://github.com/hrydgard/ppsspp/issues/11669#issuecomment-470393342

LunaMoo avatar Jul 17 '21 16:07 LunaMoo

Thanks for narrowing that down @LunaMoo . I think we'll just disable this speedhack for Ys like you did in your fork.

hrydgard avatar Jul 17 '21 18:07 hrydgard

Here's what happens when i disable BlockTransferAllowCreateFB for Ys Seven

BlockTransferAllowCreateFB off: ezgif com-gif-maker (1)

BlockTransferAllowCreateFB on: ezgif com-gif-maker (2)

@hrydgard please re-enable again BlockTransferAllowCreateFB on this game YS7.

To avoid glitch Panderner reproduce add this game also to ClearToRAM 👍 tried and tested.

ghost avatar Oct 11 '21 02:10 ghost

Enabling again for Ys Seven is bad idea @Gamemulatorer because it will result black screen save icons and BlockTransferAllowCreateFB off gif is a correct one.

Here's a gif for both: BlockTransferAllowCreateFB and ClearToRAM enabled: ezgif com-gif-maker (1)

Panderner avatar Oct 11 '21 03:10 Panderner

Enabling again for Ys Seven is bad idea @Gamemulatorer because it will result black screen save icons and BlockTransferAllowCreateFB off gif is a correct one.

My save state png still correct. Screenshot_2021-10-11-12-00-53

ghost avatar Oct 11 '21 04:10 ghost

Here's a gif for both: BlockTransferAllowCreateFB and ClearToRAM enabled: ezgif com-gif-maker (1)

Can you explain what's wrong in this gif?

ghost avatar Oct 11 '21 04:10 ghost

Here's a gif for both: BlockTransferAllowCreateFB and ClearToRAM enabled: ezgif com-gif-maker (1)

Can you explain what's wrong in this gif?

This gif is incorrect.

And this one is correct: 104910080-c5629880-59c3-11eb-8baf-cb4dfd506542

Panderner avatar Oct 11 '21 04:10 Panderner

Here's a gif for both: BlockTransferAllowCreateFB and ClearToRAM enabled: ezgif com-gif-maker (1)

Can you explain what's wrong in this gif?

This gif is incorrect.

And this one is correct: 104910080-c5629880-59c3-11eb-8baf-cb4dfd506542

Can you explain more on details I didn't get what's wrong and correct o_0

Please elaborate more.

ghost avatar Oct 11 '21 04:10 ghost

As I mentioned in #13975 enabling BlockTransferAllowCreateFB for this game might cause save icons that draw black screen.

Panderner avatar Oct 11 '21 04:10 Panderner

As I mentioned in #13975 enabling BlockTransferAllowCreateFB for this game might cause save icons that draw black screen.

Well as you can see my savestate icon still correct maybe this is adreno gpu bug?

ghost avatar Oct 11 '21 04:10 ghost

Here's a gif using software rendering: ezgif com-gif-maker (3)

Panderner avatar Oct 11 '21 04:10 Panderner

My save icon png still correct Screenshot_2021-10-11-12-23-29

ghost avatar Oct 11 '21 04:10 ghost

Here's a gif using software rendering: ezgif com-gif-maker (3)

Please explain more on details I didn't get your problem with this :/

ghost avatar Oct 11 '21 04:10 ghost

My save icon png still correct Screenshot_2021-10-11-12-23-29

Look at in-game save screen: Screenshot (44)

Panderner avatar Oct 11 '21 04:10 Panderner