On Wednesday, September 29, 2010, Nigel Cunningham wrote:Hi.
On 30/09/10 07:13, Rafael J. Wysocki wrote:From: Rafael J. Wysocki<rjw@xxxxxxx>
Subject: PM / Hibernate: Modify signature used to mark swap
Since we are adding compression to the kernel's hibernate code,
change signature used by it to mark swap spaces, so that earlier
kernels don't attempt to restore compressed images they cannot
handle.
Signed-off-by: Rafael J. Wysocki<rjw@xxxxxxx>
---
kernel/power/swap.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
Index: linux-2.6/kernel/power/swap.c
===================================================================
--- linux-2.6.orig/kernel/power/swap.c
+++ linux-2.6/kernel/power/swap.c
@@ -29,7 +29,7 @@
#include "power.h"
-#define SWSUSP_SIG "S1SUSPEND"
+#define HIBERNATE_SIG "LINHIB0001"
/*
* The swap map is a data structure used for keeping track of each page
@@ -195,7 +195,7 @@ static int mark_swapfiles(struct swap_ma
if (!memcmp("SWAP-SPACE",swsusp_header->sig, 10) ||
!memcmp("SWAPSPACE2",swsusp_header->sig, 10)) {
memcpy(swsusp_header->orig_sig,swsusp_header->sig, 10);
if no compression
- memcpy(swsusp_header->sig,SWSUSP_SIG, 10);
else
+ memcpy(swsusp_header->sig, HIBERNATE_SIG, 10);
??
I thought about that, but we'll need to drop the old signature when the image
format changes (I think it will after your patch series) anyway. And the
benefit is not really worth it IMO (it only affects people using the in-kernel
hibernation on x86-64).