diff options
| author | Jim Cromie <jim.cromie@gmail.com> | 2021-10-14 01:07:25 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-10-14 11:59:10 +0300 |
| commit | 7a5e202dfb8ab7572eba703e43f08dabdc8b7808 (patch) | |
| tree | 2748cb297222c9b65e8680c2d2af85e0a407ab29 /lib | |
| parent | f0ada6da3a0d69682e21f1783d02676e0fbf1bc1 (diff) | |
| download | linux-7a5e202dfb8ab7572eba703e43f08dabdc8b7808.tar.xz | |
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')
| -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; } |