diff options
author | Anton Vorontsov <avorontsov@ru.mvista.com> | 2009-10-05 03:44:27 -0400 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2009-10-30 02:20:52 -0400 |
commit | 64eb38a6e9732f45d518b9e522d613195c930e8f (patch) | |
tree | b33676121d0d4b94a7799c8a6226eff41e871464 | |
parent | 7424639af480a05cac428ec7e7e38a11d6ff5734 (diff) |
powerpc: Make it possible to select hibernation on all PowerPCs
Just as with kexec, hibernation may fail even on well-tested platforms:
some PCI device, a driver of which doesn't play well with hibernation,
is enough to break resuming.
Hibernation code is not much platform dependent, and hiding features only
because these were not verified on a particular hardware is
counterproductive: we just prevent the features from being widely tested.
For example, with this patch I just tested hibernation on a MPC83xx
board, and it works quite well, modulo a few drivers that need some
fixing.
So, let's make it possible to select hibernation support for all
PowerPCs, then let's wait for any possible bug reports, and actually fix
(or just collect ;-) the bugs instead of hiding them. If some platforms
really can't stand hibernation, we can make a blacklist, with proper
comments why exactly hibernation doesn't work, whether it is possible to
fix, and what needs to be done to fix it.
CONFIG_HIBERNATION is still =n by default, so the commit doesn't change
anything apart from ability to set it to =y.
I'm not sure if EXPERIMENTAL dependency is needed, I'd rather not add it
for a few reasons:
1) It doesn't matter much, for distro kernels user has no clue that some
feature is experimental. Majority of defconfigs enable EXPERIMENTAL
anyway (90 vs. 4, which, btw, means that EXPERIMENTAL is overused
in Kconfigs);
2) EXPERIMENTAL is a good thing for features that change default
behaviour of a kernel, while for hibernation user has to explicitly
issue 'echo disk > /sys/power/state' to trigger any hibernation bugs;
3) Per init/Kconfig, EXPERIMENTAL is a good thing to scare and discourage
users from 'widespread use of a feature', while we want to encourage
that use.
Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
-rw-r--r-- | arch/powerpc/Kconfig | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig index 2ba14e77296c..c01580d86fdd 100644 --- a/arch/powerpc/Kconfig +++ b/arch/powerpc/Kconfig | |||
@@ -199,19 +199,8 @@ config DEFAULT_UIMAGE | |||
199 | config REDBOOT | 199 | config REDBOOT |
200 | bool | 200 | bool |
201 | 201 | ||
202 | config HIBERNATE_32 | ||
203 | bool | ||
204 | depends on (PPC_PMAC && !SMP) || BROKEN | ||
205 | default y | ||
206 | |||
207 | config HIBERNATE_64 | ||
208 | bool | ||
209 | depends on BROKEN || (PPC_PMAC64 && EXPERIMENTAL) | ||
210 | default y | ||
211 | |||
212 | config ARCH_HIBERNATION_POSSIBLE | 202 | config ARCH_HIBERNATION_POSSIBLE |
213 | bool | 203 | bool |
214 | depends on (PPC64 && HIBERNATE_64) || (PPC32 && HIBERNATE_32) | ||
215 | default y | 204 | default y |
216 | 205 | ||
217 | config ARCH_SUSPEND_POSSIBLE | 206 | config ARCH_SUSPEND_POSSIBLE |