linux-kernel - Re: [PATCH]: Compress hibernation image with LZO (in-kernel)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1280887121.2741.5.camel@shrek.rexursive.com>
Date:	Wed, 04 Aug 2010 11:58:41 +1000
From:	Bojan Smojver <bojan@...ursive.com>
To:	Nigel Cunningham <nigel@...onice.net>
Cc:	KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH]: Compress hibernation image with LZO (in-kernel)

On Wed, 2010-08-04 at 11:50 +1000, Nigel Cunningham wrote:
> I don't see what all the fuss was about. save_image is called after
> the  snapshot is made (hibernate called hibernation_snapshot to create
> the  image, then swsusp_write which in turn calls save_image), so
> there's no  possibility of the memory allocated by it being included
> in the image or  of a memory leak ocuring as a result. 

OK, thanks for your input.

I think what was in question was whether the snapshot can change after
it was made. I was under the impression that it cannot (hence it is
called snapshot - but don't believe me - I have no idea what I'm talking
about). Kame seems to think that it can, so all routines that are
allocating memory after that point should make sure that pages are
marked to be freed upon resume or explicitly freed on resume by
remembering pointers into global variables (if I understood correctly).

But, if you say that snapshot is just that (i.e. static after beeing
made), that's that then.

PS. I'm polishing the patch a bit in terms of overlapping/in-place
compression/decompression (saves memory). Hopefully I'll have a final
version to post today.

-- 
Bojan

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/