summaryrefslogtreecommitdiffstats
path: root/kernel/module.c
diff options
context:
space:
mode:
authorMiroslav Benes <mbenes@suse.cz>2016-11-16 16:45:48 +0100
committerJessica Yu <jeyu@redhat.com>2016-11-26 11:18:03 -0800
commit71d9f5079358c148e71eba930e436a7a0cb35d95 (patch)
tree1f27be8fddcc5a38bf202abb115e0bccf2713797 /kernel/module.c
parent905dd707fc856bae57de144dcf873583881f9489 (diff)
downloadlinux-71d9f5079358c148e71eba930e436a7a0cb35d95.tar.gz
linux-71d9f5079358c148e71eba930e436a7a0cb35d95.tar.bz2
linux-71d9f5079358c148e71eba930e436a7a0cb35d95.zip
module: Fix a comment above strong_try_module_get()
The comment above strong_try_module_get() function is not true anymore. Return values changed with commit c9a3ba55bb5d ("module: wait for dependent modules doing init."). Signed-off-by: Miroslav Benes <mbenes@suse.cz> Link: http://lkml.kernel.org/r/alpine.LNX.2.00.1611161635330.12580@pobox.suse.cz [jeyu@redhat.com: style fixes to make checkpatch.pl happy] Signed-off-by: Jessica Yu <jeyu@redhat.com>
Diffstat (limited to 'kernel/module.c')
-rw-r--r--kernel/module.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/kernel/module.c b/kernel/module.c
index 927a67e30855..6281c70683d3 100644
--- a/kernel/module.c
+++ b/kernel/module.c
@@ -313,8 +313,11 @@ struct load_info {
} index;
};
-/* We require a truly strong try_module_get(): 0 means failure due to
- ongoing or failed initialization etc. */
+/*
+ * We require a truly strong try_module_get(): 0 means success.
+ * Otherwise an error is returned due to ongoing or failed
+ * initialization etc.
+ */
static inline int strong_try_module_get(struct module *mod)
{
BUG_ON(mod && mod->state == MODULE_STATE_UNFORMED);