123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193 |
- From: Felix Fietkau <nbd@nbd.name>
- Subject: [PATCH] build: add a hack for removing non-essential module info
- Signed-off-by: Felix Fietkau <nbd@nbd.name>
- ---
- --- a/include/linux/module.h
- +++ b/include/linux/module.h
- @@ -169,6 +169,7 @@ void trim_init_extable(struct module *m)
-
- /* Generic info of form tag = "info" */
- #define MODULE_INFO(tag, info) __MODULE_INFO(tag, tag, info)
- +#define MODULE_INFO_STRIP(tag, info) __MODULE_INFO_STRIP(tag, tag, info)
-
- /* For userspace: you can also call me... */
- #define MODULE_ALIAS(_alias) MODULE_INFO(alias, _alias)
- @@ -212,12 +213,12 @@ void trim_init_extable(struct module *m)
- * Author(s), use "Name <email>" or just "Name", for multiple
- * authors use multiple MODULE_AUTHOR() statements/lines.
- */
- -#define MODULE_AUTHOR(_author) MODULE_INFO(author, _author)
- +#define MODULE_AUTHOR(_author) MODULE_INFO_STRIP(author, _author)
-
- /* What your module does. */
- -#define MODULE_DESCRIPTION(_description) MODULE_INFO(description, _description)
- +#define MODULE_DESCRIPTION(_description) MODULE_INFO_STRIP(description, _description)
-
- -#ifdef MODULE
- +#if defined(MODULE) && !defined(CONFIG_MODULE_STRIPPED)
- /* Creates an alias so file2alias.c can find device table. */
- #define MODULE_DEVICE_TABLE(type, name) \
- extern const typeof(name) __mod_##type##__##name##_device_table \
- @@ -244,7 +245,9 @@ extern const typeof(name) __mod_##type##
- */
-
- #if defined(MODULE) || !defined(CONFIG_SYSFS)
- -#define MODULE_VERSION(_version) MODULE_INFO(version, _version)
- +#define MODULE_VERSION(_version) MODULE_INFO_STRIP(version, _version)
- +#elif defined(CONFIG_MODULE_STRIPPED)
- +#define MODULE_VERSION(_version) __MODULE_INFO_DISABLED(version)
- #else
- #define MODULE_VERSION(_version) \
- static struct module_version_attribute ___modver_attr = { \
- @@ -266,7 +269,7 @@ extern const typeof(name) __mod_##type##
- /* Optional firmware file (or files) needed by the module
- * format is simply firmware file name. Multiple firmware
- * files require multiple MODULE_FIRMWARE() specifiers */
- -#define MODULE_FIRMWARE(_firmware) MODULE_INFO(firmware, _firmware)
- +#define MODULE_FIRMWARE(_firmware) MODULE_INFO_STRIP(firmware, _firmware)
-
- /* Given an address, look for it in the exception tables */
- const struct exception_table_entry *search_exception_tables(unsigned long add);
- --- a/include/linux/moduleparam.h
- +++ b/include/linux/moduleparam.h
- @@ -16,6 +16,16 @@
- /* Chosen so that structs with an unsigned long line up. */
- #define MAX_PARAM_PREFIX_LEN (64 - sizeof(unsigned long))
-
- +/* This struct is here for syntactic coherency, it is not used */
- +#define __MODULE_INFO_DISABLED(name) \
- + struct __UNIQUE_ID(name) {}
- +
- +#ifdef CONFIG_MODULE_STRIPPED
- +#define __MODULE_INFO_STRIP(tag, name, info) __MODULE_INFO_DISABLED(name)
- +#else
- +#define __MODULE_INFO_STRIP(tag, name, info) __MODULE_INFO(tag, name, info)
- +#endif
- +
- #ifdef MODULE
- #define __MODULE_INFO(tag, name, info) \
- static const char __UNIQUE_ID(name)[] \
- @@ -23,8 +33,7 @@ static const char __UNIQUE_ID(name)[]
- = __stringify(tag) "=" info
- #else /* !MODULE */
- /* This struct is here for syntactic coherency, it is not used */
- -#define __MODULE_INFO(tag, name, info) \
- - struct __UNIQUE_ID(name) {}
- +#define __MODULE_INFO(tag, name, info) __MODULE_INFO_DISABLED(name)
- #endif
- #define __MODULE_PARM_TYPE(name, _type) \
- __MODULE_INFO(parmtype, name##type, #name ":" _type)
- @@ -32,7 +41,7 @@ static const char __UNIQUE_ID(name)[]
- /* One for each parameter, describing how to use it. Some files do
- multiple of these per line, so can't just use MODULE_INFO. */
- #define MODULE_PARM_DESC(_parm, desc) \
- - __MODULE_INFO(parm, _parm, #_parm ":" desc)
- + __MODULE_INFO_STRIP(parm, _parm, #_parm ":" desc)
-
- struct kernel_param;
-
- --- a/init/Kconfig
- +++ b/init/Kconfig
- @@ -2046,6 +2046,13 @@ config MODULE_COMPRESS_XZ
-
- endchoice
-
- +config MODULE_STRIPPED
- + bool "Reduce module size"
- + depends on MODULES
- + help
- + Remove module parameter descriptions, author info, version, aliases,
- + device tables, etc.
- +
- endif # MODULES
-
- config MODULES_TREE_LOOKUP
- --- a/kernel/module.c
- +++ b/kernel/module.c
- @@ -2880,6 +2880,7 @@ static void check_modinfo_retpoline(stru
-
- static int check_modinfo(struct module *mod, struct load_info *info, int flags)
- {
- +#ifndef CONFIG_MODULE_STRIPPED
- const char *modmagic = get_modinfo(info, "vermagic");
- int err;
-
- @@ -2911,6 +2912,7 @@ static int check_modinfo(struct module *
- pr_warn("%s: module is from the staging directory, the quality "
- "is unknown, you have been warned.\n", mod->name);
- }
- +#endif
-
- /* Set up license info based on the info section */
- set_license(mod, get_modinfo(info, "license"));
- --- a/scripts/mod/modpost.c
- +++ b/scripts/mod/modpost.c
- @@ -1966,7 +1966,9 @@ static void read_symbols(char *modname)
- symname = remove_dot(info.strtab + sym->st_name);
-
- handle_modversions(mod, &info, sym, symname);
- +#ifndef CONFIG_MODULE_STRIPPED
- handle_moddevtable(mod, &info, sym, symname);
- +#endif
- }
- if (!is_vmlinux(modname) ||
- (is_vmlinux(modname) && vmlinux_section_warnings))
- @@ -2110,7 +2112,9 @@ static void add_header(struct buffer *b,
- buf_printf(b, "#include <linux/vermagic.h>\n");
- buf_printf(b, "#include <linux/compiler.h>\n");
- buf_printf(b, "\n");
- +#ifndef CONFIG_MODULE_STRIPPED
- buf_printf(b, "MODULE_INFO(vermagic, VERMAGIC_STRING);\n");
- +#endif
- buf_printf(b, "\n");
- buf_printf(b, "__visible struct module __this_module\n");
- buf_printf(b, "__attribute__((section(\".gnu.linkonce.this_module\"))) = {\n");
- @@ -2127,8 +2131,10 @@ static void add_header(struct buffer *b,
-
- static void add_intree_flag(struct buffer *b, int is_intree)
- {
- +#ifndef CONFIG_MODULE_STRIPPED
- if (is_intree)
- buf_printf(b, "\nMODULE_INFO(intree, \"Y\");\n");
- +#endif
- }
-
- /* Cannot check for assembler */
- @@ -2141,10 +2147,12 @@ static void add_retpoline(struct buffer
-
- static void add_staging_flag(struct buffer *b, const char *name)
- {
- +#ifndef CONFIG_MODULE_STRIPPED
- static const char *staging_dir = "drivers/staging";
-
- if (strncmp(staging_dir, name, strlen(staging_dir)) == 0)
- buf_printf(b, "\nMODULE_INFO(staging, \"Y\");\n");
- +#endif
- }
-
- /* In kernel, this size is defined in linux/module.h;
- @@ -2248,11 +2256,13 @@ static void add_depends(struct buffer *b
-
- static void add_srcversion(struct buffer *b, struct module *mod)
- {
- +#ifndef CONFIG_MODULE_STRIPPED
- if (mod->srcversion[0]) {
- buf_printf(b, "\n");
- buf_printf(b, "MODULE_INFO(srcversion, \"%s\");\n",
- mod->srcversion);
- }
- +#endif
- }
-
- static void write_if_changed(struct buffer *b, const char *fname)
- @@ -2487,7 +2497,9 @@ int main(int argc, char **argv)
- add_staging_flag(&buf, mod->name);
- err |= add_versions(&buf, mod);
- add_depends(&buf, mod, modules);
- +#ifndef CONFIG_MODULE_STRIPPED
- add_moddevtable(&buf, mod);
- +#endif
- add_srcversion(&buf, mod);
-
- sprintf(fname, "%s.mod.c", mod->name);
|