From: Pablo Neira Ayuso <pablo@netfilter.org>
To: Florian Westphal <fw@strlen.de>
Cc: netfilter-devel <netfilter-devel@vger.kernel.org>
Subject: Re: [PATCH V2] netfilter: remove extra timer from ecache extension
Date: Tue, 4 Dec 2012 13:13:23 +0100 [thread overview]
Message-ID: <20121204121323.GA29442@1984> (raw)
In-Reply-To: <1354613751-30154-1-git-send-email-fw@strlen.de>
Hi Florian,
On Tue, Dec 04, 2012 at 10:35:51AM +0100, Florian Westphal wrote:
> This brings the (per-conntrack) ecache extension back to 24 bytes in
> size (was 112 byte on x86_64 with lockdep on).
>
> Instead we use a per-ns tasklet to re-trigger event delivery. When we
> enqueue a ct entry into the dying list, the tasklet is scheduled.
>
> The tasklet will then deliver up to 20 entries. It will re-sched
> itself unless all the pending events could be delivered.
>
> While at it, dying list handling is moved into ecache.c, since its only
> revlevant if ct events are enabled.
Just tested this. My testbed consists of two firewalls in HA running
conntrackd with event reliable mode. I've got a client that generates
lots of small TCP flows that goes through the firewalls and reach a
benchmark server.
This is my analysis:
conntrack -C shows:
253244
254798
256754
258807
261548 <--- we hit table full, dropping packets
176849 <--- it seems the tasklet gets a chance to run
given that we get less interruptions from the NIC
166449 <--- it slightly empty the dying list
131176
55602
28316
28317
28317
# hits hits/s ^h/s ^bytes kB/s errs rst tout mhtime
4796894 15727 16509 2393805 2227 0 0 0 0.005
4813038 15728 16144 2340880 2227 0 0 0 0.005
4828796 15728 15758 2284910 2227 0 0 0 0.005
4845279 15731 16483 2390035 2227 0 0 0 0.005
4860956 15731 15677 2273165 2227 0 0 0 0.005
4876826 15731 15870 2301150 2227 0 0 0 0.005
4883165 15701 6339 919155 2223 0 0 0 0.004
4883165 15651 0 0 2216 0 0 0 0.000 <--- table full
4883165 15601 0 0 2209 0 0 0 0.000
4894657 15588 11492 1666340 2207 0 0 0 3.008
4913408 15598 18751 2718895 2208 0 0 0 0.004
4931896 15607 18488 2680760 2210 0 0 0 0.004
So it seems the tasklet gets starved under heavy load.
This happens on and on, so after some time we hit table full and again
the dying list is empty.
These are old HP proliant DL145G2 from 2005, that's why the maximum
flows/s looks low.
Looking at the number and the behaviour under heavy stress, I think we
have to consider a different approach.
next prev parent reply other threads:[~2012-12-04 12:13 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-04 9:35 [PATCH V2] netfilter: remove extra timer from ecache extension Florian Westphal
2012-12-04 12:13 ` Pablo Neira Ayuso [this message]
2012-12-04 15:41 ` Florian Westphal
2012-12-04 17:21 ` Pablo Neira Ayuso
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=20121204121323.GA29442@1984 \
--to=pablo@netfilter.org \
--cc=fw@strlen.de \
--cc=netfilter-devel@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).