All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: Krzysztof Oledzki <ole@ans.pl>
Cc: Jan Engelhardt <jengelh@medozas.de>,
	Holger Eitzenberger <holger@eitzenberger.org>,
	pablo@netfilter.org,
	Netfilter Developer Mailing List
	<netfilter-devel@vger.kernel.org>
Subject: Re: nf_conntrack.acct has no effect
Date: Tue, 17 Mar 2009 14:54:59 +0100	[thread overview]
Message-ID: <49BFABB3.9020903@trash.net> (raw)
In-Reply-To: <alpine.LNX.1.10.0903171450330.14581@bizon.gios.gov.pl>

Krzysztof Oledzki wrote:
>> The implication is that xt_connbytes will not do the right thing
>> anymore as soon as user accounting is turned off, either by flipping
>> the sysctl value or deactivating the kconfig option. That is not
>> good.
> 
> The sysctl is not going to be removed and you can still set the initial 
> value with the proper kernel/module option. I fail to see the problem here.

Its mainly a usability thing. Its not really obvious that the
match will be entirely without effect if you don't set some sysctl.
And since we can assume that anyone using the connbytes match
intends for it to have some effect, we might as well enable it
automatically.

  reply	other threads:[~2009-03-17 13:55 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-16 16:53 nf_conntrack.acct has no effect Jan Engelhardt
2009-03-16 16:56 ` Patrick McHardy
2009-03-16 18:01   ` Jan Engelhardt
2009-03-17  8:24   ` Holger Eitzenberger
2009-03-17 13:02     ` Patrick McHardy
2009-03-17 13:13       ` Krzysztof Oledzki
2009-03-17 13:31         ` Patrick McHardy
2009-03-17 13:32           ` Patrick McHardy
2009-03-17 13:50           ` Krzysztof Oledzki
2009-03-17 13:51             ` Patrick McHardy
2009-03-17 13:55               ` Krzysztof Oledzki
2009-03-17 13:57                 ` Patrick McHardy
2009-03-17 14:18                   ` Krzysztof Oledzki
2009-03-17 14:22                     ` Patrick McHardy
2009-03-17 14:36                       ` Krzysztof Oledzki
2009-03-17 14:38                         ` Patrick McHardy
2009-03-17 13:11     ` Krzysztof Oledzki
2009-03-17 13:28       ` Jan Engelhardt
2009-03-17 13:52         ` Krzysztof Oledzki
2009-03-17 13:54           ` Patrick McHardy [this message]
2009-03-17 14:39         ` Jan Engelhardt
2009-03-17 14:41           ` Patrick McHardy
2009-03-17 15:41             ` Jan Engelhardt
2009-03-18  4:32               ` Patrick McHardy

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=49BFABB3.9020903@trash.net \
    --to=kaber@trash.net \
    --cc=holger@eitzenberger.org \
    --cc=jengelh@medozas.de \
    --cc=netfilter-devel@vger.kernel.org \
    --cc=ole@ans.pl \
    --cc=pablo@netfilter.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.