[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090326153747.GB7950@localhost>
Date: Thu, 26 Mar 2009 18:37:47 +0300
From: Cyrill Gorcunov <gorcunov@...il.com>
To: Patrick McHardy <kaber@...sh.net>
Cc: davem@...emloft.net, daniel.lezcano@...e.fr,
netdev@...r.kernel.org, netfilter-devel@...r.kernel.org,
xemul@...nvz.org, adobriyan@...il.com
Subject: Re: [RFC v2 5/7] net: netfilter conntrack - add per-net
functionality for TCP protocol
[Patrick McHardy - Thu, Mar 26, 2009 at 04:13:58PM +0100]
> Cyrill Gorcunov wrote:
>> Here is an updated version which uses iterative assignment
>> of sysctl data. Please review.
>> ---
>>
>> From: Cyrill Gorcunov <gorcunov@...nvz.org>
>> Subject: [RFC v2 5/7] net: netfilter conntrack - add per-net functionality for TCP protocol
>>
>> Module specific data moved into per-net site and being allocated/freed
>> during net namespace creation/deletion. For this reason module_init/exit
>> calls added.
>
>
> I guess this isn't going to get any prettier :) Could you please
> send me the latest version of the missing patches against the current
> nf-next tree?
>
Yes, no problem, wait a bit please.
Btw, I think the bestest way (and surely more clean)
would be to use some new form of the sysctl templates.
But I didn't find such a form of writting which would
sutisfy me. Since you said those enums are written in
stone it's safe to use iterative fasion indeed but
as I see you found them not that "pretty" too :)
On the other hand -- if per-net initialization is not
that critical in speed (ie some speed could be sacrificed
for the code clarity -- I could play with those templates
maybe). Just a thought.
Cyrill
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists