[RFC PATCH v5 08/28] dyndbg: accept null site in ddebug_change
From: Jim Cromie <jim.cromie@gmail.com>
Date: 2021-05-11 18:51:46
Also in:
lkml
Subsystem:
dynamic debug, library code, the rest · Maintainers:
Jason Baron, Jim Cromie, Andrew Morton, Linus Torvalds
fix a debug-print that includes site info, by adding an alternate debug message that does not. no functional changes. Signed-off-by: Jim Cromie <jim.cromie@gmail.com> --- lib/dynamic_debug.c | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c
index 0896c681db40..fa89d69dad4e 100644
--- a/lib/dynamic_debug.c
+++ b/lib/dynamic_debug.c@@ -235,10 +235,17 @@ static int ddebug_change(const struct ddebug_query *query, static_branch_enable(&dp->key.dd_key_true); #endif dp->flags = newflags; - v2pr_info("changed %s:%d [%s]%s =%s\n", - trim_prefix(dc->filename), dp->lineno, - dt->mod_name, dc->function, - ddebug_describe_flags(dp->flags, &fbuf)); + + if (dc) + v2pr_info("changed %s:%d [%s]%s =%s\n", + trim_prefix(dc->filename), dp->lineno, + dt->mod_name, dc->function, + ddebug_describe_flags(dp->flags, &fbuf)); + else + v2pr_info("changed %s:%d =%s \"%s\"\n", + dt->mod_name, dp->lineno, + ddebug_describe_flags(dp->flags, &fbuf), + dp->format); } } mutex_unlock(&ddebug_lock);
--
2.31.1