Skip to content

[BUG]: SDK reports old endpoints after PC has resumed from sleep #585

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

Open
Psychlist1972 opened this issue Mar 4, 2025 · 1 comment
Open
Assignees
Labels
bug 🐞 Something isn't working needs-investigation 🔍 Needs to be investigated before considering or solving.

Comments

@Psychlist1972
Copy link
Collaborator

Windows Version

Multiple

Service Installation Method

GitHub

SDK version, if appropriate

All recent

Location

Non-Microsoft application using Windows MIDI Services SDK (MIDI 2.0)

Type of bug

Other or Unsure (please indicate in additional notes section below)

Steps to reproduce

Unsure on steps. Needs investigation.

Reported by Giel Bremmers on Discord here
https://discord.com/channels/980245825202552942/1345860163902640290

"BTW: I sometimes see old endpoints in midi.exe (and MultitrackStudio). I have to restart the service to get up-date endpoints. It now happened after the computer had been sleeping for 12 hours, don't know it this has to do with it."

Expected behavior

No response

Additional notes

Likely related to #483

@Psychlist1972 Psychlist1972 added bug 🐞 Something isn't working needs triage 🚑 Bug needs further investigation and triage labels Mar 4, 2025
@Psychlist1972 Psychlist1972 self-assigned this Mar 4, 2025
@Psychlist1972 Psychlist1972 added needs-investigation 🔍 Needs to be investigated before considering or solving. and removed needs triage 🚑 Bug needs further investigation and triage labels Mar 4, 2025
@Psychlist1972
Copy link
Collaborator Author

This is a good bug to find, as it will presumably be common, and the user has no reasonable way around this. It's unfortunate that our device add/remove stack doesn't handle this automatically on resume (I haven't verified this myself, just going by the report).

Note to self: look into implementing a call to PowerRegisterSuspendResumeNotification https://learn.microsoft.com/en-us/windows/win32/api/powerbase/nf-powerbase-powerregistersuspendresumenotification and possibly re-enumerating hardware devices. This will be a fairly big lift because the code is currently centered around the device add/remove/update notifications, so not sure when this will be in place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working needs-investigation 🔍 Needs to be investigated before considering or solving.
Projects
Status: No status
Development

No branches or pull requests

1 participant