From: Andy Whitcroft <apw@shadowen.org>
To: linux-ia64@vger.kernel.org
Subject: Re: [patch 0/4] ia64 SPARSEMEM
Date: Wed, 01 Jun 2005 09:14:36 +0000 [thread overview]
Message-ID: <429D7C7C.2090701@shadowen.org> (raw)
In-Reply-To: <20050523175031.GC2783@localhost.localdomain>
Luck, Tony wrote:
> Dave Hansen wrote:
>
>>* The same implementation works everywhere, on every architecture.
>
> Truth in advertising might require that this patch be renamed as
> "SOMEWHATSPARSE", as it breaks down for extremely sparse physical
> address space machines. It looks like the existing SGI Altix is at
> the ragged edge of what can be practically supported.
> Overall the SPARSEMEM patch is a good clean-up to an area of code that is
> unspeakably complex (because of all the interactions between NUMA, DISCONTIG,
> and VIRT_MEM_MAP). So I'm in full agreement that something needs to be done.
> I'm just not convinced yet that the existing form of the SPARSEMEM patch is
> the greatest thing since sliced bread.
As Dave has already indicated the key here is that this is a common
implementation. As the code stood we had a common implementation of
'flat' memory. We had per-architecture memory models where this was not
sufficient. Over time a number of key architectures have met the
non-contigious problem and solved it in nearly the same manner. What we
set out to do was to try and provide a common implementation for a more
complex discontigious memory format.
SPARSEMEM isn't trying to be the answer for every problem, more a simple
and clean implementation which would fix the majority of the current
users well. Where that was insufficient we still have the option of an
architecture specific memory model and significant effort has been made
in the foundations already accepted to make them usable by such an
implementation.
That said, it also was expected that SPARSEMEM would form the foundation
for more complex memory models. As you mention there are already some
big iron machines which push the simplistic mapping we currently have to
the very limit and probabally beyond. As Dave says if this is fixed in
SPARSEMEM then we gain for any other architecture that finds itself up
against the limits.
-apw
next prev parent reply other threads:[~2005-06-01 9:14 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-05-23 17:50 [patch 0/4] ia64 SPARSEMEM Bob Picco
2005-05-24 3:29 ` David Mosberger
2005-05-24 14:33 ` Bob Picco
2005-05-24 16:27 ` Bob Picco
2005-05-26 0:32 ` Luck, Tony
2005-05-26 20:09 ` David Mosberger
2005-05-26 20:54 ` Bob Picco
2005-05-26 21:02 ` Dave Hansen
2005-05-26 21:34 ` Luck, Tony
2005-05-26 21:44 ` Jack Steiner
2005-05-26 21:51 ` Bob Picco
2005-05-26 22:03 ` Luck, Tony
2005-05-26 22:04 ` Bob Picco
2005-05-27 5:14 ` Yasunori Goto
2005-05-27 10:35 ` Bob Picco
2005-05-27 16:23 ` David Mosberger
2005-05-27 22:04 ` Jack Steiner
2005-05-30 0:18 ` KAMEZAWA Hiroyuki
2005-05-31 17:55 ` Luck, Tony
2005-05-31 18:14 ` Dave Hansen
2005-05-31 18:15 ` Jack Steiner
2005-05-31 21:41 ` Luck, Tony
2005-05-31 21:58 ` Dave Hansen
2005-06-01 1:37 ` Bob Picco
2005-06-01 9:14 ` Andy Whitcroft [this message]
2005-06-01 22:48 ` David Mosberger
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=429D7C7C.2090701@shadowen.org \
--to=apw@shadowen.org \
--cc=linux-ia64@vger.kernel.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