diff options
| author | Vitaly Kuznetsov <vkuznets@redhat.com> | 2015-06-24 16:02:06 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2015-06-24 23:48:44 +0200 |
| commit | 8c506608c3b2eac939fa244bcdea5faa31818f95 (patch) | |
| tree | 1fa7751072f64fb495333f73d363416d20a8aef7 | |
| parent | ffa64eff956a25548cad0391dbc14c672827be7b (diff) | |
| download | linux-stable-8c506608c3b2eac939fa244bcdea5faa31818f95.tar.gz linux-stable-8c506608c3b2eac939fa244bcdea5faa31818f95.tar.bz2 linux-stable-8c506608c3b2eac939fa244bcdea5faa31818f95.zip | |
PM / hibernate: re-enable nonboot cpus on disable_nonboot_cpus() failure
When disable_nonboot_cpus() fails on some cpu it doesn't bring back all
cpus it managed to offline, a consequent call to enable_nonboot_cpus() is
expected. In hibernation_platform_enter() we don't call
enable_nonboot_cpus() on error so cpus stay offlined.
create_image() and resume_target_kernel() functions handle
disable_nonboot_cpus() faults correctly, hibernation_platform_enter()
is the only one which is doing it wrong.
Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
| -rw-r--r-- | kernel/power/hibernate.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/kernel/power/hibernate.c b/kernel/power/hibernate.c index 2329daae5255..690f78f210f2 100644 --- a/kernel/power/hibernate.c +++ b/kernel/power/hibernate.c @@ -552,7 +552,7 @@ int hibernation_platform_enter(void) error = disable_nonboot_cpus(); if (error) - goto Platform_finish; + goto Enable_cpus; local_irq_disable(); syscore_suspend(); @@ -568,6 +568,8 @@ int hibernation_platform_enter(void) Power_up: syscore_resume(); local_irq_enable(); + + Enable_cpus: enable_nonboot_cpus(); Platform_finish: |