Re: [RFC][PATCH -mm 1/2] swsusp: Remove platform callbacks from restore code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I expect http://bugzilla.kernel.org/show_bug.cgi?id=7887 this bug to
pop again with this patch.

Regards,
Alex.

On 5/9/07, Rafael J. Wysocki <rjw@xxxxxxx> wrote:
From: Rafael J. Wysocki <rjw@xxxxxxx>

The restore part of the hibernation code should not call any 'platform'
functions, since it doesn't carry out any power transition of the system.
Still, such functions are called from the restore code in kernel/power/user.c .
Fix that.

Signed-off-by: Rafael J. Wysocki <rjw@xxxxxxx>
---
 kernel/power/user.c |   17 ++++-------------
 1 file changed, 4 insertions(+), 13 deletions(-)

Index: linux-2.6.21/kernel/power/user.c
===================================================================
--- linux-2.6.21.orig/kernel/power/user.c       2007-05-08 22:10:36.000000000 +0200
+++ linux-2.6.21/kernel/power/user.c    2007-05-08 23:03:17.000000000 +0200
@@ -179,34 +179,25 @@ static inline int snapshot_suspend(int p
        return error;
 }

-static inline int snapshot_restore(int platform_suspend)
+static inline int snapshot_restore(void)
 {
        int error;

        mutex_lock(&pm_mutex);
        pm_prepare_console();
-       if (platform_suspend) {
-               error = platform_prepare();
-               if (error)
-                       goto Finish;
-       }
        suspend_console();
        error = device_suspend(PMSG_PRETHAW);
        if (error)
-               goto Resume_devices;
+               goto Finish;

        error = disable_nonboot_cpus();
        if (!error)
                error = swsusp_resume();

        enable_nonboot_cpus();
- Resume_devices:
-       if (platform_suspend)
-               platform_finish();
-
+ Finish:
        device_resume();
        resume_console();
- Finish:
        pm_restore_console();
        mutex_unlock(&pm_mutex);
        return error;
@@ -272,7 +263,7 @@ static int snapshot_ioctl(struct inode *
                        error = -EPERM;
                        break;
                }
-               error = snapshot_restore(data->platform_suspend);
+               error = snapshot_restore();
                break;

        case SNAPSHOT_FREE:

_______________________________________________
linux-pm mailing list
linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/linux-pm

_______________________________________________
linux-pm mailing list
linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/linux-pm

[Index of Archives]     [Linux ACPI]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [CPU Freq]     [Kernel Newbies]     [Fedora Kernel]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux