On Sat, 5 Dec 2009, Linus Torvalds wrote: > > In other words - I'm not pulling this crazy thing. You'd better explain > why it was done that way, when we already have done the same things better > before in different ways. I get the feeling that all the crazy infrastructure was due to worrying about the suspend/resume topology. But the reason we don't worry about that during init is that it doesn't really tend to matter. Most slow operations are the things that aren't topology-aware, ie things like spinning up/down disks etc, that really could be done as a separate phase instead. For example, is there really any reason why resume doesn't look exactly like the init sequence? Drivers that do slow things can start async work to do them, and then at the end of the resume sequence we just do a "wait for all the async work", exactly like we do for the current init sequences. And yes, for the suspend sequence we obviously need to do any async work (and wait for it) before we actually shut down the controllers, but that would be _way_ more natural to do by just introducing a "pre-suspend" hook that walks the device tree and does any async stuff. And then just wait for the async stuff to finish before doing the suspend, and perhaps again before doing late_suspend (maybe somebody wants to do async stuff at the second stage too). Then, because we need a way to undo things if things go wrong in the middle (and because it's also nice to be symmetric), we'd probably want to introduce that kind of "post_resume()" callback that allows you have a separate async wakeup thing for resume time too. What are actually the expensive/slow things during suspend/resume? Am I wrong when I say it's things like disk spinup/spindown (and USB discovery, which needs USB-level support anyway, since it can involve devices that we didn't even know about before discovery started). Linus -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
