public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: torvalds@transmeta.com (Linus Torvalds)
To: linux-kernel@vger.kernel.org
Subject: Re: Horrible L2 cache effects from kernel compile
Date: Tue, 25 Feb 2003 22:18:09 +0000 (UTC)	[thread overview]
Message-ID: <b3gq31$2h8$1@penguin.transmeta.com> (raw)
In-Reply-To: 3E5BB7EE.5090301@colorfullife.com

In article <3E5BB7EE.5090301@colorfullife.com>,
Manfred Spraul  <manfred@colorfullife.com> wrote:
>
>Are you sure that this will help?

It might, under some loads.

However, I don't think it's a long-term solution, since the hashing will
mean that for any reasonably spread out load you _will_ always walk all
dentries.

So the long-term solution is to either use a local lookup (which we
ended up doing for the page cache) _or_ to limit the number of dentries
themselves some way.  The latter sounds like a bad idea.

>Btw, has anyone tried to replaced the global dcache with something 
>local, perhaps a tree instead of d_child, and then lookup in d_child_tree?

I'd love to see somebody try.  The main worry is the overhead required
per directory dentry and keeping it scalable.  The dentry tree _will_ be
quickly populated and one common case is a few huge directories, yet at
the same time for most dentries there won't be any children at all or
very few of them). 

Right now the "child" list is just a simple linked list, and changing
that to something more complex might make it possible to get rid of the
hash entirely. But increasing the size of individual dentries is a bad
idea, so it would have to be something fairly smart.

			Linus

  parent reply	other threads:[~2003-02-25 22:12 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-02-25 18:37 Horrible L2 cache effects from kernel compile Manfred Spraul
2003-02-25 18:41 ` Dave Hansen
2003-02-25 18:42 ` Andi Kleen
2003-02-25 19:29 ` Martin J. Bligh
2003-02-25 22:18 ` Linus Torvalds [this message]
2003-03-03 19:03   ` Benjamin LaHaise
2003-03-03 19:13     ` Linus Torvalds
2003-03-03 23:58       ` Alan Cox
2003-03-03 22:57         ` Andrew Morton
2003-03-03 23:01         ` Benjamin LaHaise
2003-03-03 23:09         ` Linus Torvalds
2003-03-05 20:19         ` dean gaudet
     [not found] <3E5ABBC1.8050203@us.ibm.com.suse.lists.linux.kernel>
2003-02-25 16:16 ` Andi Kleen
2003-02-27  3:24   ` Dave Hansen
2003-02-27 16:36   ` Jan Harkes
     [not found] ` <b3ekil$1cp$1@penguin.transmeta.com.suse.lists.linux.kernel>
     [not found]   ` <20030225170546.GA23772@morningstar.nowhere.lie.suse.lists.linux.kernel>
2003-02-25 17:20     ` Andi Kleen
2003-02-26 18:22       ` Jamie Lokier
  -- strict thread matches above, loose matches on Subject: below --
2003-02-25  0:41 Dave Hansen
2003-02-25  0:59 ` William Lee Irwin III
2003-02-25  1:01   ` Dave Hansen
2003-02-25  3:15   ` John Levon
2003-02-25  3:15     ` William Lee Irwin III
2003-02-25  3:35     ` Andrew Morton
2003-02-25  4:13       ` Martin J. Bligh
2003-02-25 11:57         ` John Levon
2003-02-25  2:31 ` Linus Torvalds
2003-02-25 17:05   ` John W. M. Stevens

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='b3gq31$2h8$1@penguin.transmeta.com' \
    --to=torvalds@transmeta.com \
    --cc=linux-kernel@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