diff options
author | Jani Nikula <jani.nikula@intel.com> | 2014-08-26 16:51:23 -0400 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2014-08-27 08:24:07 -0400 |
commit | 6a4c264313c4ae32dc53821a9c57e0dc9696fb81 (patch) | |
tree | 51a3e86768d16053dad28282f13782d4bb3043b0 | |
parent | 68e370289c29e3beac99d59c6d840d470af9dfcf (diff) |
module: rename KERNEL_PARAM_FL_NOARG to avoid confusion
Make it clear this is about kernel_param_ops, not kernel_param (which
will soon have a flags field of its own). No functional changes.
Cc: Rusty Russell <rusty@rustcorp.com.au>
Cc: Jean Delvare <khali@linux-fr.org>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Li Zhong <zhong@linux.vnet.ibm.com>
Cc: Jon Mason <jon.mason@intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
-rw-r--r-- | include/linux/moduleparam.h | 2 | ||||
-rw-r--r-- | kernel/module.c | 2 | ||||
-rw-r--r-- | kernel/params.c | 6 | ||||
-rw-r--r-- | security/apparmor/lsm.c | 4 |
4 files changed, 7 insertions, 7 deletions
diff --git a/include/linux/moduleparam.h b/include/linux/moduleparam.h index 494f99e852da..16fdddab856a 100644 --- a/include/linux/moduleparam.h +++ b/include/linux/moduleparam.h | |||
@@ -42,7 +42,7 @@ struct kernel_param; | |||
42 | * NOARG - the parameter allows for no argument (foo instead of foo=1) | 42 | * NOARG - the parameter allows for no argument (foo instead of foo=1) |
43 | */ | 43 | */ |
44 | enum { | 44 | enum { |
45 | KERNEL_PARAM_FL_NOARG = (1 << 0) | 45 | KERNEL_PARAM_OPS_FL_NOARG = (1 << 0) |
46 | }; | 46 | }; |
47 | 47 | ||
48 | struct kernel_param_ops { | 48 | struct kernel_param_ops { |
diff --git a/kernel/module.c b/kernel/module.c index 03214bd288e9..8a0dc91eddbc 100644 --- a/kernel/module.c +++ b/kernel/module.c | |||
@@ -135,7 +135,7 @@ static int param_set_bool_enable_only(const char *val, | |||
135 | } | 135 | } |
136 | 136 | ||
137 | static const struct kernel_param_ops param_ops_bool_enable_only = { | 137 | static const struct kernel_param_ops param_ops_bool_enable_only = { |
138 | .flags = KERNEL_PARAM_FL_NOARG, | 138 | .flags = KERNEL_PARAM_OPS_FL_NOARG, |
139 | .set = param_set_bool_enable_only, | 139 | .set = param_set_bool_enable_only, |
140 | .get = param_get_bool, | 140 | .get = param_get_bool, |
141 | }; | 141 | }; |
diff --git a/kernel/params.c b/kernel/params.c index 34f527023794..8a484fc8bde8 100644 --- a/kernel/params.c +++ b/kernel/params.c | |||
@@ -104,7 +104,7 @@ static int parse_one(char *param, | |||
104 | return 0; | 104 | return 0; |
105 | /* No one handled NULL, so do it here. */ | 105 | /* No one handled NULL, so do it here. */ |
106 | if (!val && | 106 | if (!val && |
107 | !(params[i].ops->flags & KERNEL_PARAM_FL_NOARG)) | 107 | !(params[i].ops->flags & KERNEL_PARAM_OPS_FL_NOARG)) |
108 | return -EINVAL; | 108 | return -EINVAL; |
109 | pr_debug("handling %s with %p\n", param, | 109 | pr_debug("handling %s with %p\n", param, |
110 | params[i].ops->set); | 110 | params[i].ops->set); |
@@ -318,7 +318,7 @@ int param_get_bool(char *buffer, const struct kernel_param *kp) | |||
318 | EXPORT_SYMBOL(param_get_bool); | 318 | EXPORT_SYMBOL(param_get_bool); |
319 | 319 | ||
320 | struct kernel_param_ops param_ops_bool = { | 320 | struct kernel_param_ops param_ops_bool = { |
321 | .flags = KERNEL_PARAM_FL_NOARG, | 321 | .flags = KERNEL_PARAM_OPS_FL_NOARG, |
322 | .set = param_set_bool, | 322 | .set = param_set_bool, |
323 | .get = param_get_bool, | 323 | .get = param_get_bool, |
324 | }; | 324 | }; |
@@ -369,7 +369,7 @@ int param_set_bint(const char *val, const struct kernel_param *kp) | |||
369 | EXPORT_SYMBOL(param_set_bint); | 369 | EXPORT_SYMBOL(param_set_bint); |
370 | 370 | ||
371 | struct kernel_param_ops param_ops_bint = { | 371 | struct kernel_param_ops param_ops_bint = { |
372 | .flags = KERNEL_PARAM_FL_NOARG, | 372 | .flags = KERNEL_PARAM_OPS_FL_NOARG, |
373 | .set = param_set_bint, | 373 | .set = param_set_bint, |
374 | .get = param_get_int, | 374 | .get = param_get_int, |
375 | }; | 375 | }; |
diff --git a/security/apparmor/lsm.c b/security/apparmor/lsm.c index 998100093332..65ca451a764d 100644 --- a/security/apparmor/lsm.c +++ b/security/apparmor/lsm.c | |||
@@ -668,7 +668,7 @@ static int param_set_aabool(const char *val, const struct kernel_param *kp); | |||
668 | static int param_get_aabool(char *buffer, const struct kernel_param *kp); | 668 | static int param_get_aabool(char *buffer, const struct kernel_param *kp); |
669 | #define param_check_aabool param_check_bool | 669 | #define param_check_aabool param_check_bool |
670 | static struct kernel_param_ops param_ops_aabool = { | 670 | static struct kernel_param_ops param_ops_aabool = { |
671 | .flags = KERNEL_PARAM_FL_NOARG, | 671 | .flags = KERNEL_PARAM_OPS_FL_NOARG, |
672 | .set = param_set_aabool, | 672 | .set = param_set_aabool, |
673 | .get = param_get_aabool | 673 | .get = param_get_aabool |
674 | }; | 674 | }; |
@@ -685,7 +685,7 @@ static int param_set_aalockpolicy(const char *val, const struct kernel_param *kp | |||
685 | static int param_get_aalockpolicy(char *buffer, const struct kernel_param *kp); | 685 | static int param_get_aalockpolicy(char *buffer, const struct kernel_param *kp); |
686 | #define param_check_aalockpolicy param_check_bool | 686 | #define param_check_aalockpolicy param_check_bool |
687 | static struct kernel_param_ops param_ops_aalockpolicy = { | 687 | static struct kernel_param_ops param_ops_aalockpolicy = { |
688 | .flags = KERNEL_PARAM_FL_NOARG, | 688 | .flags = KERNEL_PARAM_OPS_FL_NOARG, |
689 | .set = param_set_aalockpolicy, | 689 | .set = param_set_aalockpolicy, |
690 | .get = param_get_aalockpolicy | 690 | .get = param_get_aalockpolicy |
691 | }; | 691 | }; |