From: Jim Cromie <jim.cromie@gmail.com>
To: Andrew Morton <akpm@linux-foundation.org>,
Jason Baron <jbaron@akamai.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Shuah Khan <skhan@linuxfoundation.org>,
Simona Vetter <simona@ffwll.ch>,
Luis Chamberlain <mcgrof@kernel.org>,
Petr Pavlu <petr.pavlu@suse.com>
Cc: linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org,
dri-devel@lists.freedesktop.org, linux-arch@vger.kernel.org,
linux-modules@vger.kernel.org, linux-kselftest@vger.kernel.org,
Jim Cromie <jim.cromie@gmail.com>,
Louis Chauvet <louis.chauvet@bootlin.com>
Subject: [PATCH v7 17/29] dyndbg: reduce verbose/debug clutter
Date: Tue, 21 Jul 2026 14:57:06 -0600 [thread overview]
Message-ID: <20260721-dd-maint-2-v7-17-010fbe73b311@gmail.com> (raw)
In-Reply-To: <20260721-dd-maint-2-v7-0-010fbe73b311@gmail.com>
currently, for verbose=3, these are logged (blank lines for clarity):
dyndbg: query 0: "class DRM_UT_CORE +p" mod:*
dyndbg: split into words: "class" "DRM_UT_CORE" "+p"
dyndbg: op='+'
dyndbg: flags=0x1
dyndbg: *flagsp=0x1 *maskp=0xffffffff
dyndbg: parsed: func="" file="" module="" format="" lineno=0-0 class=...
dyndbg: no matches for query
dyndbg: no-match: func="" file="" module="" format="" lineno=0-0 class=...
dyndbg: processed 1 queries, with 0 matches, 0 errs
That is excessive, so this patch:
- shrinks 3 lines of 2nd stanza to single line
- drops 1st 2 lines of 3rd stanza
3rd line is like 1st, with result, not procedure.
2nd line is just status, retold in 4th, with more info.
New output:
dyndbg: query 0: "class DRM_UT_CORE +p"
dyndbg: split into words: "class" "DRM_UT_CORE" "+p"
dyndbg: op='+' flags=0x1 maskp=0xffffffff
dyndbg: processed 1 queries, with 0 matches, 0 errs
Also drop several verbose=3 messages in ddebug_add_module. When
modprobing a module, dyndbg currently logs/says "add-module", and then
"skipping" if the module has no prdbgs. Instead just check 1st and
return quietly.
Unmatched query diagnostics are intentionally restricted to verbose
level 3 (v3pr_info_dq) to reduce dmesg output clutter on standard
verbose levels (verbose=1 and verbose=2), aligning with the overall
de-cluttering of dynamic debug logging.
Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
Reviewed-by: Louis Chauvet <louis.chauvet@bootlin.com>
---
v7: update fingerprints of grammar-errs changed here
v4: rename vpr_dq_info to v3pr_dq_info to tell its active logging level
adjust some vX levels per doc'd intentions
v2: RvB after SoB
trivial change to verbose-debug output line to output the actual
"module" keyword rather than "mod:", and do so only when the module is
constrained by the callchain (ie as part of a modprobe).
was: query X: "(keyword value)* [+-=]flags" mod:*
now: query X: "(keyword value)* [+-=]flags"
or query X: module FOO "keyword value)* [+-=]flags"
IOW, adjust output to reflect the input grammar more closely.
---
lib/dynamic_debug.c | 24 +++---
.../selftests/dynamic_debug/dyndbg_selftest.sh | 86 +++++++++++-----------
2 files changed, 54 insertions(+), 56 deletions(-)
diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c
index 2fa034177268..251df88cb04e 100644
--- a/lib/dynamic_debug.c
+++ b/lib/dynamic_debug.c
@@ -128,7 +128,7 @@ do { \
#define v3pr_info(fmt, ...) vnpr_info(3, fmt, ##__VA_ARGS__)
#define v4pr_info(fmt, ...) vnpr_info(4, fmt, ##__VA_ARGS__)
-static void vpr_info_dq(const struct ddebug_query *query, const char *msg)
+static void v3pr_info_dq(const struct ddebug_query *query, const char *msg)
{
/* trim any trailing newlines */
int fmtlen = 0;
@@ -280,9 +280,6 @@ static int ddebug_change(const struct ddebug_query *query,
}
mutex_unlock(&ddebug_lock);
- if (!nfound && verbose)
- pr_info("no matches for query\n");
-
return nfound;
}
@@ -491,7 +488,6 @@ static int ddebug_parse_query(char *words[], int nwords,
*/
query->module = modname;
- vpr_info_dq(query, "parsed");
return 0;
}
@@ -515,7 +511,6 @@ static int ddebug_parse_flags(const char *str, struct flag_settings *modifiers)
pr_err("bad flag-op %c, at start of %s\n", *str, str);
return -EINVAL;
}
- v3pr_info("op='%c'\n", op);
for (; *str ; ++str) {
for (i = ARRAY_SIZE(opt_array) - 1; i >= 0; i--) {
@@ -529,7 +524,6 @@ static int ddebug_parse_flags(const char *str, struct flag_settings *modifiers)
return -EINVAL;
}
}
- v3pr_info("flags=0x%x\n", modifiers->flags);
/* calculate final flags, mask based upon op */
switch (op) {
@@ -545,7 +539,7 @@ static int ddebug_parse_flags(const char *str, struct flag_settings *modifiers)
modifiers->flags = 0;
break;
}
- v3pr_info("*flagsp=0x%x *maskp=0x%x\n", modifiers->flags, modifiers->mask);
+ v3pr_info("op='%c' flags=0x%x maskp=0x%x\n", op, modifiers->flags, modifiers->mask);
return 0;
}
@@ -574,7 +568,7 @@ static int ddebug_exec_query(char *query_string, const char *modname)
}
/* actually go and implement the change */
nfound = ddebug_change(&query, &modifiers);
- vpr_info_dq(&query, nfound ? "applied" : "no-match");
+ v3pr_info_dq(&query, nfound ? "applied" : "no-match");
return nfound;
}
@@ -597,7 +591,10 @@ static int ddebug_exec_queries(char *query, const char *modname)
if (!query || !*query || *query == '#')
continue;
- vpr_info("query %d: \"%s\" mod:%s\n", i, query, modname ?: "*");
+ if (modname)
+ v2pr_info("query %d: module %s \"%s\"\n", i, modname, query);
+ else
+ v2pr_info("query %d: \"%s\"\n", i, query);
rc = ddebug_exec_query(query, modname);
if (rc < 0) {
@@ -1164,11 +1161,10 @@ static int ddebug_add_module(struct _ddebug_info *di, const char *modname)
{
struct ddebug_table *dt;
- v3pr_info("add-module: %s.%d sites\n", modname, di->num_descs);
- if (!di->num_descs) {
- v3pr_info(" skip %s\n", modname);
+ if (!di->num_descs)
return 0;
- }
+
+ v3pr_info("add-module: %s %d sites\n", modname, di->num_descs);
dt = kzalloc_obj(*dt);
if (dt == NULL) {
diff --git a/tools/testing/selftests/dynamic_debug/dyndbg_selftest.sh b/tools/testing/selftests/dynamic_debug/dyndbg_selftest.sh
index 446455683df4..6e977486ea90 100755
--- a/tools/testing/selftests/dynamic_debug/dyndbg_selftest.sh
+++ b/tools/testing/selftests/dynamic_debug/dyndbg_selftest.sh
@@ -645,54 +645,56 @@ modular_tests=(
# ==============================================================================
function GOLDEN_RECORDS {
cat << 'EOF' | {
-#K= bb85d28739876b0fdf1eb426baae8ef7 FT_grammar_errs.1 dmesg
-#K= 9c405e06fcea50284df82d1ae9692403 FT_grammar_errs.2 dmesg
-#K= 0b0d9b0e6f4c45c7d75d699fd3b6aa93 FT_grammar_errs.3 dmesg
-#K= e008644e159c2a9d99a8d0806c06e71e FT_grammar_errs.4 dmesg
-#K= 5750ed178633f9a68623f928c093d16e FT_grammar_errs.5 dmesg
-#K= 2cd51e5e3c3e2501544524bed0fab620 FT_grammar_errs.6 dmesg
-#K= 3e1eba65bd936e1276c7e1d76b399595 FT_grammar_errs.7 dmesg
-#K= e3ac86676e309d4e72ff3da287736c29 FT_grammar_errs.8 dmesg
-#K= 466d325c23519119f754fa7860bae7c6 FT_grammar_errs.9 dmesg
-#K= 51c489366cd86c16280858b95adf34ac FT_grammar_errs.10 dmesg
-#K= dfc632c6c0cd6206b6141dbc5fb989db FT_grammar_errs.11 dmesg
-#K= 69650fc26e97a78082bd851eb4a5c83b FT_grammar_errs.12 dmesg
-#K= 5cfe5665962a482f416836793067f2a2 FT_grammar_errs.13 dmesg
-#K= 661cc9a2d8d3c381bfeae41392432575 FT_grammar_errs.14 dmesg
-#K= 5dd0accf9be37ce2f4bf697e9bdebe30 FT_grammar_errs.15 dmesg
-#K= bd8d9cf661c9aca54dac93c74cfc1501 FT_grammar_errs.16 dmesg
-#K= e991391048f2fdffaaed99ba36b787ed FT_grammar_errs.17 dmesg
-#K= 82d99278f471ebfbf47eeb19898525cf FT_grammar_errs.18 dmesg
-#K= f3f088fa7d276981bf2898c048087f76 FT_grammar_errs.19 dmesg
-#K= 984ecc198e0fa9fc91bc31caf7da2a5d FT_grammar_errs.20 dmesg
-#K= 76c8a33c585d414fcc3f1c8b5ea5b4b3 FT_grammar_errs.21 dmesg
-#K= 90cfc66e9d61f5c448353b97baa99145 FT_grammar_errs.22 dmesg
-#K= 37a51dfd6e84cb64b69d3a0e23e68ceb FT_grammar_errs.23 dmesg
-#K= 3aefa325b63742710b45e4531152c9d8 FT_grammar_errs.24 dmesg
-#K= bc8d2c23282cd6834879bc1f1b303d05 FT_grammar_errs.25 dmesg
-#K= 092ec5530d875be1dc96ad4202fc7dc3 FT_grammar_errs.26 dmesg
-#K= 98caa2677e9070953a852b27ab57861e FT_grammar_errs.27 dmesg
-#K= 8d755baa707c7b3b67491bf84708ef08 FT_grammar_errs.28 dmesg
-#K= 5dd0accf9be37ce2f4bf697e9bdebe30 FT_grammar_errs.29 dmesg
-#K= 2e37642dc8aee3d04b8a54a76b3b891c FT_grammar_errs.30 dmesg
-#K= 593c8b4c41932e3bde564a0008627ced FT_grammar_errs.31 dmesg
-#K= 85a2a0bee8d0cd10f883aa5477d62efb FT_grammar_errs.32 dmesg
-#K= f3f088fa7d276981bf2898c048087f76 FT_grammar_errs.33 dmesg
-#K= 629f6a7e379e02f38c543cdb48e3b84f FT_grammar_errs.34 dmesg
-#K= d9f0cffb0898c54690735a71b0d4f3bf FT_grammar_errs.35 dmesg
-#K= 4cf728b6a32043ea5336a14d0c435312 FT_grammar_errs.36 dmesg
-#K= a11ae3f86f4e5f539fd6168d318f74b4 FT_grammar_errs.37 dmesg
-#K= c7d3830b0391f6a773205066b64e15f9 FT_grammar_errs.38 dmesg
-#K= 5c506ea2bfd050439b580b91543e9be7 FT_grammar_errs.39 dmesg
-#K= a257660007408f7bb0cb43c99b1c7bb8 FT_grammar_errs.40 dmesg
-#K= 0082939f100300a7b327b143cf40cb43 FT_grammar_errs.41 dmesg
-#K= 657ac4360f978687ad25a705b2bb89f1 FT_grammar_errs.42 dmesg
+# --- Unregistered Baselines ---
+#K= f3dbd5afb9aa1750f93275b634499e22 FT_grammar_errs.1 dmesg
+#K= 200c01632c52a63f6d186da1c6460740 FT_grammar_errs.2 dmesg
+#K= 7d7141900ce6e32f15c99202309c63a4 FT_grammar_errs.3 dmesg
+#K= 1bb798a5831d0119789d424ef6cb55c4 FT_grammar_errs.4 dmesg
+#K= 5edd66e308b2792d5694df86c07a3eaf FT_grammar_errs.5 dmesg
+#K= 6f87d92ffe0812550f43287127c6f2b9 FT_grammar_errs.6 dmesg
+#K= c0eb05b58a008c722e091e1ae74440ec FT_grammar_errs.7 dmesg
+#K= 911929ec0e2ffc1f13822b479dec6805 FT_grammar_errs.8 dmesg
+#K= c1407512376369d2e591a4b25a4b607a FT_grammar_errs.9 dmesg
+#K= 2046abda72725ea06fe339d5f364f1c9 FT_grammar_errs.10 dmesg
+#K= b72f7fccf76f8a5bee47a05d7bb545fb FT_grammar_errs.11 dmesg
+#K= 98e2bd3e4f3da58536496a38ec3e6238 FT_grammar_errs.12 dmesg
+#K= b371c6ba52503d037dbc43da788af8be FT_grammar_errs.13 dmesg
+#K= cb8288d607b0c5282125852f3ab05107 FT_grammar_errs.14 dmesg
+#K= 9346a310c4ad57cc3746afbace702c3e FT_grammar_errs.15 dmesg
+#K= 533d27af85eed3c0fd2eaec961982a36 FT_grammar_errs.16 dmesg
+#K= 114e0632585e205a3347c82bac7d79f2 FT_grammar_errs.17 dmesg
+#K= 73f5c173bafdfb9674b5ecce77db3354 FT_grammar_errs.18 dmesg
+#K= 0fc110d078f60eacdd389e5975ba18d9 FT_grammar_errs.19 dmesg
+#K= 815a1c52f365510c644450bb80c07e72 FT_grammar_errs.20 dmesg
+#K= 621e3cd81b553973cb40a935bb9298f1 FT_grammar_errs.21 dmesg
+#K= 581222901232344ade18bbda58302c48 FT_grammar_errs.22 dmesg
+#K= ea0aae3e01b3bb22eb8ad7acd327b371 FT_grammar_errs.23 dmesg
+#K= 8f28189bff62a3d5ed16f537d41a725a FT_grammar_errs.24 dmesg
+#K= 19c425e5d3a645b5dc5e23758ba0f4a1 FT_grammar_errs.25 dmesg
+#K= 75415542333f2250f0e060a54dae50f8 FT_grammar_errs.26 dmesg
+#K= 0955815c0e595ab2206e25aa31fe1ef2 FT_grammar_errs.27 dmesg
+#K= 3ff4c0b60db33e44c3cd6f0e14f81e3e FT_grammar_errs.28 dmesg
+#K= 9346a310c4ad57cc3746afbace702c3e FT_grammar_errs.29 dmesg
+#K= 7aaf0a16c287e66b62e11298ee160b34 FT_grammar_errs.30 dmesg
+#K= 06350c62105b537cdd0c67736b29727d FT_grammar_errs.31 dmesg
+#K= 6ef0ec01805c8719d828553098f95377 FT_grammar_errs.32 dmesg
+#K= 0fc110d078f60eacdd389e5975ba18d9 FT_grammar_errs.33 dmesg
+#K= 72203b2d88d0617cd5c659d3b80e26f9 FT_grammar_errs.34 dmesg
+#K= 6ecb03736d5cddb7ab2aaff49d561be9 FT_grammar_errs.35 dmesg
+#K= eaa989336cb7c96c13ef4a3964fc6898 FT_grammar_errs.36 dmesg
+#K= 5b624d9c133d7bb4f5370c3ce06929ed FT_grammar_errs.37 dmesg
+#K= 127275739b1fe04c84eedad28ec154f6 FT_grammar_errs.38 dmesg
+#K= 3e2fd15a7e8c0583bc5066524bc50508 FT_grammar_errs.39 dmesg
+#K= 781995971d28f732a792522f3c56cdd3 FT_grammar_errs.40 dmesg
+#K= 6614a677d9f9ac09d9825b4e989d2c42 FT_grammar_errs.41 dmesg
+#K= 70de9afed457a6be9f9c3c81cbd6d4d5 FT_grammar_errs.42 dmesg
#K= 9a7aaed40738f1c5203af2e421a20bc2 FT_basic_queries.1 "kernel/params.c"
#K= 2a57a9e283a3912de3fa5e006fb330cc FT_basic_queries.2 "kernel/params.c"
#K= fed3186684428b2f05bcf5df960c639c FT_basic_queries.3 "kernel/params.c"
#K= 551f9801b6008553661453021ae6dfd3 FT_basic_queries.4 "kernel/params.c"
#K= 33f7162ac85020894fab9752ef07b89c FT_basic_queries.5 "kernel/params.c"
#K= 544d955c3bb9d3c78704f780d91396e0 FT_basic_queries.6 "kernel/params.c"
+# ------------------------------
EOF
# Read the K-recs and skip those for tests that can't run
while read -r line; do
--
2.55.0
next prev parent reply other threads:[~2026-07-21 20:57 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-21 20:56 [PATCH v7 00/29] fix dynamic-debug classmaps API for DRM Jim Cromie
2026-07-21 20:56 ` [PATCH v7 01/29] params: fix a pr_debug(" %p ") use - already in MM-* Jim Cromie
2026-07-21 20:56 ` [PATCH v7 02/29] selftests/dyndbg: Add kselftest script to verify dynamic-debug Jim Cromie
2026-07-21 20:56 ` [PATCH v7 03/29] drm: Fix incorrect ccflags-y spelling inside Makefile Jim Cromie
2026-07-21 20:56 ` [PATCH v7 04/29] drm: fix config dependent unused variable warning Jim Cromie
2026-07-21 20:56 ` [PATCH v7 05/29] drm: Mark CONFIG_DRM_USE_DYNAMIC_DEBUG as unBROKEN Jim Cromie
2026-07-21 20:56 ` [PATCH v7 06/29] vmlinux.lds.h: refactor BOUNDED_SECTION_* macros into bounded_sections.lds.h Jim Cromie
2026-07-21 20:56 ` [PATCH v7 07/29] vmlinux.lds.h: drop unused HEADERED_SECTION* macros Jim Cromie
2026-07-21 20:56 ` [PATCH v7 08/29] vmlinux.lds.h: Fix ALIGN(8) omission causing NULL ptr on i386 Jim Cromie
2026-07-21 20:56 ` [PATCH v7 09/29] vmlinux.lds.h: remove redundant ALIGN(8) directives Jim Cromie
2026-07-21 20:56 ` [PATCH v7 10/29] dyndbg.lds.S: fix lost dyndbg sections in modules Jim Cromie
2026-07-21 20:57 ` [PATCH v7 11/29] dyndbg: factor ddebug_match_desc out from ddebug_change Jim Cromie
2026-07-21 20:57 ` [PATCH v7 12/29] dyndbg: add stub macro for DECLARE_DYNDBG_CLASSMAP Jim Cromie
2026-07-21 20:57 ` [PATCH v7 13/29] dyndbg: reword "class unknown," to "class:_UNKNOWN_" Jim Cromie
2026-07-21 20:57 ` [PATCH v7 14/29] dyndbg-API: remove DD_CLASS_TYPE_(DISJOINT|LEVEL)_NAMES and code Jim Cromie
2026-07-21 20:57 ` [PATCH v7 15/29] dyndbg: drop NUM_TYPE_ARGS Jim Cromie
2026-07-21 20:57 ` [PATCH v7 16/29] dyndbg: bump num-tokens in a query-cmd from 9 to 15 Jim Cromie
2026-07-21 20:57 ` Jim Cromie [this message]
2026-07-21 20:57 ` [PATCH v7 18/29] lib/parser: add match_wildcard_hyphen() for agnostic matching Jim Cromie
2026-07-21 20:57 ` [PATCH v7 19/29] dyndbg: use KBUILD_MODFILE for unique builtin module names Jim Cromie
2026-07-21 20:57 ` [PATCH v7 20/29] dyndbg: refactor param_set_dyndbg_classes and below Jim Cromie
2026-07-21 20:57 ` [PATCH v7 21/29] dyndbg: tighten fn-sig of ddebug_apply_class_bitmap Jim Cromie
2026-07-21 20:57 ` [PATCH v7 22/29] dyndbg: replace classmap list with an array-slice Jim Cromie
2026-07-21 20:57 ` [PATCH v7 23/29] dyndbg: macrofy a 2-index for-loop pattern Jim Cromie
2026-07-21 20:57 ` [PATCH v7 24/29] dyndbg: pin class param storage to u32 Jim Cromie
2026-07-21 20:57 ` [PATCH v7 25/29] dyndbg,module: make proper substructs in _ddebug_info Jim Cromie
2026-07-21 20:57 ` [PATCH v7 26/29] dyndbg: move mod_name down from struct ddebug_table to _ddebug_info Jim Cromie
2026-07-21 20:57 ` [PATCH v7 27/29] dyndbg: hoist classmap-filter-by-modname up to ddebug_add_module Jim Cromie
2026-07-21 20:57 ` [PATCH v7 28/29] dyndbg-API: replace DECLARE_DYNDBG_CLASSMAP Jim Cromie
2026-07-21 20:57 ` [PATCH v7 29/29] selftests/dyndbg: enable FT_classmap_inheritance Jim Cromie
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260721-dd-maint-2-v7-17-010fbe73b311@gmail.com \
--to=jim.cromie@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=gregkh@linuxfoundation.org \
--cc=jbaron@akamai.com \
--cc=linux-arch@vger.kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=linux-modules@vger.kernel.org \
--cc=louis.chauvet@bootlin.com \
--cc=mcgrof@kernel.org \
--cc=petr.pavlu@suse.com \
--cc=simona@ffwll.ch \
--cc=skhan@linuxfoundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox