diff options
author | Jessica Yu <jeyu@kernel.org> | 2020-04-08 17:31:06 +0300 |
---|---|---|
committer | Jessica Yu <jeyu@kernel.org> | 2020-04-17 15:56:35 +0300 |
commit | db991af02f11053558431467102ee5832894d7a4 (patch) | |
tree | 9fc8174b3ed77f11f2ecab88303dff1c568794c1 /tools/lib/str_error_r.c | |
parent | 8f3d9f354286745c751374f5f1fcafee6b3f3136 (diff) | |
download | linux-db991af02f11053558431467102ee5832894d7a4.tar.xz |
module: break nested ARCH_HAS_STRICT_MODULE_RWX and STRICT_MODULE_RWX #ifdefs
Various frob_* and module_{enable,disable}_* functions are defined in a
CONFIG_ARCH_HAS_STRICT_MODULE_RWX ifdef block which also has a nested
CONFIG_STRICT_MODULE_RWX ifdef block within it. This is unecessary and
makes things hard to read. Not only that, this construction requires
redundant empty stubs for module_enable_nx(). I suspect this was
originally done for cosmetic reasons - to keep all the frob_* functions
in the same place, and all the module_{enable,disable}_* functions right
after, but as a result it made the code harder to read.
Make this more readable by unnesting the ifdef blocks and getting rid of
the redundant empty stubs.
Signed-off-by: Jessica Yu <jeyu@kernel.org>
Diffstat (limited to 'tools/lib/str_error_r.c')
0 files changed, 0 insertions, 0 deletions