From: Henrik Nordstrom <hno@marasystems.com>
To: Patrick McHardy <kaber@trash.net>,
netfilter-devel <netfilter-devel@lists.samba.org>
Cc: Harald Welte <laforge@gnumonks.org>
Subject: Re: [PATCH}: Make MARK target terminate (resend)
Date: Sat, 29 Jun 2002 12:36:36 +0200 [thread overview]
Message-ID: <200206291236.36997@henrik.marasystems.com> (raw)
In-Reply-To: <3D1D81F8.9080309@trash.net>
On Saturday 29 June 2002 11.46, Patrick McHardy wrote:
> A CONNMARK patch will follow but currently CONNMARK doesn't apply
> clean against 2.4.18/2.4.19-pre10 ..
Note: There is two versions of the CONNMARK patch. The one in extra
applies if you are using the new_nat patch, the one on old_nat if
not.
Your last posting did stir up some discussion on how to deal with
this. Adding a "terminate" option to each and every of these
psuedo-targets is clearly not the way to go, and only cover a very
small subset of what is needed.
I proposed adding a new class of iptables things between matches and
targets, being neither a match for filtering or a target that
determines the ultimate fate of the packet. The names proposed for
these in the discussion was modifiers or actions.
The implementation of these can be done without needing to change the
kernel iptables API by simply piggying back on the match list in the
table structure. The modifiers/actions need to register themselves as
a match, and for compability with old rulesets and/or userspace tools
as a target as well..
The userspace tools need to have a new option for calling a
modifier/action. These should clearly be separated from matches.
So the question to the Netfilter core team is if it would be OK to add
a new option and "module class" to the userspace tools, and have the
existing IPT_CONTINUE targets dual-register as both a target and a
match. I can try to whip something together if this is seen as
something acceptable. Should be fully backwards/forward compatible
with existing rulesets with only a minimal amount of code
duplication. The only compability issue is that if you make use the
new feature then you cannot go back to a older userspace or kernel..
Regards
Henrik
next prev parent reply other threads:[~2002-06-29 10:36 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-06-29 9:46 [PATCH}: Make MARK target terminate (resend) Patrick McHardy
2002-06-29 10:03 ` Patrick Schaaf
2002-07-02 13:02 ` Patrick McHardy
2002-06-29 10:36 ` Henrik Nordstrom [this message]
2002-06-29 12:53 ` Brad Chapman
2002-07-01 7:50 ` Balazs Scheidler
2002-07-02 14:24 ` Harald Welte
2002-07-03 11:24 ` Jozsef Kadlecsik
2002-07-03 11:36 ` Patrick Schaaf
2002-07-03 12:41 ` Fabrice MARIE
2002-07-03 14:45 ` Hervé Eychenne
2002-07-03 23:50 ` Henrik Nordstrom
2002-07-04 23:58 ` Harald Welte
2002-07-05 4:01 ` Fabrice MARIE
2002-07-05 14:21 ` Harald Welte
2002-07-05 19:25 ` Patrick Schaaf
2002-07-05 1:15 ` Joakim Axelsson
2002-07-01 9:47 ` Jozsef Kadlecsik
2002-07-01 10:11 ` Henrik Nordstrom
2002-07-01 12:08 ` Jozsef Kadlecsik
2002-07-02 15:45 ` Harald Welte
2002-07-02 13: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=200206291236.36997@henrik.marasystems.com \
--to=hno@marasystems.com \
--cc=kaber@trash.net \
--cc=laforge@gnumonks.org \
--cc=netfilter-devel@lists.samba.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.