diff options
| author | Miroslav Benes <mbenes@suse.cz> | 2016-11-16 16:45:48 +0100 |
|---|---|---|
| committer | Jessica Yu <jeyu@redhat.com> | 2016-11-26 20:18:03 +0100 |
| commit | 71d9f5079358c148e71eba930e436a7a0cb35d95 (patch) | |
| tree | 1f27be8fddcc5a38bf202abb115e0bccf2713797 /kernel/module.c | |
| parent | module: When modifying a module's text ignore modules which are going away too (diff) | |
| download | linux-71d9f5079358c148e71eba930e436a7a0cb35d95.tar.xz 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.c | 7 |
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); |