From: William Lee Irwin III <wli@holomorphy.com>
To: Christoph Lameter <clameter@sgi.com>
Cc: akpm@osdl.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org,
mpm@selenic.com
Subject: Re: [SLUB 2/2] i386 arch page size slab fixes
Date: Mon, 2 Apr 2007 16:00:51 -0700 [thread overview]
Message-ID: <20070402230051.GI2986@holomorphy.com> (raw)
In-Reply-To: <20070331193107.1800.28259.sendpatchset@schroedinger.engr.sgi.com>
On Sat, Mar 31, 2007 at 11:31:07AM -0800, Christoph Lameter wrote:
> Patch by William Irwin with only very minor modifications by me which are
> 1. Removal of HIGHMEM64G slab caches. It seems that virtualization hosts
> require a a full pgd page.
The HIGHMEM64G slab allocations are meaningfully performant vs.
page-sized allocations where virtualization is absent. I would
personally rather whip Xen into shape enough to be able to handle the
minimal pgd allocations than retain the oversized pgd allocations even
in only the Xen case. Also, the entire unshared kernel pmd shenanigan
in Xen is an artifact of its recursive pagetable affair, which can also
be done away with a SMOP.
On Sat, Mar 31, 2007 at 11:31:07AM -0800, Christoph Lameter wrote:
> 2. Add missing virtualization hook. Seems that we need a new way
> of serializing paravirt_alloc(). It may need to do its own serialization.
> 3. Remove ARCH_USES_SLAB_PAGE_STRUCT
This doesn't quite cover all bases. The changes to pageattr.c and
fault.c are dubious and need verification at the very least. They were
largely slapped together to get the files past the compiler for the
performance comparisons that were never properly done.
-- wli
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2007-04-02 23:00 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-03-31 19:30 [SLUB 0/2] SLUB: The unqueued slab allocator V6 Christoph Lameter
2007-03-31 19:31 ` [SLUB 1/2] SLUB core Christoph Lameter
2007-04-03 7:29 ` Nick Piggin
2007-03-31 19:31 ` [SLUB 2/2] i386 arch page size slab fixes Christoph Lameter
2007-03-31 19:55 ` Andrew Morton
2007-03-31 20:00 ` Christoph Lameter
2007-04-02 18:30 ` Christoph Lameter
2007-04-02 23:00 ` William Lee Irwin III [this message]
2007-04-02 23:02 ` Christoph Lameter
2007-03-31 19:31 ` [SLUB tool] slabinfo: Display slab statistics Christoph Lameter
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=20070402230051.GI2986@holomorphy.com \
--to=wli@holomorphy.com \
--cc=akpm@osdl.org \
--cc=clameter@sgi.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mpm@selenic.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