diff options
author | Mark Rustad <mark.d.rustad@intel.com> | 2014-09-10 20:17:23 -0400 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2014-09-10 20:29:25 -0400 |
commit | 184c3fc3f52fb75800deb76deffb70907d1f76ea (patch) | |
tree | 79da9107a07645399863940b4de2e63286f778b5 /include/linux/moduleparam.h | |
parent | beb50df39e91745604ce3cb9dc6a503f39f4383d (diff) |
moduleparam: Resolve missing-field-initializer warning
Resolve a missing-field-initializer warning, that is produced
by every reference to module_param_call, by using designated
initialization for the first field. That is enough to silence
the complaint.
The message is only seen when doing a W=2 build. I happened to be using gcc
4.8.3, but I think most versions would produce the warning when it is
enabled. It can either be silenced by using even a single designated
initializer as I did here, or providing values for all of the fields. Because
of the number of references to the macro, this change silences many warnings
in W=2 builds.
One instance of the full warning message looks like this:
/home/share/git/nn-mdr/include/linux/moduleparam.h:198:16: warning: missing
initializer for field ‘free’ of ‘struct kernel_param_ops’
[-Wmissing-field-initializers]
static struct kernel_param_ops __param_ops_##name = \
^
/home/share/git/nn-mdr/fs/fuse/inode.c:35:1: note: in expansion of macro
‘module_param_call’
module_param_call(max_user_bgreq, set_global_limit, param_get_uint,
^
/home/share/git/nn-mdr/include/linux/moduleparam.h:56:9: note: ‘free’
declared here
void (*free)(void *arg);
Signed-off-by: Mark Rustad <mark.d.rustad@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'include/linux/moduleparam.h')
-rw-r--r-- | include/linux/moduleparam.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/moduleparam.h b/include/linux/moduleparam.h index 593501996574..b43f4752304e 100644 --- a/include/linux/moduleparam.h +++ b/include/linux/moduleparam.h | |||
@@ -224,7 +224,7 @@ struct kparam_array | |||
224 | /* Obsolete - use module_param_cb() */ | 224 | /* Obsolete - use module_param_cb() */ |
225 | #define module_param_call(name, set, get, arg, perm) \ | 225 | #define module_param_call(name, set, get, arg, perm) \ |
226 | static struct kernel_param_ops __param_ops_##name = \ | 226 | static struct kernel_param_ops __param_ops_##name = \ |
227 | { 0, (void *)set, (void *)get }; \ | 227 | { .flags = 0, (void *)set, (void *)get }; \ |
228 | __module_param_call(MODULE_PARAM_PREFIX, \ | 228 | __module_param_call(MODULE_PARAM_PREFIX, \ |
229 | name, &__param_ops_##name, arg, \ | 229 | name, &__param_ops_##name, arg, \ |
230 | (perm) + sizeof(__check_old_set_param(set))*0, -1, 0) | 230 | (perm) + sizeof(__check_old_set_param(set))*0, -1, 0) |