Commit 76681c8f authored by Petr Mladek's avatar Petr Mladek Committed by Rusty Russell

module: return bool from within_module*()

The within_module*() functions return only true or false. Let's use bool as
the return type.

Note that it should not change kABI because these are inline functions.
Signed-off-by: default avatarPetr Mladek <pmladek@suse.cz>
Signed-off-by: default avatarRusty Russell <rusty@rustcorp.com.au>
parent 9b20a352
...@@ -396,19 +396,21 @@ bool is_module_address(unsigned long addr); ...@@ -396,19 +396,21 @@ bool is_module_address(unsigned long addr);
bool is_module_percpu_address(unsigned long addr); bool is_module_percpu_address(unsigned long addr);
bool is_module_text_address(unsigned long addr); bool is_module_text_address(unsigned long addr);
static inline int within_module_core(unsigned long addr, const struct module *mod) static inline bool within_module_core(unsigned long addr,
const struct module *mod)
{ {
return (unsigned long)mod->module_core <= addr && return (unsigned long)mod->module_core <= addr &&
addr < (unsigned long)mod->module_core + mod->core_size; addr < (unsigned long)mod->module_core + mod->core_size;
} }
static inline int within_module_init(unsigned long addr, const struct module *mod) static inline bool within_module_init(unsigned long addr,
const struct module *mod)
{ {
return (unsigned long)mod->module_init <= addr && return (unsigned long)mod->module_init <= addr &&
addr < (unsigned long)mod->module_init + mod->init_size; addr < (unsigned long)mod->module_init + mod->init_size;
} }
static inline int within_module(unsigned long addr, const struct module *mod) static inline bool within_module(unsigned long addr, const struct module *mod)
{ {
return within_module_init(addr, mod) || within_module_core(addr, mod); return within_module_init(addr, mod) || within_module_core(addr, mod);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment