From: Patrick McHardy <kaber@trash.net>
To: Jan Engelhardt <jengelh@computergmbh.de>
Cc: Laszlo Attila Toth <panther@balabit.hu>, netfilter-devel@vger.kernel.org
Subject: Re: [PATCHv4 0/2] Find address type on the packet's interface
Date: Mon, 19 Nov 2007 18:12:12 +0100 [thread overview]
Message-ID: <4741C3EC.3050500@trash.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0711191756550.32378@fbirervta.pbzchgretzou.qr>
Jan Engelhardt wrote:
> On Nov 19 2007 17:06, Patrick McHardy wrote:
>> I just read up on your and Jan's discussion, but you were too fast
>> for me :) I'm not sure whether this is really a good candidate
>> for x_tables. IPv4 and IPv6 addrtype have different meanings, the
>> IPv4 addrtype is based on routing, IPv6 solely on the address.
>> Especially things like "--addrtype local" won't work, which is
>> IMO the most useful feature. And since you don't actually add IPv6
>> support, I don't see any advantage in moving to x_tables. So I
>> think for now I'd prefer a change to the ipt_addrtype match.
>
> IMHO it does not make any difference whether it is xt_*.c or ipt_*.c,
> the cost is quite the same.
> I am all for xt_*.c, because that's the "new shiny" thing.
x_tables is meant for unified matches and targets, as long as theres
nothing to unify, there's no point in moving it over. So far I think
we only have a single xtables match that doesn't support both IPv4
and IPv6 (xt_conntrack), and I'd like to keep it that way.
next prev parent reply other threads:[~2007-11-19 17:12 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-19 15:55 [PATCHv4 0/2] Find address type on the packet's interface Laszlo Attila Toth
2007-11-19 15:55 ` [PATCHv4 1/2] Find address type on a specific or on any interface Laszlo Attila Toth
2007-11-19 15:55 ` [PATCHv4 2/2] Addrtype match: limit addrtype check to an interface. Moved to xtables Laszlo Attila Toth
2007-11-19 15:55 ` [PATCHv4 iptables] Address type match: limited to incoming or outgoing " Laszlo Attila Toth
2007-11-19 16:06 ` [PATCHv4 0/2] Find address type on the packet's interface Patrick McHardy
2007-11-19 17:00 ` Jan Engelhardt
2007-11-19 17:12 ` Patrick McHardy [this message]
2007-11-19 17:15 ` Jan Engelhardt
2007-11-19 17:20 ` Patrick McHardy
2007-11-20 10:54 ` Laszlo Attila Toth
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=4741C3EC.3050500@trash.net \
--to=kaber@trash.net \
--cc=jengelh@computergmbh.de \
--cc=netfilter-devel@vger.kernel.org \
--cc=panther@balabit.hu \
/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).