Commit bfa3ca44 authored by Jim Cromie's avatar Jim Cromie Committed by Greg Kroah-Hartman
Browse files

dyndbg: show both old and new in change-info 



print "old => new" flag values to the info("change") message.

no functional change.

Acked-by: default avatarJason Baron <jbaron@akamai.com>
Acked-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: default avatarJim Cromie <jim.cromie@gmail.com>
Link: https://lore.kernel.org/r/20220904214134.408619-4-jim.cromie@gmail.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 85d6b66d

lib/dynamic_debug.c

+6 −5
Original line number Diff line number Diff line
@@ -156,7 +156,7 @@ static int ddebug_change(const struct ddebug_query *query, 
	struct ddebug_table *dt;

	unsigned int newflags;

	unsigned int nfound = 0;

	struct flagsbuf fbuf;

	struct flagsbuf fbuf, nbuf;



	/* search for matching ddebugs */

	mutex_lock(&ddebug_lock);
@@ -217,11 +217,12 @@ static int ddebug_change(const struct ddebug_query *query, 
				static_branch_enable(&dp->key.dd_key_true);

			}

#endif

			dp->flags = newflags;

			v4pr_info("changed %s:%d [%s]%s =%s\n",

			v4pr_info("changed %s:%d [%s]%s %s => %s\n",

				  trim_prefix(dp->filename), dp->lineno,

				  dt->mod_name, dp->function,

				 ddebug_describe_flags(dp->flags, &fbuf));

				  ddebug_describe_flags(dp->flags, &fbuf),

				  ddebug_describe_flags(newflags, &nbuf));

			dp->flags = newflags;

		}

	}

	mutex_unlock(&ddebug_lock);