netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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/


      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).