module: Unify the return value type of try_module_get
authorGao Feng <fgao@ikuai8.com>
Wed, 19 Apr 2017 01:47:22 +0000 (09:47 +0800)
committerJessica Yu <jeyu@redhat.com>
Mon, 24 Apr 2017 04:13:01 +0000 (21:13 -0700)
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>
include/linux/module.h

index 0297c5c..6b79eb7 100644 (file)
@@ -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)