From: Peter Zijlstra <a.p.zijlstra@chello.nl>
To: Christoph Lameter <clameter@sgi.com>
Cc: netdev@vger.kernel.org, linux-mm@kvack.org,
David Miller <davem@davemloft.net>
Subject: Re: [RFC][PATCH 1/6] mm: slab allocation fairness
Date: Thu, 30 Nov 2006 20:40:12 +0100 [thread overview]
Message-ID: <1164915612.6588.171.camel@twins> (raw)
In-Reply-To: <Pine.LNX.4.64.0611301137120.24161@schroedinger.engr.sgi.com>
On Thu, 2006-11-30 at 11:37 -0800, Christoph Lameter wrote:
> On Thu, 30 Nov 2006, Peter Zijlstra wrote:
>
> > On Thu, 2006-11-30 at 10:52 -0800, Christoph Lameter wrote:
> >
> > > I would think that one would need a rank with each cached object and
> > > free slab in order to do this the right way.
> >
> > Allocation hardness is a temporal attribute, ie. it changes over time.
> > Hence I do it per slab.
>
> cached objects are also temporal and change over time.
Sure, but there is nothing wrong with using a slab page with a lower
allocation rank when there is memory aplenty.
I'm just not seeing how keeping all individual page ranks would make
this better.
The only thing that matters is the actual free pages limit, not that of
a few allocation ago. The stored rank is a safe shortcut for it allows
harder allocation to use easily obtainable free space not the other way
around.
next prev parent reply other threads:[~2006-11-30 19:57 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20061130101451.495412000@chello.nl>
[not found] ` <20061130101922.328418000@chello.nl>
2006-11-30 12:04 ` [RFC][PATCH 6/6] net: vm deadlock avoidance core Peter Zijlstra
[not found] ` <20061130101921.113055000@chello.nl>
2006-11-30 18:52 ` [RFC][PATCH 1/6] mm: slab allocation fairness Christoph Lameter
2006-11-30 18:55 ` Peter Zijlstra
2006-11-30 19:33 ` Christoph Lameter
2006-11-30 19:33 ` Peter Zijlstra
2006-12-01 11:28 ` Peter Zijlstra
2006-11-30 19:02 ` Peter Zijlstra
2006-11-30 19:37 ` Christoph Lameter
2006-11-30 19:40 ` Peter Zijlstra [this message]
2006-11-30 20:11 ` Christoph Lameter
2006-11-30 20:15 ` Peter Zijlstra
2006-11-30 20:29 ` Christoph Lameter
[not found] ` <20061130101922.175620000@chello.nl>
2006-11-30 18:55 ` [RFC][PATCH 5/6] slab: kmem_cache_objs_to_pages() Christoph Lameter
2006-11-30 18:55 ` Peter Zijlstra
2006-11-30 19:06 ` Christoph Lameter
2006-11-30 19:03 ` Peter Zijlstra
2006-12-01 12:14 ` Peter Zijlstra
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=1164915612.6588.171.camel@twins \
--to=a.p.zijlstra@chello.nl \
--cc=clameter@sgi.com \
--cc=davem@davemloft.net \
--cc=linux-mm@kvack.org \
--cc=netdev@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;
as well as URLs for NNTP newsgroup(s).