From: David Mosberger <davidm@napali.hpl.hp.com>
To: linux-ia64@vger.kernel.org
Subject: RE: [Linux-ia64] new kernel patch (relative to 2.4.18)
Date: Wed, 08 May 2002 19:21:27 +0000 [thread overview]
Message-ID: <marc-linux-ia64-105590701905555@msgid-missing> (raw)
In-Reply-To: <marc-linux-ia64-105590701905293@msgid-missing>
>>>>> On Wed, 8 May 2002 11:25:08 -0700 , "Luck, Tony" <tony.luck@intel.com> said:
Tony> David Mosberger wrote:
>> If you look at the patch, you'll also see that I started merging
>> the "mem_map in virtual memory" patch (CONFIG_VIRTUAL_MEM_MAP),
>> which is currently needed on zx1 platforms with >1GB of memory.
>> However, the merge isn't complete and the new code isn't working
>> yet.
Tony> When I started working on a new discontigmem patch for ia64,
Tony> Kanoj Sarcar dissuaded me from a virtual mem_map with a lot of
Tony> fast handwaving on the cost of TLB faults to access page
Tony> structures. He stated (and I agreed) that there wasn't much
Tony> reason to expect good locality of reference to page structs
Tony> ... so this could have a measurable impact. Sadly, neither of
Tony> us actually made any measurements to back up this belief.
Tony> Have you looked at the possible performance effect of this
Tony> change?
The person who wrote this code (John Marvin) did look into this and
for the test he looked at (which included lmbench and kernel
compilation), couldn't find a noticable performance difference.
Nevertheless, I'm not completely comfortable with this approach. The
reason is that (a) it's not completely general (there could be machine
configurations where the mem_map is so large that it would exceed the
mappable space in region 5) and (b) increasing the TLB pressure is
just about the last thing I want to do (I'm not just worried about the
cost of accessing mem_map per se, but about thrashing the TLB and
kicking out more useful translations).
On the other hand, the virtual mem_map implementation has a compelling
simplicity to it and is more general than most everything else I have
seen so far.
Tony> Would the discontigmem patch meet the needs of the zx1
Tony> platform?
The patches I have seen so far always assume that you *know* where the
holes are. That just doesn't cut it. DIG makes absolutely no
guarantees about the physical memory layout and we need a kernel that
can work on all (reasonable) DIG machines.
There have been discussions on this topic on the lkml recently and
different solutions have been proposed. I don't really care too much
which solution is used in the end as long as it is able to accommodate
arbitrary physical layouts and has no dramatic performance impact.
For now, the virtual mem-map approach is working (modulo my merging
goofs) and it should be interesting to compare it to other approaches.
--david
next prev parent reply other threads:[~2002-05-08 19:21 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-03-15 5:02 [Linux-ia64] new kernel patch (relative to 2.5.7-pre1) David Mosberger
2002-04-10 21:42 ` [Linux-ia64] new kernel patch (relative to 2.4.18) David Mosberger
2002-04-10 22:23 ` Stephane Eranian
2002-04-11 18:22 ` Chris McDermott
2002-04-11 20:04 ` Saxena, Sunil
2002-04-11 23:49 ` [Linux-ia64] new kernel patch (relative to 2.5.8-pre3) David Mosberger
2002-04-12 0:46 ` Greg KH
2002-04-12 0:49 ` David Mosberger
2002-04-12 1:01 ` Jes Sorensen
2002-04-12 1:04 ` Greg KH
2002-04-12 2:15 ` David Mosberger
2002-04-12 3:50 ` Greg KH
2002-04-12 4:43 ` Greg KH
2002-04-12 5:31 ` David Mosberger
2002-04-22 7:02 ` [Linux-ia64] new kernel patch (relative to 2.4.18) Zach, Yoav
2002-04-22 14:47 ` n0ano
2002-04-22 14:57 ` Andreas Schwab
2002-04-22 15:34 ` David Mosberger
2002-04-22 15:40 ` n0ano
2002-05-02 17:41 ` Steffen Persvold
2002-05-03 7:35 ` Ross Elliott
2002-05-03 17:23 ` David Mosberger
2002-05-03 23:36 ` Steffen Persvold
2002-05-08 17:49 ` David Mosberger
2002-05-08 18:25 ` Luck, Tony
2002-05-08 19:21 ` David Mosberger [this message]
2002-05-08 19:43 ` Jack Steiner
2002-05-08 20:29 ` David Mosberger
2002-05-08 23:26 ` Luck, Tony
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=marc-linux-ia64-105590701905555@msgid-missing \
--to=davidm@napali.hpl.hp.com \
--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