From: David Mosberger <davidm@napali.hpl.hp.com>
To: Christoph Hellwig <hch@infradead.org>
Cc: davidm@hpl.hp.com, torvalds@transmeta.com, linux-kernel@vger.kernel.org
Subject: Re: bk pull
Date: Wed, 9 Apr 2003 15:23:34 -0700 [thread overview]
Message-ID: <16020.40294.606401.506996@napali.hpl.hp.com> (raw)
In-Reply-To: <20030409215650.A10311@infradead.org>
>>>>> On Wed, 9 Apr 2003 21:56:50 +0100, Christoph Hellwig <hch@infradead.org> said:
Christoph> Hey, I didn't mean _always_ build out of the box :) I'm
Christoph> much more looking forward to 2.6.x releases I'd like to
Christoph> see just build for ia64.
That's a goal I share. But I won't reach it without some help.
>> I'm working on) and the virtual mem_map support. The latter I
>> haven't pushed at all so far, mostly because I just haven't had
>> the time/energy/interest to do so. Also, I'm always optimistic
>> someone else comes along to help with the work... ;-)
Christoph> Well, I could try to help pushing some stuff. OTOH
Christoph> pushing stuff I don't understand is a bad idea.
That's true.
Christoph> Could you please explain e.g. the mm/bootmem.c and
Christoph> mm/page_alloc.c changes?
The bootmem patch is a performance speed up and some minor indendation
fixups. The problem was that the bootmem code was (a) hugely slow and
(b) had execution that grew quadratically with the size of the bootmap
bitmap. This causes noticable slowdowns, especially on machines with
(relatively) large holes in the physical memory map. Issue (b) is
addressed by maintaining the "last_success" cache, so that we start
the next search from the place where we last found some memory (this
part of the patch could stand additional reviewing/testing). Issue
(a) is addressed by using find_next_zero_bit() instead of the slow
bit-by-bit testing.
The page_alloc.c changes are in support of virtual mem_map: it needs
an architecture-specific hook when initializing the mem_map. The
patch should be safe for arches that don't use virtual mem_map.
--david
prev parent reply other threads:[~2003-04-09 22:11 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-04-09 19:27 David Mosberger
2003-04-09 19:38 ` Christoph Hellwig
2003-04-09 20:49 ` David Mosberger
2003-04-09 20:56 ` Christoph Hellwig
2003-04-09 22:23 ` David Mosberger [this message]
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=16020.40294.606401.506996@napali.hpl.hp.com \
--to=davidm@napali.hpl.hp.com \
--cc=davidm@hpl.hp.com \
--cc=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@transmeta.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
all inboxes | Powered by JetHome®