Wed, Mar 08, 2017 at 01:03:42PM CET, ji...@kernel.org wrote:
>From: Jiri Kosina <jkos...@suse.cz>
>
>The original reason [1] for having hidden qdiscs (potential scalability
>issues in qdisc_match_from_root() with single linked list in case of large
>amount of qdiscs) has been invalidated by 59cc1f61f0 ("net: sched: convert
>qdisc linked list to hashtable").
>
>This allows us for bringing more clarity and determinism into the dump by
>making default pfifo qdiscs visible.
>
>We're not turning this on by default though, at it was deemed [2] too
>intrusive / unnecessary change of default behavior towards userspace.
>Instead, TCA_DUMP_INVISIBLE netlink attribute is introduced, which allows
>applications to request complete qdisc hierarchy dump, including the
>ones that have always been implicit/invisible.
>
>Singleton noop_qdisc stays invisible, as teaching the whole infrastructure
>about singletons would require quite some surgery with very little gain
>(seeing no qdisc or seeing noop qdisc in the dump is probably setting
>the same user expectation).
>
>[1]
>http://lkml.kernel.org/r/1460732328.10638.74.ca...@edumazet-glaptop3.roam.corp.google.com
>[2]
>http://lkml.kernel.org/r/20161021.105935.1907696543877061916.da...@davemloft.net
>
>Signed-off-by: Jiri Kosina <jkos...@suse.cz>
>---
[...]
>diff --git a/include/uapi/linux/rtnetlink.h b/include/uapi/linux/rtnetlink.h
>index 262f0379d83a..c7de00e09797 100644
>--- a/include/uapi/linux/rtnetlink.h
>+++ b/include/uapi/linux/rtnetlink.h
>@@ -542,6 +542,7 @@ enum {
> TCA_FCNT,
> TCA_STATS2,
> TCA_STAB,
>+ TCA_DUMP_INVISIBLE,
> TCA_PAD,
You are changing UAPI value of TCA_PAD...
> __TCA_MAX
> };
