diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2017-09-13 14:28:19 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2017-09-13 14:28:19 -0400 |
| commit | 4791bcccf8ce02c2bf6f9dfbc328a3a46d9e9569 (patch) | |
| tree | 5224a477d2c8b81acf6798f719cf302de71a9c96 /kernel | |
| parent | 3882a734c19b3cd7feb9e30e1dbd8ae54ac0905a (diff) | |
| parent | 0bf8bf50eddc7511b52461bae798cbfaa0157a34 (diff) | |
Merge tag 'modules-for-v4.14' of git://git.kernel.org/pub/scm/linux/kernel/git/jeyu/linux
Pull modules updates from Jessica Yu:
"Summary of modules changes for the 4.14 merge window:
- minor code cleanups and fixes
- modpost: avoid building modules that have names that exceed the
size of the name field in struct module"
* tag 'modules-for-v4.14' of git://git.kernel.org/pub/scm/linux/kernel/git/jeyu/linux:
module: Remove const attribute from alias for MODULE_DEVICE_TABLE
module: fix ddebug_remove_module()
modpost: abort if module name is too long
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/module.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/kernel/module.c b/kernel/module.c index 40f983cbea81..de66ec825992 100644 --- a/kernel/module.c +++ b/kernel/module.c | |||
| @@ -2707,21 +2707,21 @@ static void add_kallsyms(struct module *mod, const struct load_info *info) | |||
| 2707 | } | 2707 | } |
| 2708 | #endif /* CONFIG_KALLSYMS */ | 2708 | #endif /* CONFIG_KALLSYMS */ |
| 2709 | 2709 | ||
| 2710 | static void dynamic_debug_setup(struct _ddebug *debug, unsigned int num) | 2710 | static void dynamic_debug_setup(struct module *mod, struct _ddebug *debug, unsigned int num) |
| 2711 | { | 2711 | { |
| 2712 | if (!debug) | 2712 | if (!debug) |
| 2713 | return; | 2713 | return; |
| 2714 | #ifdef CONFIG_DYNAMIC_DEBUG | 2714 | #ifdef CONFIG_DYNAMIC_DEBUG |
| 2715 | if (ddebug_add_module(debug, num, debug->modname)) | 2715 | if (ddebug_add_module(debug, num, mod->name)) |
| 2716 | pr_err("dynamic debug error adding module: %s\n", | 2716 | pr_err("dynamic debug error adding module: %s\n", |
| 2717 | debug->modname); | 2717 | debug->modname); |
| 2718 | #endif | 2718 | #endif |
| 2719 | } | 2719 | } |
| 2720 | 2720 | ||
| 2721 | static void dynamic_debug_remove(struct _ddebug *debug) | 2721 | static void dynamic_debug_remove(struct module *mod, struct _ddebug *debug) |
| 2722 | { | 2722 | { |
| 2723 | if (debug) | 2723 | if (debug) |
| 2724 | ddebug_remove_module(debug->modname); | 2724 | ddebug_remove_module(mod->name); |
| 2725 | } | 2725 | } |
| 2726 | 2726 | ||
| 2727 | void * __weak module_alloc(unsigned long size) | 2727 | void * __weak module_alloc(unsigned long size) |
| @@ -3715,7 +3715,7 @@ static int load_module(struct load_info *info, const char __user *uargs, | |||
| 3715 | goto free_arch_cleanup; | 3715 | goto free_arch_cleanup; |
| 3716 | } | 3716 | } |
| 3717 | 3717 | ||
| 3718 | dynamic_debug_setup(info->debug, info->num_debug); | 3718 | dynamic_debug_setup(mod, info->debug, info->num_debug); |
| 3719 | 3719 | ||
| 3720 | /* Ftrace init must be called in the MODULE_STATE_UNFORMED state */ | 3720 | /* Ftrace init must be called in the MODULE_STATE_UNFORMED state */ |
| 3721 | ftrace_module_init(mod); | 3721 | ftrace_module_init(mod); |
| @@ -3779,7 +3779,7 @@ static int load_module(struct load_info *info, const char __user *uargs, | |||
| 3779 | module_disable_nx(mod); | 3779 | module_disable_nx(mod); |
| 3780 | 3780 | ||
| 3781 | ddebug_cleanup: | 3781 | ddebug_cleanup: |
| 3782 | dynamic_debug_remove(info->debug); | 3782 | dynamic_debug_remove(mod, info->debug); |
| 3783 | synchronize_sched(); | 3783 | synchronize_sched(); |
| 3784 | kfree(mod->args); | 3784 | kfree(mod->args); |
| 3785 | free_arch_cleanup: | 3785 | free_arch_cleanup: |
