Re: [PATCH net-next v2 05/10] net: sched: use Qdisc rcu API instead of relying on rtnl lock

On Mon, Sep 17, 2018 at 12:19 AM Vlad Buslov [off-list ref] wrote:

+static void tcf_qdisc_put(struct Qdisc *q, bool rtnl_held)
+{
+       if (!q)
+               return;
+
+       if (rtnl_held)
+               qdisc_put(q);
+       else
+               qdisc_put_unlocked(q);
+}

This is very ugly. You should know whether RTNL is held or
not when calling it.

What's more, all of your code passes true, so why do you
need a parameter for rtnl_held?