public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: Nigel Cunningham <ncunningham@linuxmail.org>
Cc: Linux-pm mailing list <linux-pm@lists.osdl.org>,
	Pavel Machek <pavel@ucw.cz>
Subject: Re: swsusp3: system suspend/resume from userspace
Date: Thu, 1 Sep 2005 17:11:06 +0200	[thread overview]
Message-ID: <20050901151105.GA1561@openzaurus.ucw.cz> (raw)
In-Reply-To: <1125575747.4785.20.camel@localhost>

[-- Attachment #1: Type: text/plain, Size: 1332 bytes --]

Hi!

> > This is preview of what I'm doing. I call it swsusp3 for the
> > moment. Idea is to push image reading and writing to userspace, so it
> > can be compressed, can have nice progress bar etc.
> > 
> > In the current state I do not write image anywhere, and user <->
> > kernel interface is *very* ugly, but you get the idea. It should allow
> > us to do most of stuff suspend2 does _and_ shrink kernel a bit at the
> > same time.
> > 
> > Warning: Preview only. Code is ugly and probably broken.
> 
> Ummm... it's really hard to know what to say. I guess I'll just have to
> wait and see how you progress.
> 
> A few general comments:
> 
> - I don't see any indication of support for storing a complete image of memory. Planning it?

I was afraid you would ask. This is the area where I don't know how to
do it cleanly. I could only free as much memory as strictly neccessary;
that should leave pagecache "mostly intact". Perhaps that would be good enough?

> - Aren't new syscalls generally viewed as pretty undesirable?

Well, yes, they are, except when alternatives are worse.

> - Should comments be in English?

Is there some czech comment in there? Well code is very ugly at places, "first
make it work, then make it cleaner".

				Pavel

-- 
64 bytes from 195.113.31.123: icmp_seq=28 ttl=51 time=448769.1 ms         


[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



  reply	other threads:[~2005-09-01 15:11 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-09-01 11:05 swsusp3: system suspend/resume from userspace Pavel Machek
2005-09-01 11:55 ` Nigel Cunningham
2005-09-01 15:11   ` Pavel Machek [this message]
2005-09-02  4:54   ` Greg KH
  -- strict thread matches above, loose matches on Subject: below --
2005-09-01 18:49 Brown, Len
2005-09-01 20:17 ` Pavel Machek
2005-09-01 21:13   ` Nigel Cunningham
2005-09-20 23:41   ` Benjamin Herrenschmidt
2005-09-21 10:18     ` Pavel Machek
2005-09-21 10:32       ` Benjamin Herrenschmidt
2005-09-21 12:45         ` Pavel Machek
2005-09-21 12:53           ` Greg KH

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20050901151105.GA1561@openzaurus.ucw.cz \
    --to=pavel@ucw.cz \
    --cc=linux-pm@lists.osdl.org \
    --cc=ncunningham@linuxmail.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox