diff mbox series

[net-next] net: sched: Clarify error message when qdisc kind is unknown

Message ID 20220118171909.4375-1-victor@mojatatu.com (mailing list archive)
State Accepted
Commit 973bf8fdd12f0e70ea351c018e68edd377a836d1
Delegated to: Netdev Maintainers
Headers show
Series [net-next] net: sched: Clarify error message when qdisc kind is unknown | expand

Checks

Context Check Description
netdev/tree_selection success Clearly marked for net-next
netdev/fixes_present success Fixes tag not required for -next series
netdev/subject_prefix success Link
netdev/cover_letter success Single patches do not need cover letters
netdev/patch_count success Link
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 3 this patch: 3
netdev/cc_maintainers success CCed 6 of 6 maintainers
netdev/build_clang success Errors and warnings before: 22 this patch: 22
netdev/module_param success Was 0 now: 0
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/verify_fixes success No Fixes tag
netdev/build_allmodconfig_warn success Errors and warnings before: 8 this patch: 8
netdev/checkpatch success total: 0 errors, 0 warnings, 0 checks, 8 lines checked
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/source_inline success Was 0 now: 0

Commit Message

Victor Nogueira Jan. 18, 2022, 5:19 p.m. UTC
When adding a tc rule with a qdisc kind that is not supported or not
compiled into the kernel, the kernel emits the following error: "Error:
Specified qdisc not found.". Found via tdc testing when ETS qdisc was not
compiled in and it was not obvious right away what the message meant
without looking at the kernel code.

Change the error message to be more explicit and say the qdisc kind is
unknown.

Signed-off-by: Victor Nogueira <victor@mojatatu.com>
---
 net/sched/sch_api.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

patchwork-bot+netdevbpf@kernel.org Jan. 20, 2022, 11:30 a.m. UTC | #1
Hello:

This patch was applied to netdev/net.git (master)
by David S. Miller <davem@davemloft.net>:

On Tue, 18 Jan 2022 14:19:09 -0300 you wrote:
> When adding a tc rule with a qdisc kind that is not supported or not
> compiled into the kernel, the kernel emits the following error: "Error:
> Specified qdisc not found.". Found via tdc testing when ETS qdisc was not
> compiled in and it was not obvious right away what the message meant
> without looking at the kernel code.
> 
> Change the error message to be more explicit and say the qdisc kind is
> unknown.
> 
> [...]

Here is the summary with links:
  - [net-next] net: sched: Clarify error message when qdisc kind is unknown
    https://git.kernel.org/netdev/net/c/973bf8fdd12f

You are awesome, thank you!
diff mbox series

Patch

diff --git a/net/sched/sch_api.c b/net/sched/sch_api.c
index c9c6f49f9c28..eedb2df7cc6e 100644
--- a/net/sched/sch_api.c
+++ b/net/sched/sch_api.c
@@ -1204,7 +1204,7 @@  static struct Qdisc *qdisc_create(struct net_device *dev,
 
 	err = -ENOENT;
 	if (!ops) {
-		NL_SET_ERR_MSG(extack, "Specified qdisc not found");
+		NL_SET_ERR_MSG(extack, "Specified qdisc kind is unknown");
 		goto err_out;
 	}