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

Crash during multiple bank import from synth #16

Closed
GriffReborn opened this issue Oct 11, 2020 · 10 comments
Closed

Crash during multiple bank import from synth #16

GriffReborn opened this issue Oct 11, 2020 · 10 comments
Labels
bug Something isn't working

Comments

@GriffReborn
Copy link

Importing from synth fails if you import more than one bank at a time, it hangs at 99% after the first bank. This happens if selecting more than one bank, or using import all button.

Importing individual banks works more reliably, but still occasionally hangs at 99%. If you cancel the import, the data is not there, if you start the import again, the app crashes and closes.

Windows 10 1903 - 18362.1082
MOTU MIDI Express XT with latest drivers.
Virus OS 4.9

@christofmuc
Copy link
Owner

Thank you for the bug report! I have seen the same behavior with my Virus B, but not frequently. I suspect in case a MIDI message goes missing, the app is still waiting like for patch #99 and doesn't accept that the patches are done.

I am using the same MIDI Interface, so that might point me in the right direction.

Certainly there is also another problem in that canceling the download and restarting shouldn't crash, I will look into it!

Again, thank you for reporting!

@christofmuc christofmuc added the bug Something isn't working label Oct 11, 2020
@GriffReborn GriffReborn changed the title Virus B - Crash during import from synth Crash during multiple bank import from synth Oct 11, 2020
@GriffReborn
Copy link
Author

GriffReborn commented Oct 11, 2020

Changed issue description, as I've just had the same thing happen importing from Rev2: Selected import all, and it hung at F3.

@GriffReborn
Copy link
Author

GriffReborn commented Oct 11, 2020

Just happened when importing a single bank from Rev2:

Faulting application name: KnobKraftOrm.exe, version: 0.0.0.0, time stamp: 0x5f818afd
Faulting module name: ntdll.dll, version: 10.0.18362.1049, time stamp: 0xb5beef21
Exception code: 0xc0000005
Fault offset: 0x00000000000072a6
Faulting process ID: 0x278
Faulting application start time: 0x01d69fc69089692b
Faulting application path: C:\Users*REDACTED*\AppData\Local\Programs\KnobKraftOrm\KnobKraftOrm.exe
Faulting module path: C:\WINDOWS\SYSTEM32\ntdll.dll
Report ID: c4873a8b-6bfa-4b0d-8e10-4e5a924b147a
Faulting package full name:
Faulting package-relative application ID:

KnobKraftOrm_6p8TzBoAAF

mmc_DtpHMpK7go

@christofmuc
Copy link
Owner

Is you Rev2 connected via the MIDI Express XT or directly via USB?

@GriffReborn
Copy link
Author

Rev2 is on the XT. Can't get it to work over USB, I get message "Failed to open MIDI device Rev2, maybe it's turned off or used by another software?".

@christofmuc
Copy link
Owner

This suggests that multi-client MIDI is not there yet with Windows USB devices, unless they use the new API (which I think JUCE as a framework doesn't): https://www.gearslutz.com/board/music-computers/1214965-microsofts-native-midi-driver.html

Do you have a DAW or MIDI OX open which could block access to the USB for the Orm?

@GriffReborn
Copy link
Author

No, no other MIDI apps open. And I set everything in GLOBAL from MIDI to USB.

@christofmuc
Copy link
Owner

Rev2 is on the XT. Can't get it to work over USB, I get message "Failed to open MIDI device Rev2, maybe it's turned off or used by another software?".

That might be another bug, for me USB works out of the box as expected.

@christofmuc
Copy link
Owner

Closing this, as release 1.7.3 contains multiple stability bug fixes. Those were around cancel-behavior, however. The reason the download gets stuck is unclear to me, the crashes I could reproduce e.g. by switching off the MidiExpress in the middle of a download were due to cleanup problems after cancel. I have yet to see it crash during a normal download.

What I have seen is that most synths don't like it very much when other MIDI messages are sent while a bigger sysex transfer is under way, e.g. keep playing while the download happens. And if a single message goes missing, the bank dump will not complete because it will simply count the patch messages and wait until it got enough.

@christofmuc
Copy link
Owner

I added a new issue for the Rev2 problem - need to test it again after I changed a lot in the USB code:
#30

The Virus hanging at least was also reported now for the Virus C, so I opened a fresh ticket for that:
#31

The crashes should be fixed by now, I have not seen any new crashes in that area.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants