public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@suse.cz>
To: Shaohua Li <shaohua.li@intel.com>
Cc: Linux-pm mailing list <linux-pm@lists.osdl.org>
Subject: Re: swsusp: which page should be saved?
Date: Tue, 21 Mar 2006 10:42:30 +0100	[thread overview]
Message-ID: <20060321094230.GM24523@elf.ucw.cz> (raw)
In-Reply-To: <1142907568.11430.16.camel@sli10-desk.sh.intel.com>

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

Hi!

> > Ok, I guess it is okay to go in if it stays in -mm for long enough to 
> > get a lot of testing.
> I'll do more tests and back to you. BTW, I wonder if BIOS already saved
> reserved memory (those doing communication with OS) in the 'platform'
> method of S4.

Yep, it should be safe. I bet it will break some obscure machine, but
it will probably fix some obscure machine, too... Just needs lots of testing.

> > > Anyway, skipping kernel text should be safe, isn't it?
> > 
> > It probably is. But you need to save modules. 
> I just consider the region from kernel start(1M) to the end of rodata.
> In my test, the region is about 4M memory. Just adding several lines to
> save 4M memory is worthy.

Well, few lines to save 4MB is nice. OTOH 4MB are saved in about
100msec, and if it brings in hard-to-debug bug on obscure
machine... we did not win much.

> > And we do use some 
> > self-modifying code these days, no? (Called runtime patching or 
> > something like that.) 
> Alternative instructions? The resume OS will do the same modification
> anyway.

Okay, hopefully.

> > Ouch and IIRC top-level pagedir or something 
> > like that lives in kernel "text" -- it is in assembly and wrongly 
> > placed.
> i386 does the right thing and put the pagedir in data segment. x86_64
> not, I think we could clean it up.

This probably should be done, first, and gotten past Andi.

								Pavel
-- 
Picture of sleeping (Linux) penguin wanted...

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



  parent reply	other threads:[~2006-03-21  9:42 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-16  3:53 swsusp: which page should be saved? Shaohua Li
2006-03-15 23:08 ` Pavel Machek
2006-03-17  1:12   ` Shaohua Li
2006-03-17  6:59     ` Pavel Machek
2006-03-21  2:19       ` Shaohua Li
2006-03-21  3:33         ` Nigel Cunningham
2006-03-21  9:42         ` Pavel Machek [this message]
2006-04-07  3:46           ` Shaohua Li
2006-04-11  7:39             ` Pavel Machek
2006-04-11  7:52               ` Shaohua Li
2006-03-17 10:50     ` Rafael J. Wysocki
2006-03-16 16:05 ` Rafael J. Wysocki

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=20060321094230.GM24523@elf.ucw.cz \
    --to=pavel@suse.cz \
    --cc=linux-pm@lists.osdl.org \
    --cc=shaohua.li@intel.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