net: sched: remove the unused return value of unregister_qdisc
authorZhengchao Shao <shaozhengchao@huawei.com>
Mon, 15 Aug 2022 03:04:17 +0000 (11:04 +0800)
committerJakub Kicinski <kuba@kernel.org>
Wed, 17 Aug 2022 02:37:06 +0000 (19:37 -0700)
Return value of unregister_qdisc is unused, remove it.

Signed-off-by: Zhengchao Shao <shaozhengchao@huawei.com>
Link: https://lore.kernel.org/r/20220815030417.271894-1-shaozhengchao@huawei.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
include/net/pkt_sched.h
net/sched/sch_api.c

index 3372a1f67cf4eb45827cda62e8b0b6af126792b0..29f65632ebc5987d22ca7c8134ccf557e58d102e 100644 (file)
@@ -100,7 +100,7 @@ struct Qdisc *fifo_create_dflt(struct Qdisc *sch, struct Qdisc_ops *ops,
                               struct netlink_ext_ack *extack);
 
 int register_qdisc(struct Qdisc_ops *qops);
-int unregister_qdisc(struct Qdisc_ops *qops);
+void unregister_qdisc(struct Qdisc_ops *qops);
 void qdisc_get_default(char *id, size_t len);
 int qdisc_set_default(const char *id);
 
index bf87b50837a84088ff930c1b6c5a7827c89b1150..8b4d575a3bbd389c355b3544eb594f53a8ee5782 100644 (file)
@@ -171,7 +171,7 @@ out_einval:
 }
 EXPORT_SYMBOL(register_qdisc);
 
-int unregister_qdisc(struct Qdisc_ops *qops)
+void unregister_qdisc(struct Qdisc_ops *qops)
 {
        struct Qdisc_ops *q, **qp;
        int err = -ENOENT;
@@ -186,7 +186,8 @@ int unregister_qdisc(struct Qdisc_ops *qops)
                err = 0;
        }
        write_unlock(&qdisc_mod_lock);
-       return err;
+
+       WARN(err, "unregister qdisc(%s) failed\n", qops->id);
 }
 EXPORT_SYMBOL(unregister_qdisc);