From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Pavel Machek <pavel@suse.cz>
Cc: William Lee Irwin III <wli@holomorphy.com>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] swsuspend and CONFIG_DISCONTIGMEM=y
Date: 15 Nov 2002 14:16:12 +0100 [thread overview]
Message-ID: <1037366172.877.30.camel@zion> (raw)
In-Reply-To: <20021115120233.GC25902@atrey.karlin.mff.cuni.cz>
On Fri, 2002-11-15 at 13:02, Pavel Machek wrote:
> Hi!
>
> > > The following dropped hunk from Pavel should repair it:
> >
> > [cc: list trimmed to spare the uninterested]
> >
> > Hmm, there are some oddities here in count_and_copy_data_pages(). It
> > looks like the CONFIG_HIGHMEM panic() is there because copy_page() is
> > done without kmapping, and the CONFIG_DISCONTIGMEM panic() is there
> > because the pgdat list etc. are not walked according to VM
> > conventions.
>
> How much memory is needed for HIGHMEM to be neccessary? Is it 1GB? If
> so, I can well imagine 1GB laptop....
Depends on the arch & other matters. 768Mb on PPC at least, and
it starting to be common within laptops as well.
> This certainly does not work. We'd need to do some deep magic in
> suspend_asm.S to copy pages back. [Well, deep magic... Same
> kmap_atomic.] But suspend_asm.S has to guarantee not touching any
> memory so the change is not quite trivial.
At worst, that could be an arch provided routine. On most PPC32's
I can then just disable data translation on the MMU and access
all pages without kmap'ing them. But that's not terribly portable
and each arch would need different kind of hacking.
Ben.
next prev parent reply other threads:[~2002-11-15 13:08 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-11-15 8:10 [PATCH] swsuspend and CONFIG_DISCONTIGMEM=y Arnaldo Carvalho de Melo
2002-11-15 8:49 ` William Lee Irwin III
2002-11-15 9:48 ` William Lee Irwin III
2002-11-15 12:02 ` Pavel Machek
2002-11-15 12:09 ` William Lee Irwin III
2002-11-15 18:09 ` Pavel Machek
2002-11-15 18:33 ` William Lee Irwin III
2002-11-15 13:16 ` Benjamin Herrenschmidt [this message]
2002-11-15 18:12 ` Pavel Machek
2002-11-15 18:56 ` William Lee Irwin III
2002-11-16 2:41 ` Alan Cox
2002-11-16 3:06 ` William Lee Irwin III
2002-11-16 17:35 ` Oliver Xymoron
2002-11-15 11:54 ` Pavel Machek
2002-11-15 11:57 ` Arnaldo Carvalho de Melo
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=1037366172.877.30.camel@zion \
--to=benh@kernel.crashing.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pavel@suse.cz \
--cc=wli@holomorphy.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