All of lore.kernel.org
 help / color / mirror / Atom feed
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.


  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 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.