From: "David S. Miller" <davem@redhat.com>
To: Robert Olsson <Robert.Olsson@data.slu.se>
Cc: kuznet@ms2.inr.ac.ru, netdev@oss.sgi.com, Robert.Olsson@data.slu.se
Subject: Re: [PATCH] option for large routing hash
Date: Tue, 9 Dec 2003 12:20:31 -0800 [thread overview]
Message-ID: <20031209122031.048b406f.davem@redhat.com> (raw)
In-Reply-To: <16341.58771.558850.163216@robur.slu.se>
On Tue, 9 Dec 2003 16:09:07 +0100
Robert Olsson <Robert.Olsson@data.slu.se> wrote:
> I think patch should be useful as it helps performance a lot during high
> flow load. I have some numbers if you are interested.
I'm very hesitant about this, and it is not because I don't believe
that it brings better performance in your tests on your machines :)
Recently there was a thread on linux-kernel by the folks, such as Jes,
working on super-duper-huge NUMA boxes and how big the hash tables
get sized to on these machines.
In some of their configurations it was trying to allocate 1GB TCP
hash tables or something totally rediculious like this.
The problem with all of our current algorithms for size selection is
that it considers only one parameter when there are actually two.
It considers currently only relative memory consumption. It needs
to also consider hard limits that exist for useful hash table sizes.
There is a point at which hash table size exceeds it's usefulness in
that the gains you are getting from the O(1) lookup are offset by the
fact that the access to the hash table heads are constantly taking cpu
cache misses.
You've obtained good results in your tests with a _specific_ hash
table size for the routing cache, but the algorithm you are proposing
for the kernel computes things relative to the amount of memory in the
machine. It cannot be a function of only this parameter.
Do you see my point?
next prev parent reply other threads:[~2003-12-09 20:20 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-12-09 15:09 [PATCH] option for large routing hash Robert Olsson
2003-12-09 20:20 ` David S. Miller [this message]
2003-12-09 22:28 ` Robert Olsson
2003-12-10 8:15 ` David S. Miller
-- strict thread matches above, loose matches on Subject: below --
2003-12-10 14:47 Robert Olsson
2003-12-10 23:05 ` David S. Miller
2003-12-12 23:10 ` Robert Olsson
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=20031209122031.048b406f.davem@redhat.com \
--to=davem@redhat.com \
--cc=Robert.Olsson@data.slu.se \
--cc=kuznet@ms2.inr.ac.ru \
--cc=netdev@oss.sgi.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;
as well as URLs for NNTP newsgroup(s).