Emulator Issues #10636
closedDolphin treats 2 GiB SD cards as SDHC
0%
Description
Game Name?
Super Smash Bros. Brawl
Game ID? (right click the game in the game list, properties, info tab)
RSBE01
MD5 Hash? (right click the game in the game list, properties, info tab, MD5 Hash: Compute)
Wiimms ISO Tools edited ISO, works fine on both Dolphin and real Wii. Only the music files were edited.
What's the problem? Describe what went wrong.
Dolphin treats all SD cards as SDHC. This can be a problem for older SD setups as the SDHC Extension 1.1 (forums.kc-mm.com/index.php?topic=65342.0) is required for SDHC cards. A 2GB virtual SD card (non-SDHC size) will fail to load codes without the SDHC Extension 1.1, which should not be required for non-SDHC cards.
To fix this problem I propose:
- Renaming the current SD implementation to "SDHC"
- Add a separate configuration option for SD emulation (one that doesn't require the SDHC Extension for non-SDHC virtual SD cards) once this problem is fixed
Adding the SDHC Extension 1.1 code to the ini codes is probably a bad idea since many setups already include this code in the virtual SD's .gct file. If we ever needed to choose between SDHC and SD support, I would want SDHC over SD as SDHC has significantly more space to work with.
What steps will reproduce the problem?
Setup:
- Download the "SD Nov 14" version of Smash Bros Infinite here: http://forums.kc-mm.com/Gallery/BrawlView.php?Number=213356
- Insert the contents into a 2GB virtual SD card (non-SDHC size)
- Set Brawl as Default ISO
After initial setup:
- Rename attached RSBE01SDHCcode.gct to RSBE01.gct and replace it with the one in the virtual SD's \codes folder
- Load Gecko OS app
- Select "Launch Game"
- Observe that the character select screen has modifications. This shows that codes can be loaded with the SDHC Extension 1.1 on a non-SDHC virtual SD card.
- Rename attached RSBE01noSDHCcode.gct to RSBE01.gct and replace it with the one in the virtual SD's \codes folder
- Load Gecko OS app
- Select "Launch Game"
- Observe that Brawl codes are incorrectly not applied. This will not occur in Dolphin 5.0-5037 and earlier.
The only difference between RSBE01SDHCcode.gct and RSBE01noSDHCcode.gct is the SDHC Extension 1.1 code.
Is the issue present in the latest development version? For future reference, please also write down the version number of the latest development version.
5.0-5810
Is the issue present in the latest stable version?
No
If the issue isn't present in the latest stable version, which is the first broken version? (You can find the first broken version by bisecting. Windows users can use the tool https://forums.dolphin-emu.org/Thread-green-notice-development-thread-unofficial-dolphin-bisection-tool-for-finding-broken-builds and anyone who is building Dolphin on their own can use git bisect.)
5.0-5044
What are your PC specifications? (CPU, GPU, Operating System, more)
Win 10 Pro x64
Intel Core i7-4702MQ CPU @2.20GHz
GeForce GT 750M
GeForce Game Ready Driver 388.00
Files
Updated by ryanebola16 about 7 years ago
It'd also be nice if someone knows a different method for checking Dolphin's interpretation of SD vs SDHC for the virtual SD card to verify my findings.
Updated by JMC4789 about 7 years ago
- Assignee set to leoetlino
- Priority changed from Normal to High
Damn.
I think it's better to just automatically use SD for 2GB and lower, and SDHC for higher than 2GB.
Updated by ryanebola16 about 7 years ago
That does sound better than a new configuration option. I wonder how the Wii determines if a card is SD or SDHC.
Updated by leoetlino about 7 years ago
- Status changed from New to Accepted
- Regression changed from No to Yes
- Easy changed from No to Yes
I like off-by-one bugs.
Updated by JosJuice about 7 years ago
- Milestone set to Current
- Regression start set to 5.0-5044
Updated by JosJuice about 7 years ago
- Subject changed from Dolphin treats all SD cards as SDHC to Dolphin treats 2 GiB SD cards as SDHC
- Status changed from Accepted to Fix pending
Updated by JosJuice about 7 years ago
- Status changed from Fix pending to Fixed
- Fixed in set to 5.0-5821