Re: [patch 1/1] [PATCH] include storage keys in hibernation image.

From: Martin Schwidefsky
Date: Wed Aug 17 2011 - 03:44:09 EST


On Tue, 16 Aug 2011 19:56:47 +0200
"Rafael J. Wysocki" <rjw@xxxxxxx> wrote:

> On Tuesday, August 16, 2011, Martin Schwidefsky wrote:
> > From: Martin Schwidefsky <schwidefsky@xxxxxxxxxx>
> >
> > For s390 there is one additional byte associated with each page,
> > the storage key. This byte contains the referenced and changed
> > bits and needs to be included into the hibernation image.
> > If the storage keys are not restored to their previous state all
> > original pages would appear to be dirty. This can cause
> > inconsistencies e.g. with read-only filesystems.
> >
> > Cc: Pavel Machek <pavel@xxxxxx>
> > Cc: Rafael J. Wysocki <rjw@xxxxxxx>
> > Cc: Jiri Slaby <jslaby@xxxxxxx>
> > Cc: linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx
> > Cc: linux-kernel@xxxxxxxxxxxxxxx
> > Cc: linux-s390@xxxxxxxxxxxxxxx
> > Signed-off-by: Martin Schwidefsky <schwidefsky@xxxxxxxxxx>
>
> OK, I don't have any complaints. Do you want me to take this
> patch or do you want to push it through the s390 tree?

The patch affects the common power management code. At minimum I
would like an acked-by to take it into the s390 tree. I would feel
more comfortable if you'd take it via the power management tree
though.

--
blue skies,
Martin.

"Reality continues to ruin my life." - Calvin.

--
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/