On Fri, Aug 03, 2018 at 01:57:05PM +0100, Jon Hunter wrote: > For soundcards that have several DAI links and many DAPM widgets the > time taken for snd_soc_suspend to execute has been observed to be > several milliseconds. The time is largely spent executing > dapm_power_widgets() for each for the DAI links that need to be > suspended. Given that dapm_power_widgets() is called again after > suspending/resuming the DAI links, one way to optimise the > suspend/resume time is to avoid calling dapm_power_widgets() for > each DAI link and reduces the suspend time significantly. It's a bit alarming that dapm_power_widgets() is taking substantial enough time to be worth worring about - it's *supposed* to be relatively well optimized so it's effectively free. It'll never be quite free, but close enough. The goal is that if nothing changes we end up testing a few nodes at most before we figure out that nothing changed state and stop. Do you have any idea what it's spending its time on, we do call it a lot so if there's any optimization opportunties there we can probably get a lot of benefit out of them. One thing that it occurs to me might help is to start the suspend process by powering down all the input and output widgets that aren't ignoring suspend in one operation, that should hopefully have the effect of ensuring that most of the system that's going to get powered down does so on the first pass through. > Please note that this has been observed on the Tegra210 Jetson TX1 > platform which is not currently supported in the mainline for audio > but has been tested with out-of-tree patches to enable I2S audio. If someone could get the platform booting reliably in -next that'd be good but that's a separate issue... > In the resume path, it is not clear if there could be any issues from > not sync'ing the DAPM power state until after unmuting and resuming > the CPU DAI drivers, because this will happen later with this change. This is a definite problem, we want to have the audio path powered up before we start playing audio otherwise we loose the start of the audio which may be important.
Attachment:
signature.asc
Description: PGP signature
