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

Fix glitched corner in Nickelodeon Unite. #7549

Merged
merged 1 commit into from Nov 8, 2018

Conversation

JMC47
Copy link
Contributor

@JMC47 JMC47 commented Nov 7, 2018

Nicktoon's Unite (also known as Spongebob and Friends Unite) has a black box over the upper left corner of the screen with Store EFB Copies to Texture Only Enabled

gnox78-1

The game also breaks with Defer EFB Copies, so, I've disabled both.

defer

@JMC47
Copy link
Contributor Author

JMC47 commented Nov 7, 2018

@stenzek please review

Copy link
Contributor

@stenzek stenzek left a comment

Choose a reason for hiding this comment

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

Still wondering how the game is synchronizing with the GPU, if at all, but for now this seems fine. Maybe it is just a misbehaving title.

@stenzek stenzek merged commit 80259b0 into dolphin-emu:master Nov 8, 2018
@mimimi085181
Copy link
Contributor

Is it too early to talk about optimisation of Defer EFB Copies? As i understand it, in single core mode, Dolphin runs the cpu thread for x cycles and then it runs the gpu thread for y commands/cycles/whatever. Shouldn't it be safe to defer the efb copies on single core mode until just before it switches back to the cpu thread? So something like Defer EFB Copies Light for single core only.

Is there any possiblity that this issue is not a wrong read from the cpu, but a change to the graphics object before efb2ram? So the efb copy is created, efb2ram is delayed, the object is changed, and the deferred copy process is made after that change, is that a possibility?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants