summaryrefslogtreecommitdiff
path: root/tools/lib/str_error_r.c
diff options
context:
space:
mode:
authorJessica Yu <jeyu@kernel.org>2020-04-08 17:31:06 +0300
committerJessica Yu <jeyu@kernel.org>2020-04-17 15:56:35 +0300
commitdb991af02f11053558431467102ee5832894d7a4 (patch)
tree9fc8174b3ed77f11f2ecab88303dff1c568794c1 /tools/lib/str_error_r.c
parent8f3d9f354286745c751374f5f1fcafee6b3f3136 (diff)
downloadlinux-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