diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2015-01-22 11:13:14 +1030 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2015-01-22 11:15:54 +1030 |
commit | d5db139ab3764640e0882a1746e7b9fdee33fd87 (patch) | |
tree | 62f806a29a43c749c84354fbef3bdaa21dc3ce6f /kernel/debug/kdb | |
parent | c749637909eea5d4090c6f50b89c2c20b534a280 (diff) | |
download | linux-d5db139ab3764640e0882a1746e7b9fdee33fd87.tar.gz linux-d5db139ab3764640e0882a1746e7b9fdee33fd87.tar.bz2 linux-d5db139ab3764640e0882a1746e7b9fdee33fd87.zip |
module: make module_refcount() a signed integer.
James Bottomley points out that it will be -1 during unload. It's
only used for diagnostics, so let's not hide that as it could be a
clue as to what's gone wrong.
Cc: Jason Wessel <jason.wessel@windriver.com>
Acked-and-documention-added-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Reviewed-by: Masami Hiramatsu <maasami.hiramatsu.pt@hitachi.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'kernel/debug/kdb')
-rw-r--r-- | kernel/debug/kdb/kdb_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/debug/kdb/kdb_main.c b/kernel/debug/kdb/kdb_main.c index 379650b984f8..2934889f2cce 100644 --- a/kernel/debug/kdb/kdb_main.c +++ b/kernel/debug/kdb/kdb_main.c @@ -1979,7 +1979,7 @@ static int kdb_lsmod(int argc, const char **argv) kdb_printf("%-20s%8u 0x%p ", mod->name, mod->core_size, (void *)mod); #ifdef CONFIG_MODULE_UNLOAD - kdb_printf("%4ld ", module_refcount(mod)); + kdb_printf("%4d ", module_refcount(mod)); #endif if (mod->state == MODULE_STATE_GOING) kdb_printf(" (Unloading)"); |