net/sched: sch_htb: warn about class in use while deleting
authorPedro Tammela <pctammela@mojatatu.com>
Fri, 28 Jul 2023 15:35:36 +0000 (12:35 -0300)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 1 Aug 2023 08:47:24 +0000 (10:47 +0200)
Add extack to warn that delete was rejected because
the class is still in use

Acked-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: Pedro Tammela <pctammela@mojatatu.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
net/sched/sch_htb.c

index 05c8291865ae8cdc75cbd61c1815a3d50bb96e0f..0d947414e61611f20f5983e60aafbaa9e4da2598 100644 (file)
@@ -1709,8 +1709,10 @@ static int htb_delete(struct Qdisc *sch, unsigned long arg,
         * tc subsys guarantee us that in htb_destroy it holds no class
         * refs so that we can remove children safely there ?
         */
-       if (cl->children || qdisc_class_in_use(&cl->common))
+       if (cl->children || qdisc_class_in_use(&cl->common)) {
+               NL_SET_ERR_MSG(extack, "HTB class in use");
                return -EBUSY;
+       }
 
        if (!cl->level && htb_parent_last_child(cl))
                last_child = 1;