Project

General

Profile

Actions

Emulator Issues #9685

closed

Wallace & Gromit in Project Zoo Bypass FIFO reset, Black patch error

Added by harry-tom over 8 years ago. Updated over 8 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
% Done:

0%

Operating system:
N/A
Issue type:
Bug
Milestone:
Regression:
Yes
Relates to usability:
No
Relates to performance:
No
Easy:
No
Relates to maintainability:
No
Regression start:
4.0-5695
Fixed in:
5.0-286

Description

game name
Wallace & Gromit in Project Zoo

game id
GWLE6L

MD5
3cf7f689e268ae505f36545bf1dd5e14

What's the problem?

When I skipping the 1st cutscene on new game my game crashes with the patch i get this error box.
http://imgur.com/J2zX7Fm
or this
http://imgur.com/ia7da8E

but with out the patch i get this with dual core and without
http://imgur.com/D4pb2S9 and http://imgur.com/m25fZQk

What steps will reproduce the problem?

dx11/open gl
start a new game, watch or skip the 1st cutscreen then crash.

Which versions of Dolphin did you test on? Does using an older version of Dolphin solve your issue? If yes, which versions of Dolphin used to work?

4.0.5692 works
4.0.5695 borken
5.0 broken

What are your PC specifications? (CPU, GPU, Operating System, more)

i5 2500k
gtx asus 780oc 3gb
win 10 64bit
16gb ram.

Actions #1

Updated by JMC4789 over 8 years ago

I just bought this game last week, perfect timing for a bug report. Will investigate.

Actions #2

Updated by ZephyrSurfer over 8 years ago

This issue is seemingly simple at first look, however it's brought up a difficult issue.

This change in behavior is because of a merge in the .ini for PAL and NTSC-U game which actually require their own .ini.

Issue *1 (this report):
GWLE6L.ini (old local):
0x8028EF00:dword:0x48000638
GWL.ini (new global):
0x8028EE80:dword:0x48000638

Issue *2:
But the more serious issue is a old regression from the 3.5 era:
https://dolphin-emu.org/download/dev/24e9aedc1420d1947f1a7bf5b57ddbab3c5fb2dd/

There was a regression from the branch 'global-user-directory' which stops the patch from working properly.

Actions #3

Updated by JosJuice over 8 years ago

  • Regression changed from No to Yes
  • Regression start set to 4.0-5695

If it looks like global-user-directory caused a regression, there's a 99% chance that it's a configuration issue, since all that branch does is change where the configuration is read from.

Merging an OnFrame patch into a region-neutral INI is a mistake. I will make a PR to fix it.

Actions #4

Updated by JosJuice over 8 years ago

  • Status changed from New to Fix pending
Actions #5

Updated by ZephyrSurfer over 8 years ago

Just tested it and it works fine.
Should there still be a global ini when a local one is needed instead?

About the global-user-directory issue I brought up it's real. What was happening before that branch was merged was that a + sign before a $ like +$Bypass FIFO reset would automatically enable the patch.

After the branch was merged that feature from patchengine.cpp or somewhere broke so patches have to be enabled by the user manually.

Actions #6

Updated by JosJuice over 8 years ago

Yes, we want to always have a global INI so that region-neutral settings (for instance texture cache or EFB) are used for all regions of the game. That way, regional releases that we aren't aware of will only lack the patch, not all settings. It only applies to the emulation rating in this case, but I do it this way for all games to be consistent.

What you say about patches and the + character makes sense. However, it might be intentional. We want to rely on patches as little as possible, and enabling them automatically is the opposite of that. If you want to discuss that more, please create a separate issue report.

Actions #7

Updated by JosJuice over 8 years ago

  • Status changed from Fix pending to Fixed
  • Fixed in set to 5.0-286
Actions #8

Updated by harry-tom over 8 years ago

Thank you all who helped to get this working again.

Actions

Also available in: Atom PDF