Re: ALTQ
:
:Hi all,
:the two patches http://leaf.dragonflybsd.org/~joerg/altq.diff
:(kernel part) and http://leaf.dragonflybsd.org/~joerg/pfctl.diff
:(pfctl support) add the infrastructure for ALTQ.
:
:Currently supported are tun(4), sppp(4), ppp(4) and wi(4). I'll
:add support for more drivers later.
:
:If noone objects, this goes into the tree tomorrow.
:
:Joerg
It looks ok except for the various IFQ_*() macros. Passing a variable
name as an argument to a macro (e.g. err) which then assigns the
variable creates a lot of confusion. The NFSM code did this (albeit
much more aggregiously) and it created a real mess.
Could you make those IFQ_*() macros real procedures instead of macros?
Or at least real inline procedures.
-Matt
Matthew Dillon
<dillon@backplane.com>
討論串 (同標題文章)