Created attachment 128890 [details] [review] proposed patch When unloading the module-suspend-on-idle, the sinks/sources that were suspended are not resumed anymore. This causes playback/record to not work anymore after the unload. The proposed patch is to manually resume all devices when unloading the module.
The patch resumes all devices even if they are already running, which results in some unwanted log messages (see the code in resume()). Also, a couple of formatting accidents: too small indentation and space between a function's name and its argument list. Otherwise a nice fix, though!
We could make resume conditional on the sink/source current state.
Created attachment 128926 [details] [review] proposed patch
I added an improved patch: - correct indendation - only resume suspended sinks/sources - don't reset the timer if we're going to destroy it later anyway - improved log message
Thanks! Applied.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.