From: Pablo Neira Ayuso <pablo@netfilter.org>
To: Arturo Borrero Gonzalez <arturo@debian.org>
Cc: netfilter-devel@vger.kernel.org
Subject: Re: [conntrack-tools PATCH 2/4] conntrackd: make the daemon run in RT mode by default
Date: Tue, 6 Jun 2017 13:10:40 +0200 [thread overview]
Message-ID: <20170606111040.GA1839@salvia> (raw)
In-Reply-To: <149674671245.18546.17167682826049346258.stgit@nfdev2.cica.es>
Hi Arturo,
On Tue, Jun 06, 2017 at 12:58:32PM +0200, Arturo Borrero Gonzalez wrote:
> In order to prevent netlink buffer overrun, conntrackd is recommended to run
> at max priority.
> Make conntrackd to use a RT (SHED_RR) scheduler by default at max priority.
> This is common among other HA daemons. For example corosync uses SCHED_RR
> by default.
> This change should help ease the configuration of conntrackd.
>
> Note that a sched priority that high makes the nice value useless, so deprecate
> both options now.
>
> The code is moved to the init() routine. In case of error setting the
> scheduler, the system default will be used. Report a message to the user
> and continue working.
I think we should provide a good default if someone doesn't specify
anything. So defaulting to RT is fine to me so we converge to what
other HA software is doing.
But I think we should keep the Nice and Scheduler clauses. Just in
case anyone wants to do this fine grain tunning.
So my proposal is:
1) Remove them from the examples configuration files.
2) Keep these toggles documented in manpage.
3) Provide this default if someone doesn't specify anything.
So the idea is that we provide good defaults.
BTW, an option I would really deprecate is the Checksum, a lot of
experimentation was going on at the time I added this (more than 10
years ago), this should really go away since I don't see a usecase for
this.
Thanks!
next prev parent reply other threads:[~2017-06-06 11:11 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-06 10:58 [conntrack-tools PATCH 1/4] conntrackd: evaluate configuration earlier Arturo Borrero Gonzalez
2017-06-06 10:58 ` [conntrack-tools PATCH 2/4] conntrackd: make the daemon run in RT mode by default Arturo Borrero Gonzalez
2017-06-06 11:10 ` Pablo Neira Ayuso [this message]
2017-06-07 20:53 ` Arturo Borrero Gonzalez
2017-06-12 8:15 ` Pablo Neira Ayuso
2017-06-06 10:58 ` [conntrack-tools PATCH 3/4] conntrackd: cleanup if failed forking Arturo Borrero Gonzalez
2017-06-06 16:11 ` Pablo Neira Ayuso
2017-06-06 10:58 ` [conntrack-tools PATCH 4/4] conntrackd: deprecate unix backlog configuration Arturo Borrero Gonzalez
2017-06-06 11:11 ` Pablo Neira Ayuso
2017-06-06 11:21 ` Pablo Neira Ayuso
2017-06-06 16:13 ` Pablo Neira Ayuso
2017-06-06 16:11 ` [conntrack-tools PATCH 1/4] conntrackd: evaluate configuration earlier 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=20170606111040.GA1839@salvia \
--to=pablo@netfilter.org \
--cc=arturo@debian.org \
--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).