highmem handling again

From: Pavel Machek
Date: Sun Aug 08 2004 - 14:27:53 EST


Hi!

I agree that swsusp_free is not the right place to restore_highmem(),
but I can't find "right" place to do it... Best I could come up is
with is:

It did not work at the end of swsusp_resume, or at the end of
swsusp_restore, IIRC.

Pavel

--- clean-mm/kernel/power/disk.c 2004-07-28 23:39:49.000000000 +0200
+++ linux-mm/kernel/power/disk.c 2004-08-08 21:11:38.000000000 +0200
@@ -184,8 +187,11 @@
error = power_down(pm_disk_mode);
pr_debug("PM: Power down failed.\n");
}
- } else
+ } else {
+ extern int restore_highmem(void);
+ restore_highmem();
pr_debug("PM: Image restored successfully.\n");
+ }
swsusp_free();
Done:
finish();
--- clean-mm/kernel/power/swsusp.c 2004-07-28 23:39:49.000000000 +0200
+++ linux-mm/kernel/power/swsusp.c 2004-08-08 20:55:59.000000000 +0200
@@ -523,7 +523,7 @@
return 0;
}

-static int restore_highmem(void)
+int restore_highmem(void)
{
while (highmem_copy) {
struct highmem_page *save = highmem_copy;




--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/