All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: Netfilter Development Mailinglist <netfilter-devel@lists.netfilter.org>
Cc: Pablo Neira Ayuso <pablo@netfilter.org>
Subject: NFNL_NFA_NEST
Date: Wed, 21 Mar 2007 06:08:11 +0100	[thread overview]
Message-ID: <4600BDBB.8020205@trash.net> (raw)

One of the worst mistakes in nfnetlink in my opinion was the
introduction of the NFNL_NFA_NEST bit. It prevents us from
using a large part of the generic netlink stuff, since that
just interprets it as a really huge attribute type. Since
its not used even for anything, this is really annoying.

Unfortunately there is no easy way to get rid of it, current
userspace code sends it to the kernel, so even if we stop
including it in the kernel, we need to deal with it for
compatibilty.

What I want to propose is to stop sending it on both kernel
and userspace side immediately, but continue to accept it
for another year or two. After this time, we switch to use
the generic netlink stuff, which would break using old
libnfnetlink users. Since we just introduced a new API and
are fading out the old one anyway, this doesn't seem too
bad.

Any comments?

             reply	other threads:[~2007-03-21  5:08 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-21  5:08 Patrick McHardy [this message]
2007-03-21 10:04 ` NFNL_NFA_NEST Jozsef Kadlecsik
2007-03-21 10:13   ` NFNL_NFA_NEST Patrick McHardy
2007-03-21 10:39     ` NFNL_NFA_NEST Jozsef Kadlecsik
2007-03-21 22:54 ` NFNL_NFA_NEST Pablo Neira Ayuso
2007-03-22 11:00   ` NFNL_NFA_NEST Patrick McHardy
2007-03-22 13:18     ` NFNL_NFA_NEST Pablo Neira Ayuso
2007-03-22 13:29       ` NFNL_NFA_NEST Patrick McHardy
2007-03-22 16:44         ` NFNL_NFA_NEST Pablo Neira Ayuso
2007-03-22 17:01           ` NFNL_NFA_NEST Patrick McHardy
2007-03-23 12:18             ` NFNL_NFA_NEST Pablo Neira Ayuso
2007-03-23 12:55               ` NFNL_NFA_NEST Pablo Neira Ayuso
2007-03-23 13:01                 ` NFNL_NFA_NEST Patrick McHardy
2007-03-23 13:00               ` NFNL_NFA_NEST Patrick McHardy
2007-03-23 17:37                 ` NFNL_NFA_NEST Pablo Neira Ayuso
2007-03-24 10:49                   ` NFNL_NFA_NEST Patrick McHardy
2007-03-24 11:30                     ` NFNL_NFA_NEST Pablo Neira Ayuso
2007-03-24 14:37                       ` NFNL_NFA_NEST 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=4600BDBB.8020205@trash.net \
    --to=kaber@trash.net \
    --cc=netfilter-devel@lists.netfilter.org \
    --cc=pablo@netfilter.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.