NB: this is already in mm, but I need it in this patchset because kernel/params is subjected to testing by the selftest script, and the test depends upon its stability.
kernel/params.c has a pr_debug() which prints the address of a parameter setter function, this is not legitimately useful. Since the value is known and useful to the user, print that instead. Cc: [email protected] Signed-off-by: Jim Cromie <[email protected]> --- kernel/params.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/params.c b/kernel/params.c index a668863a4bb6..5c8a05921e28 100644 --- a/kernel/params.c +++ b/kernel/params.c @@ -136,8 +136,8 @@ static int parse_one(char *param, if (!val && !(params[i].ops->flags & KERNEL_PARAM_OPS_FL_NOARG)) return -EINVAL; - pr_debug("handling %s with %p\n", param, - params[i].ops->set); + pr_debug("handling %s with value '%s'\n", param, + val ? val : "no-arg"); kernel_param_lock(params[i].mod); if (param_check_unsafe(¶ms[i])) err = params[i].ops->set(val, ¶ms[i]); -- 2.55.0

