diff options
| author |   Gao Feng <fgao@ikuai8.com> | 2017-04-19 09:47:22 +0800 |
|---|---|---|
| committer |   Jessica Yu <jeyu@redhat.com> | 2017-04-23 21:13:01 -0700 |
| commit | 8ba4fcdf0f4068407e98cd9cc0f230c2dd8d56de (patch) | |
| tree | ae8fbc64db5f265f29b672b1b78f2a3036426b7e /include/linux/module.h | |
| parent | module: set .init_array alignment to 8 (diff) | |
| download | linux-dev-8ba4fcdf0f4068407e98cd9cc0f230c2dd8d56de.tar.xz linux-dev-8ba4fcdf0f4068407e98cd9cc0f230c2dd8d56de.zip | |
module: Unify the return value type of try_module_get
The prototypes of try_module_get are different with different macro.
When enable module and module unload, it returns bool, but others not.
Make the return type for try_module_get consistent across all module
config options.
Signed-off-by: Gao Feng <fgao@ikuai8.com>
[jeyu: slightly amended changelog to make it clearer]
Signed-off-by: Jessica Yu <jeyu@redhat.com>
Diffstat (limited to '')
| -rw-r--r-- | include/linux/module.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/linux/module.h b/include/linux/module.h index 0297c5cd7cdf..6b79eb76a523 100644 --- a/include/linux/module.h +++ b/include/linux/module.h @@ -582,7 +582,7 @@ extern bool try_module_get(struct module *module); extern void module_put(struct module *module); #else /*!CONFIG_MODULE_UNLOAD*/ -static inline int try_module_get(struct module *module) +static inline bool try_module_get(struct module *module) { return !module || module_is_live(module); } @@ -674,9 +674,9 @@ static inline void __module_get(struct module *module) { } -static inline int try_module_get(struct module *module) +static inline bool try_module_get(struct module *module) { - return 1; + return true; } static inline void module_put(struct module *module) |