public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Nigel Cunningham <ncunningham@clear.net.nz>
To: Pavel Machek <pavel@suse.cz>
Cc: Linus Torvalds <torvalds@transmeta.com>,
	swsusp-devel <swsusp-devel@lists.sourceforge.net>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Thoughts wanted on merging Software Suspend enhancements
Date: Sun, 13 Jul 2003 02:55:23 +1200	[thread overview]
Message-ID: <1058021722.1687.16.camel@laptop-linux> (raw)
In-Reply-To: <20030712140057.GC284@elf.ucw.cz>

Don't ask me why I'm awake at 3am. I don't know.

On Sun, 2003-07-13 at 02:00, Pavel Machek wrote:
> Hi!
> 
> > Hi Linus.
> >
> 
> Well, you probably did want Linus to answer, but I see some new stuff
> here.

Yeah, I did. But it wasn't going to happen :>

> > - save a full image rather than freeing just about all the memory first
> > - highmem support
> > - image compression support
> > - swapfile support in progress
> 
> This seems extremely hard to do right. If you can do it right and not
> rewrite half of kernel, that's okay, but I don't think you can do that.

I assume you mean swapfile only. The first two have been working for a
while now. For swapfile support, my approach is simple. I'll store the
info rw_swap_page_base calculates and save it in the pagedir, using it
to read back the data at resume without requiring any knowledge of the
filesystem. (Of course I'm assuming the data isn't compressed etc but
since swapfiles work, this seems to be a reasonable assumption; I can
always mirror changes in the swapfile code). I'll also take advantage of
the fact that much of the pagedir data we're storing is very easily
compressable itself to ensure that this doesn't result in severe bloat
of the pagedirs. I've already got a little patch to test it the theory,
and results look very encouraging but I won't try to explain it here.
I'll just confuse the issue.

> > - nice display
> > - user can abort at any time during suspend (oh, I forgot, I wanted
> > to...) by just pressing Escape
> 
> That seems like missfeature. We don't want joe random user that is at
> the console to prevent suspend by just pressing Escape. Maybe magic
> key to do that would be acceptable...

Magic key? Do you mean a password? I suppose that could be a
configuration option. I for one love being able to just press escape
when I realise there was something I wanted to do before I shut the
computer down, and I'll bet there'll be tons of people who think the
same way. I could certainly go for a password option. The only question
would be, how to configure the password. Perhaps a root-only proc entry?

Regards,

Nigel

-- 
Nigel Cunningham
495 St Georges Road South, Hastings 4201, New Zealand

You see, at just the right time, when we were still powerless,
Christ died for the ungodly.
	-- Romans 5:6, NIV.


  reply	other threads:[~2003-07-12 14:51 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-07-11 22:45 Thoughts wanted on merging Software Suspend enhancements Nigel Cunningham
2003-07-12 14:00 ` Pavel Machek
2003-07-12 14:55   ` Nigel Cunningham [this message]
2003-07-12 15:37     ` Pavel Machek
2003-07-12 19:38       ` Nigel Cunningham
2003-07-12 20:15         ` Pavel Machek
2003-07-12 20:22           ` [Swsusp-devel] " Nigel Cunningham
2003-07-12 22:52             ` Pavel Machek
2003-07-13  2:17               ` Jan Rychter
2003-07-12 20:20       ` Nigel Cunningham
2003-07-12 22:56         ` Pavel Machek
2003-07-13 13:27         ` Jamie Lokier
2003-07-13 19:23           ` Pavel Machek
2003-07-13 19:29             ` Jamie Lokier
2003-07-13 18:17       ` [Swsusp-devel] " Michael Frank
2003-07-13 19:29         ` Pavel Machek
2003-07-12 22:34   ` Dmitry Torokhov
2003-07-12 22:51     ` Pavel Machek
2003-07-13 13:35       ` Jamie Lokier
2003-07-13 19:31         ` Pavel Machek
2003-07-13 21:01           ` [Swsusp-devel] " Nigel Cunningham
2003-07-13 21:09             ` Pavel Machek
2003-07-13 21:21               ` Jan Rychter
2003-07-14  1:54               ` Nigel Cunningham
2003-07-14  3:41                 ` Michael Frank
2003-07-14 13:11                 ` Pavel Machek
2003-07-14 17:30                   ` Jan Rychter
2003-07-14 18:44                     ` Nigel Cunningham
2003-07-14 20:12                 ` Vojtech Pavlik
2003-07-14 20:18                   ` Pavel Machek
2003-07-14 20:30                     ` Nigel Cunningham
2003-07-14 20:43                       ` Pavel Machek
2003-07-14 21:20                         ` Nigel Cunningham
2003-07-14 20:41                     ` Vojtech Pavlik
     [not found]                       ` <20030714230219.GB11283@elf.ucw.cz>
2003-07-15  6:36                         ` Vojtech Pavlik
2003-07-15 10:08                           ` Pavel Machek
2003-07-15 11:11                             ` Markus Gaugusch
2003-07-15 11:31                               ` Pavel Machek
     [not found]             ` <Pine.LNX.4.53.0307132305500.1090@phoenix.kerstin.at>
2003-07-14  1:15               ` Jamie Lokier
2003-07-14  1:26                 ` Lyle Seaman
2003-07-14  1:43                   ` Jamie Lokier
2003-07-14  1:55                     ` Lyle Seaman
2003-07-14  1:57                       ` Nigel Cunningham
2003-07-14  2:17                       ` Lyle Seaman
2003-07-14 20:10           ` Vojtech Pavlik
2003-07-14 20:21             ` Nigel Cunningham
2003-07-14 20:29               ` Pavel Machek
2003-07-14 20:32                 ` Nigel Cunningham
2003-07-14 20:38                   ` Pavel Machek
2003-07-15 16:23         ` Kent Borg
2003-07-15 16:27           ` Pavel Machek
2003-07-15 17:27           ` Jamie Lokier
2003-07-15 19:05             ` Nigel Cunningham

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=1058021722.1687.16.camel@laptop-linux \
    --to=ncunningham@clear.net.nz \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@suse.cz \
    --cc=swsusp-devel@lists.sourceforge.net \
    --cc=torvalds@transmeta.com \
    /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