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

From: Martin Schwidefsky
Date: Fri Jul 08 2011 - 04:29:45 EST


On Thu, 7 Jul 2011 23:36:25 +0200
"Rafael J. Wysocki" <rjw@xxxxxxx> wrote:

> On Wednesday, June 15, 2011, Martin Schwidefsky wrote:
> > On Tue, 14 Jun 2011 22:50:14 +0200
> > "Rafael J. Wysocki" <rjw@xxxxxxx> wrote:
> ...
> >
> > Well before the preallocation kicked in we don't know which are the relevant
> > storage keys. The only other option would be to store all of them which I
> > consider to be a inferior solution.
>
> I've been thinking about that recently a bit.
>
> Don't we need to restore the keys of the page frames that aren't copied
> into the image too?

Pages that are free (= not part of the image) will get a zero storage key
once they are allocated again. So no, we do not need to include these
keys into the hibernation image.


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