diff options
author | Miroslav Benes <mbenes@suse.cz> | 2016-11-16 10:45:48 -0500 |
---|---|---|
committer | Jessica Yu <jeyu@redhat.com> | 2016-11-26 14:18:03 -0500 |
commit | 71d9f5079358c148e71eba930e436a7a0cb35d95 (patch) | |
tree | 1f27be8fddcc5a38bf202abb115e0bccf2713797 /kernel/module.c | |
parent | 905dd707fc856bae57de144dcf873583881f9489 (diff) |
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 { | |||
313 | } index; | 313 | } index; |
314 | }; | 314 | }; |
315 | 315 | ||
316 | /* We require a truly strong try_module_get(): 0 means failure due to | 316 | /* |
317 | ongoing or failed initialization etc. */ | 317 | * We require a truly strong try_module_get(): 0 means success. |
318 | * Otherwise an error is returned due to ongoing or failed | ||
319 | * initialization etc. | ||
320 | */ | ||
318 | static inline int strong_try_module_get(struct module *mod) | 321 | static inline int strong_try_module_get(struct module *mod) |
319 | { | 322 | { |
320 | BUG_ON(mod && mod->state == MODULE_STATE_UNFORMED); | 323 | BUG_ON(mod && mod->state == MODULE_STATE_UNFORMED); |