cvs commit: src/sbin/ipfw ipfw2.c

Julian Elischer julian at elischer.org
Mon Aug 7 17:14:19 UTC 2006


Gleb Smirnoff wrote:

>On Sat, Aug 05, 2006 at 01:58:50PM +0000, Stefan Farfeleder wrote:
>S> stefanf     2006-08-05 13:58:50 UTC
>S> 
>S>   FreeBSD src repository
>S> 
>S>   Modified files:
>S>     sbin/ipfw            ipfw2.c 
>S>   Log:
>S>   Use the SLIST_NEXT macro instead of sle_next.
>
>Uhhrm. This code is a hack really, and using bad style emphasizes
>that. It says: BEWARE! We use queue.h internals for our own dirty
>needs. So I left "sle_next" here intentionally.
>
>The hack was made here to keep ABI compatibility, and make the later
>changes mergeable to RELENG_6.
>
>Let it be SLIST_NEXT() now. Anyway there is a plan of major rewrite
>of API and ipfw(8) itself.
>
>  
>
great.. I have been in ipfw(2) the last week and have some sugestions for
increasing its efficiency.. especially the code that times out dynamic 
rules.


More information about the cvs-src mailing list