diff options
| author | Jim Cromie <jim.cromie@gmail.com> | 2021-10-14 00:07:25 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-10-14 10:59:10 +0200 |
| commit | 7a5e202dfb8ab7572eba703e43f08dabdc8b7808 (patch) | |
| tree | 2748cb297222c9b65e8680c2d2af85e0a407ab29 /lib/dynamic_debug.c | |
| parent | device property: Add missed header in fwnode.h (diff) | |
| download | linux-7a5e202dfb8ab7572eba703e43f08dabdc8b7808.tar.xz linux-7a5e202dfb8ab7572eba703e43f08dabdc8b7808.zip | |
dyndbg: vpr-info on remove-module complete, not starting
On qemu --smp 3 runs, remove-module can get called 3 times.
So don't print on entry; instead print "removed" after entry is
found and removed, so just once.
Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
Link: https://lore.kernel.org/r/20211013220726.1280565-3-jim.cromie@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'lib/dynamic_debug.c')
| -rw-r--r-- | lib/dynamic_debug.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index a44ae0b4b022..d706abba8dad 100644 --- a/lib/dynamic_debug.c +++ b/lib/dynamic_debug.c @@ -1025,8 +1025,6 @@ int ddebug_remove_module(const char *mod_name) struct ddebug_table *dt, *nextdt; int ret = -ENOENT; - v2pr_info("removing module \"%s\"\n", mod_name); - mutex_lock(&ddebug_lock); list_for_each_entry_safe(dt, nextdt, &ddebug_tables, link) { if (dt->mod_name == mod_name) { @@ -1036,6 +1034,8 @@ int ddebug_remove_module(const char *mod_name) } } mutex_unlock(&ddebug_lock); + if (!ret) + v2pr_info("removed module \"%s\"\n", mod_name); return ret; } |