On Fri, Mar 04, 2016 at 01:11:47PM +0100, Phil Sutter wrote:
> Signed-off-by: Phil Sutter <p...@nwl.cc>

great to see TC doc!
Few comments:

> +TC, the Traffic Control utility, has been there for a very long time - 
> forever
> +in my humble perception. It is still (and has ever been if I'm not mistaken) 
> the
> +only tool to configure QoS in Linux.

please make the doc less subjective. The above paragraph can be removed.

> +\section*{Conclusion}
> +
> +My personal impression is that although the \cmd{tc} utility is an absolute
> +necessity for anyone aiming at doing QoS in Linux professionally, there are 
> way
> +too many loose ends and trip wires present in it's environment. Contributing 
> to
> +this is the fact, that much of the non-essential functionality is redundantly
> +available in netfilter. Another problem which adds weight to the first one 
> is a
> +general lack of documentation. Of course, there are many HOWTOs and guides in
> +the internet, but since it's often not clear how up to date these are, I 
> prefer
> +the usual resources such as man or info pages. Surely nothing one couldn't 
> fix
> +in hindsight, but quality certainly suffers if the original author of the 
> code
> +does not or can not contribute to that.

This one is also very subjective and not suitable for the iproute2 doc.
Please remove it.

> +All that being said, once the steep learning curve has been mastered, the
> +conglomerate of (classful) qdiscs, filters and actions provides a highly
> +sophisticated and flexible infrastructure to perform QoS, which plays nicely
> +along with routing and firewalling setups.

The rest of the doc is great. Thank you.

Reply via email to