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 --]
next prev 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