Hi!

> <mochel@osdl.org> (03/09/09 1.1217.3.31)
>    [power] Revert swsusp to 2.6.0-test3 state.
>    
>    - From Pavel (mostly, though with some fixups).
>    - Note that I would never publically admit to putting such code into the 
>      kernel. 
>    - Someone ought to really review this patch some day. 

@@ -6,11 +6,12 @@
 #include <asm/desc.h>
 #include <asm/i387.h>

-static inline void
+static inline int
 arch_prepare_suspend(void)
 {
        if (!cpu_has_pse)
-               panic("pse required");
+               return -EPERM;
+       return 0;
 }

 /* image of the saved processor state */

You still want to do some printk(), otherwise we'll get very ugly bug
reports ("it does not suspend and I don't know why"). Also without
check in swsusp.c, this could corrupt data (page table accessed bits
might flip something during copy).

--- /usr/src/tmp/linux/kernel/power/swsusp.c	2003-09-10 19:58:08.000000000 +0200
+++ /usr/src/linux/kernel/power/swsusp.c	2003-09-10 19:57:16.000000000 +0200
@@ -697,7 +697,8 @@
 static void do_software_suspend(void)
 {
 	printk("Doing software_suspend()\n");
-	arch_prepare_suspend();
+	if (arch_prepare_suspend())
+		return;
 	if (pm_prepare_console())
 		printk( "%sCan't allocate a console... proceeding\n", name_suspend);
 	if (!prepare_suspend_processes()) {


Minor nits:

--- linux-2.5-virgin/kernel/power/pmdisk.c      Wed Dec 31 16:00:00 1969
+++ linux-2.5-power/kernel/power/pmdisk.c       Tue Sep  9 13:54:33 2003
@@ -0,0 +1,942 @@
+/*
+ * kernel/power/pmdisk.c - Suspend-to-disk implmentation
                                                ~ missing 'e'


+                               /* we ignore all swap devices that are
not the resume_file */
+                               if (1) {
+// FIXME                               if(resume_device ==
swap_info[i].swap_device) {
+                                       swapfile_used[i] =
SWAPFILE_SUSPEND;

If you want this FIXME fixed, I got patch from Aristeu Sergio Rozanski
Filho <aris@cathedrallabs.org>. Not yet tested, that's why I was
holding it. (attached).

								Pavel
-- 
When do you have a heart between your knees?
[Johanka's followup: and *two* hearts?]