From: Pablo Neira Ayuso <pablo@netfilter.org>
To: Jan Engelhardt <jengelh@medozas.de>
Cc: hadi@cyberus.ca,
Netfilter Development Mailinglist
<netfilter-devel@vger.kernel.org>,
Patrick McHardy <kaber@trash.net>
Subject: Re: user-space xtables ABI [was Re: [Fwd: Re: iptables pull request]]
Date: Sun, 17 May 2009 18:12:01 +0200 [thread overview]
Message-ID: <4A103751.9060501@netfilter.org> (raw)
In-Reply-To: <alpine.LSU.2.00.0905171711230.6819@fbirervta.pbzchgretzou.qr>
Jan Engelhardt wrote:
> On Sunday 2009-05-17 16:46, Pablo Neira Ayuso wrote:
>> With this policy, design errors accumulate along time so we learn the
>> lesson from our own mistakes and, then, we work on a new version 2 of
>> the API to resolve the accumulated issues after some time. This is how
>> I'm managing existing netfilter libraries. This policy makes the
>> developement of libraries/public interfaces slower but I think that
>> users are way happier (no binary breakages).
>
> With that policy we end up with the same crap that is happening
> in the kernel.
I didn't say that this was nice, it's functional :)
> Just because we magically managed to stuff lots of functions into an
> .so file does not make it public. It just so happens to save a bunch
> of kilobytes in all of the binaries that were previously statically
> linked to xtables.c, and was deemed one way to figure out how to deal
> with intrusive m_ipt. Sorry, but in all efforts that went in so far,
> I discharge libxtables having a stable API. For that, the iptables
> code is not beauty enough yet.
I think that we should prioritize backward compatibility versus beauty.
> Now, I had to just think of Xtables-addons that has a similar issue.
Indeed, actually I think that a stable ABI would make this easier for you.
> For the kernel modules, it uses a separate compat_xtables.c glue
> module that hides the hacks needed for older versions.
>
> Would tc profit from something similar for libxtables? It would gain
> the capability to work with iptableses potentially older than the
> reference point. But is that truly required? Upgrading userspace is a
> lot easier than the kernel. If a newer tc is installed on one's
> system, the user might just as well do so for iptables in the same
> run.
Sorry, I don't like that policy at all.
--
"Los honestos son inadaptados sociales" -- Les Luthiers
next prev parent reply other threads:[~2009-05-17 16:12 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <4A0E9786.8060407@netfilter.org>
[not found] ` <1242566196.3996.23.camel@dogo.mojatatu.com>
2009-05-17 14:46 ` user-space xtables ABI [was Re: [Fwd: Re: iptables pull request]] Pablo Neira Ayuso
2009-05-17 15:14 ` jamal
2009-05-17 15:40 ` Pablo Neira Ayuso
2009-05-17 16:48 ` jamal
2009-05-17 17:06 ` Jan Engelhardt
2009-05-17 17:11 ` jamal
2009-05-18 14:18 ` Pablo Neira Ayuso
2009-05-17 16:01 ` Jan Engelhardt
2009-05-17 16:12 ` Pablo Neira Ayuso [this message]
2009-05-17 16:39 ` Jan Engelhardt
2009-05-17 16:59 ` jamal
2009-05-17 17:11 ` Jan Engelhardt
2009-05-17 22:09 ` jamal
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=4A103751.9060501@netfilter.org \
--to=pablo@netfilter.org \
--cc=hadi@cyberus.ca \
--cc=jengelh@medozas.de \
--cc=kaber@trash.net \
--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.