Yep, recreated the weird bug you mentioned there. So, for now, MSU1 music only works again when you reload the ROM, just no 'snapshotting'.

 

Dayumn. By the way, maybe it's normal/expected, if you snapshot when inside a FMV and load that file, you return to a green screen (or sometimes a full crash, like in Super Road Blaster).

 

damn, hope u guys figure it out

 

Hello! Surely many know this but I would like to comment on it in the hope that we will find a solution as a team.

I have used this emulator for many years and have a problem with the sound in many games, like "super mario rpg" and "chrono trigger" when the menu is opened, in other games it happens randomly as in "zelda a link to the Past "in dungeons more often, there is a distortion in sound when playing a time and more often with melodies that are constantly repeated.

In other games the problem is more frequent as in "terranigma" and "illusion of time".

A similar problem happens in the gameboy emulator "vbagx"

These games are examples, there are others where the problem is almost non-existent as in the series megaman x.

In "retroarch" this problem does not happen but the idea is that snes9x is as perfect as possible.

Many contributors who have participated before like Zopenko, Askot and Burnt Lasagna would be interested in this purpose and of course to the maximum responsible of this creation, tantric.

Many thanks for bringing the super nintendo to the wii.

 

That's essentially an independent re-implementation of this fix I already made and completely forgot about. Derp The real issue was that when you request samples from the MSU-1 and it doesn't have any (song not playing, maybe other reasons I can't think of), it should still return the requested buffer size, padded with 0's. It was just returning the available samples, which in some cases was nothing at all. The rest of the changes in that commit ended up not being real issues after all, like increasing the counter variable sizes and shrinking the counter values by a factor of 10, since I was worried about potential counter overflows, but that really wasn't an issue. The relevant change in both is at line 357 or 425, depending on which version you're looking at.

Bad:

Code:

        else
          {
              MSU1.MSU1_STATUS &= ~(AudioPlaying | AudioRepeating);
              return;
          }

Good:

Code:

        else
          {
              MSU1.MSU1_STATUS &= ~(AudioPlaying | AudioRepeating);
              partial_samples -= 3204;
              *(bufPos++) = 0;
          }

As far as the snapshot issues, I'm almost positive it's in the GUI code somewhere, but I don't really have time to look into it. You're right that the issue happens when you press one of the buttons. It's completely unloading the MSU-1 related files, which causes the next MSU-1 call to fail and drop back to SPC fallback. If there's anybody more familiar with the GUI code, that's where you should be looking. Trace through the callbacks for those buttons and see if you can find where it's closing files or releasing open file handles. Once we find the offending code, it's probably an incredibly simple fix. I'm just not very familiar with the code, and unfortunately, runtime debugging is not a simple task.

(also, just a note, if you want to get my attention, I don't actively follow this thread at all, but @ mentioning me sends me email notifications, so if I don't respond, try that)

 

Right now, we're trying to track down one remaining issue (fast-forward causes a desync), at which point you should probably just drop in the current main-line versions of msu1.h/cpp instead of trying to cherry-pick changes. I also just fixed a fairly significant memory leak upstream just now. I'll try to pull it down, along with cherry-picking any other changes that have been made to apu.cpp, since those can't just be dropped in due to GX using the older APU.

 

Grab it from my repo: https://github.com/qwertymodo/snes9xgx/tree/msu1 just give me a bit, something's broken and the Wiimote disconnects when I try to open a folder. I'm running a bisect now to try and figure out where the problem is. I also added .bps patch support just now, so you'll want to grab that. Right now, the relevant changes are in apu.cpp, memmap.cpp, and msu1.h/cpp

 

Ok, I pushed 2 commits. There's still weird glitches in the audio that actually sounds like the channels are flipping. Weird, but at least now it's basically synced with upstream Snes9x, so further updates should be easier.

 

Cool, thanks for the feedback. Just need to figure out snapshots now.

Sent from my m8wl using Tapatalk

 

I noticed this when playing on a mono CRT, but mostly on the Turtles in Time MSU-1 game (right at the 1st level).

@qwertymodo I've finally bit the bullet and tried your Chrono Trigger MSU-1 patch. Noticed some minor pops here and there at the intro gameplay demo, right after the pendulum swing. When the pendulum scene comes again, skips were a bit more noticeable. Would this be affected whether i use the SD card or USB? I'll try to make some other tests this afternoon.

EDIT: though there were a little bit less skips when using USB, they were still noticeable. It seems to happen consistently after at least 1 full minute of MSU music playing. Again, i used Chrono Trigger for this round of tests. Just to clarify, this behaviour is the same as before the new cpp/h patches.