All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: Jan Engelhardt <jengelh@computergmbh.de>
Cc: netfilter-devel@vger.kernel.org
Subject: Re: [PATCH 3/8] [NETFILTER]: rename NF_ARP to NFPROTO_ARP and assign a non-clashing value
Date: Thu, 10 Apr 2008 03:10:47 +0200	[thread overview]
Message-ID: <47FD6917.7070109@trash.net> (raw)
In-Reply-To: <alpine.LNX.1.10.0804091930191.3866@fbirervta.pbzchgretzou.qr>

Jan Engelhardt wrote:
> On Wednesday 2008-04-09 19:05, Patrick McHardy wrote:
>>> I would have been going for NFPROTO_ now, with a translation
>>> table for use for things like the xfrm code.
>> We need to keep the INET/INET6 values identical anyway,
>> so there no need for translation here.
> 
> The problem is with non-IP,non-IPv6 that passes through xfrm.
> That is, if IPX can get xfrmd at all, probably does.

It doesn't. We also don't support IPX, this is IMO a non-issue
that can be discusses once IPX gains IPSEC support (aka never).

>>> Should I change all AF_INET6 to NFPROTO_ in the same run
>>> (i.e. mostly xt_[A-Za-z]*.c), or leave it as is?
>> If we introduce those values, they should be used everywhere.
>>
> That would amount to a huge patch (perhaps I split to make it feasible).

I have no problems taking a huge sed patch if it
makes sense.



  reply	other threads:[~2008-04-10  1:10 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-08 15:31 [PATCH 1/8] [NETFILTER]: Rename ipt_recent to xt_recent Jan Engelhardt
2008-04-08 15:31 ` [PATCH 2/8] [NETFILTER]: xt_recent: IPv6 support Jan Engelhardt
2008-04-09 12:48   ` Patrick McHardy
2008-04-09 13:30     ` Patrick McHardy
2008-04-09 13:44       ` Jan Engelhardt
2008-04-09 13:48         ` Patrick McHardy
2008-04-08 15:31 ` [PATCH 3/8] [NETFILTER]: rename NF_ARP to NFPROTO_ARP and assign a non-clashing value Jan Engelhardt
2008-04-09 12:52   ` Patrick McHardy
2008-04-09 13:09     ` Jan Engelhardt
2008-04-09 13:12       ` Patrick McHardy
2008-04-09 13:17         ` Jan Engelhardt
2008-04-09 13:21           ` Patrick McHardy
2008-04-09 13:34             ` Jan Engelhardt
2008-04-09 13:42               ` Patrick McHardy
2008-04-09 13:48                 ` Jan Engelhardt
2008-04-09 13:51                   ` Patrick McHardy
2008-04-09 13:59                     ` Jan Engelhardt
2008-04-09 14:30                       ` Patrick McHardy
2008-04-09 14:39                         ` Jan Engelhardt
2008-04-09 14:57                           ` Patrick McHardy
2008-04-09 15:12                             ` Jan Engelhardt
2008-04-09 15:15                               ` Patrick McHardy
2008-04-09 16:31                                 ` Jan Engelhardt
2008-04-09 16:35                                   ` Patrick McHardy
2008-04-09 16:46                                     ` Jan Engelhardt
2008-04-09 16:50                                       ` Patrick McHardy
2008-04-09 16:59                                         ` Jan Engelhardt
2008-04-09 17:05                                           ` Patrick McHardy
2008-04-09 17:38                                             ` Jan Engelhardt
2008-04-10  1:10                                               ` Patrick McHardy [this message]
2008-04-08 15:31 ` [PATCH 4/8] [NETFILTER]: Implement AF_UNSPEC as a wildcard for extensions Jan Engelhardt
2008-04-08 15:31 ` [PATCH 5/8] [NETFILTER]: Give AF-independent extensions an arpt_ alias Jan Engelhardt
2008-04-08 15:31 ` [PATCH 6/8] [NETFILTER]: Make Ebtables use Xtables infrastructure Jan Engelhardt
2008-04-09 13:08   ` Patrick McHardy
2008-04-09 13:12     ` Jan Engelhardt
2008-04-09 16:52     ` Jan Engelhardt
2008-04-10 20:11     ` Bart De Schuymer
2008-04-10 20:52       ` Jan Engelhardt
2008-04-13  5:24         ` Patrick McHardy
2008-04-08 15:31 ` [PATCH 7/8] [NETFILTER]: Collapse tcpmss_reverse_mtu{4,6} into one function Jan Engelhardt
2008-04-15 12:51   ` Patrick McHardy
2008-04-15 13:57     ` Jan Engelhardt
2008-04-15 13:59       ` Patrick McHardy
2008-04-08 15:31 ` [PATCH 8/8] [NETFILTER]: Deploy a prefix_length-to-network mask mapping table Jan Engelhardt
2008-04-09 12:45 ` [PATCH 1/8] [NETFILTER]: Rename ipt_recent to xt_recent Patrick McHardy
2008-04-09 12:50   ` Jan Engelhardt
2008-04-09 12:53     ` 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=47FD6917.7070109@trash.net \
    --to=kaber@trash.net \
    --cc=jengelh@computergmbh.de \
    --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 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.