From: Patrick McHardy <kaber@trash.net>
To: Frederic Leroy <fredo@starox.org>
Cc: Jan Engelhardt <jengelh@medozas.de>,
Bart De Schuymer <bdschuym@pandora.be>,
netfilter-devel@vger.kernel.org
Subject: Re: [arptables] rfc: add classify target
Date: Fri, 12 Nov 2010 08:49:59 +0100 [thread overview]
Message-ID: <4CDCF1A7.2050402@trash.net> (raw)
In-Reply-To: <20101111124505.345643a3@caresse>
On 11.11.2010 12:45, Frederic Leroy wrote:
> Le Thu, 11 Nov 2010 11:38:41 +0100,
> Patrick McHardy <kaber@trash.net> a écrit :
>
>> On 09.11.2010 21:51, Frederic Leroy wrote:
>>> Le Tue, 9 Nov 2010 21:28:09 +0100 (CET),
>>> Jan Engelhardt <jengelh@medozas.de> a écrit :
>>>
>>>> Alas, when I originally coded NFPROTO_UNSPEC wildcard support,
>>>> I allowed for same-rev overloading, as in:
>>>>
>>>> static struct xt_target classify_tg_reg[] __read_mostly = {
>>>> {
>>>> [...]
>>>> };
>>>>
>>>
>>> Here is a patch against my previous patch with your insights.
>>> I had time to test it.
>>
>> This seems like the best we can do for now. Does it work as intended?
>
> Yes, it works as intended.
>
> Nevertheless, I plan to update kernel and arptables to match NF_INET_*.
> I should have time to do it for sunday.
You can't change the numerical values, that would break compatibility.
That basically leaves the option of using NF_INET_PRE_ROUTING instead
of NF_ARP_IN etc, which would make things highly confusing :)
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2010-11-12 7:49 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-07 14:26 [arptables] rfc: add classify target Frederic Leroy
2010-11-07 15:18 ` Jan Engelhardt
2010-11-09 14:25 ` Patrick McHardy
2010-11-09 16:10 ` Frederic Leroy
2010-11-09 16:48 ` Patrick McHardy
2010-11-09 17:39 ` Bart De Schuymer
2010-11-09 20:18 ` Frederic Leroy
2010-11-09 20:28 ` Jan Engelhardt
2010-11-09 20:34 ` Frederic Leroy
2010-11-09 21:27 ` Jan Engelhardt
2010-11-09 21:38 ` Frederic Leroy
2010-11-09 20:51 ` Frederic Leroy
2010-11-11 10:38 ` Patrick McHardy
2010-11-11 11:45 ` Frederic Leroy
2010-11-12 7:49 ` Patrick McHardy [this message]
2010-11-13 15:29 ` Frederic Leroy
2010-11-14 15:36 ` Bart De Schuymer
2010-11-15 12:32 ` Frederic Leroy
2010-11-15 19:31 ` Bart De Schuymer
2010-11-15 10:44 ` Patrick McHardy
2010-11-15 12:28 ` [PATCH] netfilter: xtables: add arp support, allow CLASSIFY target on any table Frederic Leroy
2010-11-15 12:59 ` 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=4CDCF1A7.2050402@trash.net \
--to=kaber@trash.net \
--cc=bdschuym@pandora.be \
--cc=fredo@starox.org \
--cc=jengelh@medozas.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.