[PATCH] Introducing the Change API
kaber at trash.net
Fri Dec 17 18:26:43 CET 2004
Harald Welte wrote:
>On Fri, Dec 17, 2004 at 06:28:55AM +0100, Patrick McHardy wrote:
>>I would like to give it some cleanup first. Since it doesn't need to
>>deal with ipchains compatibility there is a lot possible cleanup.
>>I will probably do this in the next couple of weeks.
>What kind of cleanup are you planning? the provisions in conntrack are
>not that big for the compat stuff... it's more the provisions in nat
>(which we don't have yet for nf_conntrack).
Hmm I haven't looked into it very deeply yet, but I would like to kill
the nf_conntrack_standalone file and remove unneeded EXPORT_SYMBOLs.
I would also like to make naming more consistent and get rid of these
horrible long function names (nf_conntrack_do_something) by using the
nf_ct_ prefix everywhere.
>I'm willing to put some effort into this. The question is: Do we
>really need this cleanup before the merge, or can we do it afterwards?
>I mean, it seems to be stable - and it's not any more unclean than the
>current ip_conntrack code. And we'll mark it as EXPERIMENTAL first
>anyway, so we have more freedom to play with it ;)
Yes, but it's just less overhead doing this in your own tree than
submitting patches for all cleanups.
More information about the netfilter-devel