From: "Catalin(ux) M. BOIE" <catab@embedromix.ro>
To: "Graeme Fowler" <graeme@graemef.net>
Cc: "David Miller" <davem@davemloft.net>,
jmack@wm7d.net, netdev@vger.kernel.org,
lvs-devel@vger.kernel.org
Subject: Re: [PATCH] IPVS: Allow boot time change of hash size.
Date: Thu, 4 Dec 2008 00:47:51 -0700 (MST) [thread overview]
Message-ID: <48003.91.201.80.240.1228376871.squirrel@mail.embedromix.ro> (raw)
In-Reply-To: <1228338713.24149.10.camel@ernie.internal.graemef.net>
> On Tue, 2008-12-02 at 16:16 -0700, Catalin(ux) M. BOIE wrote:
>> I have a need, I didn't wake up some day and I dreamed to change this.
>> I have a gateway with LVS and I have 4 web servers behind.
>> I saw the help text at that option and I saw that I could raise the
>> limit.
>
> OK, let's ask the same question a different way, after going a little
> further into your posting:
>
>> I was looking for anything that could get me past of 88.000 request per
>> seconds.
>> The help text told me to raise that value if I have big number of
>> connections. I just needed an easy way to test.
>
> OK, so from here:
>
> http://www.austintek.com/LVS/LVS-HOWTO/HOWTO/LVS-HOWTO.operation.html#hash_table
>
> and onward... have you read all of that? It explains how the hash table
> size has been developed over the years, and everything that has changed
> with and relating to it.
>
> To pull out an example, a hash table size of 21 bits does not give a
> connection limit of 2^21 entries, since each part of the hash is a
> linked list which contains multiple entries, up to the RAM limit in the
> server.
>
> In the HOWTO, the example given shows that for a traffic pattern with 4
> seconds session coherence and 1/8th of the traffic hitting the director
> being a SYN for the available config *appears* to require 2^21 bits.
> However, because each bit of the hash is a linked list, using 17 bits
> gets 16 entries in each list - this is not a problem, as it takes the
> CPU no time at all to search 16 entries.
>
> What you need to do for us, please, is to demonstrate that your problem
> (not exceeding 88k RPS) is categorically something to do with lookups in
> the hash table. I suspect (although I've been wrong before) that your
> problem is probably to do with the number of interrupts your hardware
> can process. There have been many posts of this type on lvs-users
> recently - search the archives to see what the solutions were.
>
> We're trying to help, but the collective wisdom here is that the hash
> table size is not the cause of your problem *which you haven't yet
> described fully*.
>
> Graeme
>
Hello!
After some replys I understood that I explained very bad what was my
intention.
I was not complaining that hash size is too low or too high.
I just provided a patch that I found useful: allow easy changing of a
variable at boot time and not to recompile the kernel to change it.
So, I didn't hit the stage when I do some tests to prove that the hash
size is too low/high.
After all you guys explained to me, it is pretty obvious that I will have
bigger problems than the walking of a linked list too deep.
So, I agree, my patch is useless.
Again, sorry for eating your precious time with this!
Thank you very much!
--
Catalin(ux) M. BOIE
http://kernel.embedromix.ro/
prev parent reply other threads:[~2008-12-04 7:47 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-26 13:36 [PATCH] IPVS: Allow boot time change of hash size Catalin(ux) M. BOIE
2008-11-26 14:40 ` Joseph Mack NA3T
2008-11-26 23:27 ` David Miller
2008-11-27 7:05 ` Catalin(ux) M. BOIE
2008-11-27 7:37 ` David Miller
2008-11-27 6:58 ` Catalin(ux) M. BOIE
2008-11-27 15:58 ` Joseph Mack NA3T
2008-11-28 8:49 ` Catalin(ux) M. BOIE
2008-11-28 14:55 ` Joseph Mack NA3T
2008-12-02 15:34 ` Catalin(ux) M. BOIE
2008-12-02 22:51 ` David Miller
2008-12-02 23:16 ` Catalin(ux) M. BOIE
2008-12-03 0:37 ` David Miller
2009-12-28 18:49 ` Mark Bergsma
2009-12-29 1:34 ` Simon Horman
2010-01-04 13:57 ` Patrick McHardy
2010-01-04 23:24 ` Simon Horman
2010-01-05 11:02 ` Mark Bergsma
2010-01-06 15:25 ` Catalin(ux) M. BOIE
2010-01-05 0:20 ` Simon Horman
2010-01-05 4:56 ` Patrick McHardy
2008-12-03 21:11 ` Graeme Fowler
2008-12-04 7:47 ` Catalin(ux) M. BOIE [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=48003.91.201.80.240.1228376871.squirrel@mail.embedromix.ro \
--to=catab@embedromix.ro \
--cc=davem@davemloft.net \
--cc=graeme@graemef.net \
--cc=jmack@wm7d.net \
--cc=lvs-devel@vger.kernel.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).