Hi! > > OK > > > >> Here is an alternative implementation of the patch. My test machine is > >> currently unavailable, so it is not yet been tested. How does this one look? > > > > Well, I'd like to do that cleanly from the start. > > > > Now, the problem is that PM_SLEEP_SMP selects HOTPLUG_CPU, because > > that's necessary for the other architectures to make SMP suspend work, but it's > > not necessary on your architecture. Moreover, you don't need to compile > > enable_nonboot_cpus() at all. > At least for the architecture I am enabling this support for > (PPC_PSERIES), upon looking closer, it looks like PM_SLEEP_SMP was > never defined, so enable_nonboot_cpus and disable_nonboot_cpus were > always nooped before, which I didn't previously realize. We probably > want to retain this behavior. > (Please wrap at column 80) This patch is already way better than the original one, but... Why do you want enable/disable_nonboot_cpus to be noped out? Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html _______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm
